On Oct 20, 9:39 am, "Edward K. Ream" <[email protected]> wrote:
> This is a serious bug! The menu item now executes the command bound
> to Alt-A! Very bad.
>
> I am pretty sure I know what is happening: fixing it may be tricky.
The situation is nasty. leoMenu.createMenuEntries calls
c.config.getShortcut
but that will find the old shortcut even if a new shortcut has been
bound to *another* command.
We can't delete the old binding, because the old binding is (probably)
associated with leoSettings.leo, and we will need that binding if we
open another .leo file that *doesn't* override the binding.
It looks like getShortcut is going to have to work harder, so that it
determines the proper priority. This might be good news (because the
fix will be confined to a single spot) or not (because getShortcut
might not have enough information). We shall see...
BTW, getShortcut always takes a single argument, a command name,
although in several places the argument is named "shortcutName". This
"dyslexia" can be fixed easily.
Edward
--
You received this message because you are subscribed to the Google Groups
"leo-editor" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/leo-editor?hl=en.
