We know. It was not my intention. I was slowly trying to identify what object (and not models) need it.
Denis pushed it and now we will remove it.
The idea is to
- place it in Model
- I introduced DisplayableObject as a superclass of the objects that still do icon: named and others.
Now you cannot imagine how many plain objects (yes this is terrible to me) are receiver of it.
Stef
> On 4 Oct 2019, at 15:36, Sean P. DeNigris <
[hidden email]> wrote:
>
> ci-pharo-ci-jenkins2 wrote
>> "2442-Commander-should-not-use-Smalltalk-ui-icons-but-self-iconNamed"
>
> I was surprised to see `Object>>#iconNamed:`. While it certainly reduces the
> duplication of `Smalltalk ui icons`, we have seemed extremely reluctant in
> the past to add to Object's inevitable bloat. I'm curious what made this one
> in particular worth it?
>
>
>
> -----
> Cheers,
> Sean
> --
> Sent from:
http://forum.world.st/Pharo-Smalltalk-Developers-f1294837.html>
Locked
