> On 28-10-2017, at 1:17 PM, tim Rowledge <tim at rowledge.org> wrote:
> 
> 
>> On 28-10-2017, at 12:55 AM, Marcel Taeumel <marcel.taeumel at hpi.de> wrote:
>>  Please do not forget to add #windowTitle to Model, which I would prefer over #labelString. :) #labelString should be deprecated.
> 
> Hmm, well I agree that is a better name, but there is going to be some fallout. Take a look at SystemWindow>update: and use of #relabel. And then the senders of #relabel, and the implementors of labelString. That’s quite a few bits of code needing touching, so do please feel free to touch them while I keep working on these new dialogs.

Urk. Well it turns out there is a *load* more fallout than I’d expected. The usage of label is so widespread for window titles as well as buttons, menus, debugger logs and so on, that it makes for very long list of code to examine and I’m currently guessing at over a hundred methods to change, all to swap a simple term.

Those windows built via ToolBuilder can be told to use any message you like, so there’s no problem there BUT even in ToolBuilder the concept is labelled as… err… label. I also noticed a number of places where ‘changed: #labelString’ is sent and yet no plausible implementations of any use of that to actually do something. My guess is a change in the past that moved to #relabel and didn’t completely update everywhere required. It certainly pre-dates 2.8 and I really can’t be bothered to search back any further. The point is that there’s a number of places that appear to expect to update the title bar and don’t. Sigh.

Given all that I’m going to revert the deprecation - so we’ll have a deprecated deprecation to deprecate later - of labelString and a couple of related method changes. It’s far more turmoil than is warranted for a simple nomenclature swap in my view.


tim
--
tim Rowledge; tim at rowledge.org; http://www.rowledge.org/tim
Cloister: a pretentious clam