Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#7428 closed defect (fixed)

Redrawing chart in FF3 causes it to be duplicated below

Reported by: jwcnmr Owned by: Eugene Lazutkin
Priority: high Milestone: 1.3
Component: Charting Version: 1.1.1
Keywords: Chart2D, Firefox 3 Cc:
Blocked By: Blocking:

Description

If you create a simple x-y line Chart2D plot using a button to cause it to be drawn, and click the button again, the chart is redrawn in the same spot in IE, but is drawn again below the previous plot in FF3. I believe that the IE behavior is the expected one since creating a chart with the same ID is supposed to replaced the previous one. In the attached example, a flag is set to switch between two slightly different data sets so you can see the the plot change in IE.

Attachments (2)

test4.htm (2.1 KB) - added by jwcnmr 11 years ago.
Chart2D demo of re-plotting probelm in FF3
test4.html (1.8 KB) - added by Eugene Lazutkin 11 years ago.
the updated example that calls destroy().

Download all attachments as: .zip

Change History (11)

Changed 11 years ago by jwcnmr

Attachment: test4.htm added

Chart2D demo of re-plotting probelm in FF3

comment:1 Changed 11 years ago by Eugene Lazutkin

Milestone: tbdfuture

Moving all open ticketd to the future.

comment:2 Changed 11 years ago by Eugene Lazutkin

Component: DojoX GFXCharting
Status: newassigned

comment:3 Changed 11 years ago by Eugene Lazutkin

Can be related to #7293.

comment:4 Changed 11 years ago by Eugene Lazutkin

Priority: normalhigh

comment:5 Changed 11 years ago by Eugene Lazutkin

#7863 is the pre-requisite for this ticket.

comment:6 Changed 11 years ago by Eugene Lazutkin

There are two problems:

  1. You forgot to call destroy() on the old chart before creating the new one.
  2. There is a bug in dojox.charting that prevents the correct destruction (see #7293).

I am adding the corrected example that calls destroy() when appropriate. The fix for the dojox.charting bug will be committed shortly.

Changed 11 years ago by Eugene Lazutkin

Attachment: test4.html added

the updated example that calls destroy().

comment:7 Changed 11 years ago by Eugene Lazutkin

Resolution: fixed
Status: assignedclosed

(In [15483]) dojox.charting: adding the proper chart destruction, !strict, fixes #7293, fixes #7428.

comment:8 Changed 11 years ago by Eugene Lazutkin

(In [15484]) dojox.charting: adding the proper destroy() method to the widget, !strict, refs #7293, refs #7428.

comment:9 Changed 11 years ago by Adam Peller

Milestone: future1.3

batch move of tickets marked 'future' in the 1.3 timeframe

Note: See TracTickets for help on using tickets.