Re: [Pharo-dev] About icon: and iconNamed:

[stepharo]

So I'm trying to think

menuOn: aBuilder
    "Specify the menu used when writing text."
    <contextMenu>
    <RubLineNumberMenu>
    (aBuilder item: #'Find...' translated)
        keyText: 'f';
        selector: #find;
        icon: (Smalltalk ui icons iconNamed: #smallFindIcon)

====>

menuOn: aBuilder
    "Specify the menu used when writing text."
    <contextMenu>
    <RubLineNumberMenu>
    (aBuilder item: #'Find...' translated)
        keyText: 'f';
        selector: #find;
        icon: #smallFindIcon


Now I do not really like to pass a symbol.

icon: aSymbol

    aSymbol asIcon

But we cannot plug a provider

So for now I would like to continue


Stef

what if you want to have side by side two different themes to compare the best icons

choices?

This is a valid concern, but I do not understand how this would work if the only thing you pass in iconName: is just one symbol:

act: aBlock iconName: *aSymbol* entitled: aString

So there needs to be a lookup. This lookup will depend on the current icon theme. So, you could open on window with one theme, switch the theme and then open another window. Or did I misunderstand something?

My idea was that we leave the public API of Glamour to be:

act: aBlock icon: *aSymbolOrForm* entitled: aString

and then if you want to play with things, you can also explicitly pass one icon or another (beside relying on the default lookup behavior).

What do you think?

Doru

In this way we do not have to change the external interface, and only the internal implementation has to send an “asIcon” before using it?

Cheers,
Doru

On Oct 12, 2016, at 10:33 PM, stepharo <steph...@free.fr <mailto:steph...@free.fr>> wrote:

I'm adding

act: aBlock iconName: aSymbol entitled: aString
   self act: aBlock icon: (self iconNamed: aSymbol) entitled: aString

and

act: aBlock iconName: aSymbol on: aCharacter entitled: aString

self act: aBlock icon: (self iconNamed: aSymbol) on: aCharacter entitled: aString

and others ....

--
www.tudorgirba.com <http://www.tudorgirba.com>
www.feenk.com

"Every thing has its own flow."

--
www.tudorgirba.com <http://www.tudorgirba.com>
www.feenk.com

"We are all great at making mistakes."