On Mon, 24 Jun 2013 06:41:08 -0700 (PDT)
Fidel Pérez <[email protected]> wrote:

> So with my fix, if you want leo to just make the button, you set the value 
> to true and you are set. It will both dont care about not making a command, 
> and set the text to what you intend.

In order to add only one new parameter instead of two, I'd still favour
the button_text approach, with whatever changes are needed to make the
function accept an empty 'text' argument gracefully.  "One instead of
two", because for the

createIconButton("add braces", command, statline, button_text='{}')

case adding a bool flag would be a second new parameter.  So I'm
proposing that the command be made to work for this case:

createIconButton("", command, statline, button_text='⅏±Ø⅏') for your use.

I know having to give an empty parameter is putzy, although if you
think of the 'text' parameter as 'command name' then it's not so bad,
empty command name indicating no need to create a command.

Things are always a little odd when you're trying to keep something
backwards compatible, which is essential in this case.  Starting from
scratch you might get something cleaner.

Cheers -Terry (leaving on vacation now :-)

-- 
You received this message because you are subscribed to the Google Groups 
"leo-editor" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/leo-editor.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to