Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#18988 closed defect (fixed)

(1.12.2) dojox.charting.plot2d.Pie ignores `radius` and 0 plot values.

Reported by: mwistrand Owned by: dylan
Priority: high Milestone: 1.12.3
Component: Charting Version: 1.12.2
Keywords: Cc:
Blocked By: Blocking:

Description

When rendering a chart with options.label set to true, and with options.labelStyle set to "columns", any options.radius value is ignored. Further, labels for 0 plot values are never rendered, and in some cases result in the following error thrown from ​Pie.js L727:

TypeError: Cannot read property 'left' of undefined

As an example, this error can be generated in dojox/charting/tests/test_pie_smart_label.html by changing the first value (12.1) to 0. With v1.11, it was possible to display labels for empty (y: 0) plot values, and it is still desirable to be able to do so.

Change History (3)

comment:1 Changed 2 years ago by dylan

Component: General → Charting
Milestone: tbd → 1.12.3
Owner: set to dylan
Priority: undecided → high
Status: new → assigned

PR at ​https://github.com/dojo/dojox/pull/266 .

comment:2 Changed 2 years ago by dylans <dylan@…>

Resolution: → fixed
Status: assigned → closed

In fcb7ff6/dojox:

Error: Processor CommitTicketReference failed
Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled?

comment:3 Changed 2 years ago by dylans <dylan@…>

In 884beb8/dojox:

Error: Processor CommitTicketReference failed
Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled?
Note: See TracTickets for help on using tickets.