Opened 14 years ago
Closed 14 years ago
#2517 closed enhancement (wontfix)
Possible efficiency improvement for chart lines in VML
Reported by: | Owned by: | Tom Trenka | |
---|---|---|---|
Priority: | high | Milestone: | |
Component: | Charting | Version: | 0.4.1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
In dojo/src/charging/vml/Plotters.js, line 536 (the Line: plotter) there is:
if (i==0){
cmd.push("m"); cmd.push(x+","+y);
}else{
cmd.push("l"); cmd.push(x+","+y);
}
My reading of the VML spec suggests that the following is equally correct, and results in a shorter command set. It works in IE7.
if (i==0){
cmd.push("m");
}else if (i==1){
cmd.push("l");
}else{
cmd.push(x+","+y);
Change History (2)
comment:1 Changed 14 years ago by
comment:2 Changed 14 years ago by
Resolution: | → wontfix |
---|---|
Status: | new → closed |
Going to close this; porting charting to use dojox.gfx, and that efficiency is abstracted there.
Note: See
TracTickets for help on using
tickets.
Oops - typo. The last else{ should not be there. The cmd.push(x+","+y) at the end should be unconditional.