PDA

View Full Version : [OPEN] ExtJS 6.2.1 classic: missing pie label display "inside" setting in docs



fabricio.object
20 Jan 2017, 1:45 AM
Hello! This one caught me off-guard! But we had an example that relied on this setting that broke as soon as we got rid of the "undocumented" setting, and it really works great with that missing one!

On Ext.chart.series.sprite.PieSlice definition, line 234 (http://docs.sencha.com/extjs/6.2.1/classic/src/PieSlice.js.html#line234), it takes in consideration the possible value of label.display = "inside".

If we look at Sencha Documentation on ExtJS 6.2.1's Ext.chart.series.Pie (http://docs.sencha.com/extjs/6.2.1/classic/Ext.chart.series.Pie.html#cfg-label) (and possibly older versions), we can read the following for the "display" part of the config:



display : String
Specifies the presence and position of the labels. The possible values depend on the series type. For Line and Scatter series: 'under' | 'over' | 'rotate'. For Bar and 3D Bar series: 'insideStart' | 'insideEnd' | 'outside'. For Pie series: 'outside' | 'rotate' | 'horizontal' | 'vertical'. Area, Radar and Candlestick series don't support labels. For Area and Radar series please consider using tooltips instead. 3D Pie series currently always display labels 'outside'. For all series: 'none' hides the labels.
Default value: 'none'.


Closing in to the Pie part of the setting, we have:



For Pie series: 'outside' | 'rotate' | 'horizontal' | 'vertical'.


So, no mention for the "inside" setting, that's at least supported by PieSlice linked above. And it really made a difference in an example we have here (http://examples4.ext.net/#/Chart/Combination/Infographic/), possibly ported from Sencha examples to our platform.

So, please add back (or at once) the setting to docs so other people can know about it and we don't remove it again by mistake. We could see the feature we took off for no reason, other people might just miss it for good. :)

greg.barry
25 Jan 2017, 9:18 PM
Thanks for the report! I have opened a bug in our bug tracker.