On Jan 28, 11:49 am, "Edward K. Ream" <[email protected]> wrote:
> At last, some real unit tests are complete. I'll be pushing to the trunk > soon. > > This should fix bug 879331: Redefining a key binding breaks menu items > with same binding It appears that the serious part of the bug has been fixed, but there is still a problem with menu accelerators. I just ran the following hand test. In ekr.leo I bound print-settings = ctrl-n. Within ekr.leo, ctrl-n is indeed bound properly to print- settings. More importantly, selecting the File:New command does work as expected--redefining ctrl-n doesn't break the menu's operation. However, the File:New menu item still shows Ctrl+N as the accelerator. I'll attempt a fix today... Btw, a simple change now shows *all* bindings for command in each menu item. I think this is always a good idea, but it can be easily reverted if people have objections... 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.
