[API Review]: RT-28148 - Add createSymbols Property to AreaChart / StackedAreaChart

[Jasper Potts]

I feel like adding "createSymbols" boolean property to Area, & StackedArea charts makes sense to match LineChart.

I don't like the idea of adding API to the series object on XYChart as that is more generic and used by many chart types. If the user needs fine grain control of symbols like in RT-21539 they can turn auto symbol generation off with "createSymbols = false" then create their own symbol nodes for the cases when they do want them.

Thanks

Jasper

On Mar 8, 2013, at 10:08 AM, Paru Somashekar <parvathi.somashekar at oracle.com> wrote:

> There is a request for adding the createSymbols flag at the XYChart's Series level instead of on Chart.
> JIRA : http://javafx-jira.kenai.com/browse/RT-21539
> 
> I think it might be a good idea to add it at the Series level so that one can turn ON / OFF symbols per Series rather than for all the series of a chart.
> The API could continue to be the same - except live at the Series level. Each chart can then create symbols for each of its Series only if this flag is turned on.
> This might however not make much sense for Scatter, Bubble and BarCharts.
> What do you think Jasper?
> 
> -Paru.
> 
> On 3/8/13 4:12 AM, Sven Reimers wrote:
>> Hi all
>> 
>> AreaChart and StackedAreaChart are missing an API to simply disable the
>> creation of symbols. At the moment this is only possible by complex css
>> style acrobatics. LineChart on the other hand already provides a simple way
>> to do this. The proposed tweak takes the existing API from LineChart and
>> adds this to AreaChart and StackedAreaChart.
>> 
>> Desired API change:
>> 
>>  /**
>>      * Indicates whether symbols for data points will be created or not.
>>      *
>>      * @return true if symbols for data points will be created and false
>> otherwise.
>>      */
>>     public final boolean getCreateSymbols() { return
>> createSymbols.getValue(); }
>>     public final void setCreateSymbols(boolean value) {
>> createSymbols.setValue(value); }
>>     public final BooleanProperty createSymbolsProperty() { return
>> createSymbols; }
>> 
>> Desired additional CSS property (incldues additional StyleableProperty):
>> 
>> -fx-create-symbols
>> 
>> 
>> JIRA:
>> http://javafx-jira.kenai.com/browse/RT-28148
>> 
>> Thank
>> 
>> -Sven
>> 
>> P.S. An updated patch will hopefully be available there too.
>> 
>