Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#6714 closed enhancement (fixed)

Add removeXXX() methods

Reported by: Eugene Lazutkin Owned by: Eugene Lazutkin
Priority: high Milestone: 1.2.2
Component: Charting Version: 1.1.0
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by Eugene Lazutkin)

Add removeSeries(), removeAxis(), and removePlot() methods to support truly dynamic charts.

Attachments (1)

removeSeriesByName.js.txt (1.8 KB) - added by enzo 11 years ago.
[patch] [cla] Uploaded by Enzo Michelangeli

Download all attachments as: .zip

Change History (13)

comment:1 Changed 12 years ago by Eugene Lazutkin

Component: GeneralCharting
Status: newassigned

comment:2 Changed 11 years ago by enzo

I uploaded a removeSeriesByName() function that appears to work for me, but I'm not sure it performs all the necessary housekeeping (it should be reviewed by someone who knows the architecture of Chart2d and gfx).

Changed 11 years ago by enzo

Attachment: removeSeriesByName.js.txt added

[patch] [cla] Uploaded by Enzo Michelangeli

comment:3 Changed 11 years ago by Eugene Lazutkin

Milestone: 1.2future

Moving all open ticketd to the future.

comment:4 Changed 11 years ago by Eugene Lazutkin

Description: modified (diff)
Milestone: future1.2.1
Summary: Add removeSeries() method.Add removeXXX() methods

comment:5 Changed 11 years ago by Eugene Lazutkin

Priority: normalhigh

comment:6 Changed 11 years ago by Adam Peller

this should be moved to 1.3

comment:7 Changed 11 years ago by Eugene Lazutkin

This functionality is easy to add, and frequently asked. I don't see why it should be delayed.

comment:8 Changed 11 years ago by Adam Peller

that would violate our policy on dot releases. We're trying to keep dot releases minimal so that people can easily inspect the diffs and aren't afraid of taking them. It's a slippery slope if we start making exceptions, but we can certainly discuss it on list or in #dojo-meeting.

comment:9 Changed 11 years ago by Eugene Lazutkin

Resolution: fixed
Status: assignedclosed

The trac is broken. It doesn't show any changesets related to this ticket. Nevertheless, I close this ticket as fixed.

comment:10 Changed 11 years ago by Eugene Lazutkin

Relevant changesets: [15465], [15464], [15463], [15462], [15461], [15460], [15459].

comment:11 Changed 11 years ago by Eugene Lazutkin

(In [15486]) dojox.charting: adding dependent dynamic features to the legend and updating a test, !strict, refs #7234, refs #6982, refs #6714, fixes #7898.

comment:12 Changed 11 years ago by Eugene Lazutkin

Milestone: 1.2.11.2.2
Note: See TracTickets for help on using tickets.