Opened 11 years ago
Closed 11 years ago
#10856 closed enhancement (fixed)
[patch][ccla] Charting Animation for Zoom/Grow
Reported by: | ruanqi | Owned by: | Eugene Lazutkin |
---|---|---|---|
Priority: | high | Milestone: | 1.5 |
Component: | Charting | Version: | 1.4.0 |
Keywords: | charting, animation | Cc: | [email protected]…, [email protected]…, [email protected]… |
Blocked By: | Blocking: |
Description (last modified by )
FYI. this enhancement enable chart(area,lines,markers...) grow/zoom animation.
1.chart grows from the bottom when it render, set "animation" to turn it on. e.g.
chart.addPlot("default", {type: "Areas",animation:{duration:2000}}).render();
2.chart zooms by the scale&offset of axes, add a argument to turn it on. e.g.
chart.setWindow(scaleX, scaleY, offsetX, offsetY, {duration:1800}).render();
This patch also implements a "zoomIn" method in Chart2D, it supports chart zooming to a certain axis range. e.g.
chart.zoomIn("x",[50,80]);//chart zooms to 50-80 range on axis x
demo: test_axisZoomControl.html in the attachment.
Attachments (4)
Change History (25)
comment:1 Changed 11 years ago by
Status: | new → assigned |
---|
comment:2 follow-up: 3 Changed 11 years ago by
BTW, the demo link is behind the firewall --- can't use it.
comment:3 Changed 11 years ago by
Replying to elazutkin:
BTW, the demo link is behind the firewall --- can't use it.
test_win2d.html in the attachment patch could took as a demo.
Changed 11 years ago by
Attachment: | chart-animate-zoom_20100318.patch added |
---|
Enable chart grow/zoom animation, from ruanqi (IBM, CCLA)
comment:4 Changed 11 years ago by
Description: | modified (diff) |
---|---|
Summary: | [patch] Charting Animation for Zoom/Grow → [patch][ccla] Charting Animation for Zoom/Grow |
Changed 11 years ago by
Attachment: | chart-animate-zoom_20100322.patch added |
---|
updated patch from ruanqi
comment:5 Changed 11 years ago by
Description: | modified (diff) |
---|
comment:6 Changed 11 years ago by
Cc: | [email protected]… [email protected]… added |
---|
comment:7 follow-up: 8 Changed 11 years ago by
This patch clashes with #10898. Need to reconcile. :-(
comment:8 follow-ups: 9 10 Changed 11 years ago by
comment:9 Changed 11 years ago by
comment:10 follow-up: 11 Changed 11 years ago by
comment:11 follow-up: 12 Changed 11 years ago by
[21836] has landed. All plots were updated, and I cannot apply many patches mechanically even if I wanted to do so. Fear not, I'll merge it manually. Of course, if you have time on hands, you can update the patch to the current trunk saving me some time. ;-)
comment:12 Changed 11 years ago by
ok, I'll create a new patch based on the latest dojo trunk and ask adam to help me upload that patch, thank you.
Changed 11 years ago by
Attachment: | chart-animate-zoom_20100414.patch added |
---|
updated patch from zhouxiang
comment:13 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:14 Changed 11 years ago by
Action items:
- Make sure that the animation enhancement API for the default plot matches the same API for the bar chart (or vice-versa). We cannot afford different APIs for the same functionality.
- We need to extend the animation to other plot types. The default plot is arguably the most involved one, the rest are much easier.
comment:15 follow-up: 16 Changed 11 years ago by
Follow your action items, I make the animation works on bar chart and extend the animation to all the 2D plot types except for Pie chart. I've created a new patch for it and asked adam to upload it. Please check... thank you.
comment:16 Changed 11 years ago by
Replying to zhouxiang:
Follow your action items, I make the animation works on bar chart and extend the animation to all the 2D plot types except for Pie chart. I've created a new patch for it and asked adam to upload it. Please check... thank you.
Thank you for the initiative. Bar/column chart already has this functionality and I wanted to make sure that the API is the same.
You should be able to attach patches yourself, if not, you can send them to me directly, so I can do it. Unless, of course, you have some additional reasons to go through Adam.
comment:17 follow-up: 18 Changed 11 years ago by
Hi, elazutkin, IBM has a ossc process we should follow, we can not attach patches ourselves. Any code go outside should go through Adam.
comment:18 Changed 11 years ago by
Replying to zhouxiang:
Hi, elazutkin, IBM has a ossc process we should follow, we can not attach patches ourselves. Any code go outside should go through Adam.
Fair enough. Let's wait for Adam then.
comment:19 Changed 11 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
comment:20 Changed 11 years ago by
test_win2d.html
--- mouse dragging is broken in the test, should be calculated differently.
Changed 11 years ago by
Attachment: | chart-animate-zoom_20100422.patch added |
---|
updated patch from zhouxiang
comment:21 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
I assume your submission is covered by CLA/CCLA (IBM?). If it is so, please add to the summary [CCLA] prefix.