Rev 4672 contains the most important part of the fix for bug 879331:
Redefining a key binding breaks menu items with same binding.

**Warning**: all unit tests pass, which is a good test, but serious
problems could remain. Use caution.

Here is the checking log::

QQQQQ
Menu bindings now are completely independent of key bindings.

Still to do: Properly account for key redefinitions.  At present the
wrong shortcut can be shown in menu items.

What I did:

- Eliminated k.masterMenuHandler. It was only called from
masterMenuCallback.

- Changed the arguments to k.masterCommand: arguments are now in
alphabetical order, with all defaulting to None.

- Added the following code to k.masterCommand::

    if commandName and not func:
        func = c.commandsDict.get(commandName)

    This allows commandName to override the stroke argument.

- Refactored createMenuEntries to eliminate sections.

    Should have been done long ago.

- Added traceMenuTable helper.
QQQQQ

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.

Reply via email to