Two ChoiceBox API additions

[Tom Schindl]

Hi Jonathan (now with the list included),

Exactly this Text-Example made me think that the converter you are
setting here is something different!

The "converter" you are passing here is more a kind of label provider
(this term is borrowed from SWT/JFace). Naming it labelConverter btw
still leaves room for an imageConverter (though as discussed in the bug
Richard thought ChoiceBox is not the right UI-Control I'm still not so
sure about that).

I think the term converter should only be used if to* and from* are used
but that's maybe just me.

I'm not sure on this one now but couldn't one think about providing a
label converter to all structured controls (combobox, list, table, ...)
and then it suddenly once more gets a well known concept.

Tom

Am 12.01.12 09:15, schrieb Jonathan Giles:
> The reason why it is named 'converter' is for consistency across the
> API. The intention is that future controls may also have a converter
> that is a StringConverter. Already ComboBox has this API (but of course
> uses both toString(..) and fromString(..), which ChoiceBox does not).
> 
> I can see your thinking behind the name - it is more specifically what
> it is - but at the same time I personally like the improved discovery
> (and lessened mental overhead) of using consistent API naming.
> 
> What do others think?
> 
> -- Jonathan
> 
> On Thursday, 12 January 2012 8:40:41 p.m., Tom Schindl wrote:
>> Am 12.01.12 00:59, schrieb Paru Somashekar:
>>> The controls team is planning to add 2 new APIs to ChoiceBox control.
>>> They are :-
>>>
>>> 1. valueProperty : (getValue(), setValue() and valueProperty())
>>> The value of a ChoiceBox is defined as the selected item in the
>>> ChoiceBox selection model. The valueProperty is synchronized with the
>>> selectedItem, in that when a user sets the value property the selected
>>> item/index is updated accordingly. And when selection changes, the value
>>> property also reflects the change. This property allows for
>>> bi-directional binding of external properties to the ChoiceBox and
>>> updates the selection model accordingly.
>>>
>>> 2. converterProperty : (getConverter(), setConverter(),
>>> converterProperty())
>>
>> Wouldn't it better named labelConverterProperty?
>>
>> Tom
>>


-- 
B e s t S o l u t i o n . a t                        EDV Systemhaus GmbH
------------------------------------------------------------------------
tom schindl                 geschäftsführer/CEO
------------------------------------------------------------------------
eduard-bodem-gasse 5-7/1   A-6020 innsbruck     fax      ++43 512 935833
http://www.BestSolution.at                      phone    ++43 512 935834