We can use text: I should put it back. the point is not that, the point is that each widget should expose its valueHolder and we should get rid of the when...changed: I was probably confused because the refactoring broke in the middle Le 5/3/15 09:39, Nicolai Hess a écrit :
2015-03-04 14:00 GMT+01:00 Sean P. DeNigris <sean@clipperadams.com <mailto:sean@clipperadams.com>>:
What is the reason for this? I searched the lists & fogbugz for "labelmodel text" to no avail. To me, "label label: aString" seems odd. Why is this better than "label text: aString"?
I think the idea behind this was, one common api for all widgets with a label CheckBoxModel /RadioButtonModel /ButtonModel .... all have a label: method.
But of course, one can argue, a label itself has a text.
In general, for refactors like this, where the purpose is not obvious as it would be for a bug fix or new feature, it would be good to somehow link somewhere for more info. Maybe the deprecation message, since by definition will be deleted in the next verison, would be a good place to put an issue number or url to a discussion. Also, might I humbly suggest having a native English speaker review changes where the intent is to increase "clarity" so we don't have another #ensureCreateDirectory incident ;) j/k... But a grammar/idiom review would be super quick/easy. Maybe a tag or status in fogbugz would help?
----- Cheers, Sean -- View this message in context: http://forum.world.st/LabelModel-text-Deprecation-tp4809424.html Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.