#18 NEdit's accelerator keys not working

closed-fixed
nobody
None
5
2009-11-28
2005-07-04
No

From version 94.0 on, NEdit's accelerator keys aren't
working
anymore.

To reproduce, open nedit with default settings (ie, w/o
preferences file). Create two entries for the macro menu
named A and B, respectivley, with content dialog("A") and
dialog("B"), and with accelerator key Shift+Ctrl+B (or also
Alt+Ctrl+B) for A and Ctrl+B for B.

Hitting Shift+Ctrl+B will display `B', then.

Now, I found that there is a patch in manager.c v1.3
that is intended to fix the Motif-Caps/NumLock-accelerator
bugs. (AFAIK nedit works around these problems.)

Unfortunately, this patch causes the problem described
above.
I built 94.4 with manager.c v1.2 and the problem is away.

Discussion

  • Hans de Goede

    Hans de Goede - 2007-08-30

    Logged In: YES
    user_id=922849
    Originator: NO

    Hi,

    I'm a Fedora developer who is currently doing some lesstif bug smashing, this problem is indeed caused by the changes between 1.2 and 1.3 of manager.c. This patch fixes it while still keeping the make accelkeys work with capslock on behaviour:
    --- lesstif-0.95.0/lib/Xm-2.1/Manager.c.accelkeys 2005-04-02 16:58:11.000
    +++ lesstif-0.95.0/lib/Xm-2.1/Manager.c 2007-08-31 00:19:19.000000000 +0200
    @@ -1764,7 +1764,8 @@ _XmAcceleratorHandler(Widget w, XtPointe
    modifiers" or "these modifiers plus any others" */
    if (/*MGR_KeyboardList(w)[i].eventType == event->type && */
    MGR_KeyboardList(w)[i].key == event->xkey.keycode &&
    - (MGR_KeyboardList(w)[i].modifiers & event->xkey.state) == MGR_Keybo
    + MGR_KeyboardList(w)[i].modifiers ==
    + (event->xkey.state & ~(Mod2Mask | LockMask)) &&
    MGR_KeyboardList(w)[i].component == comp &&
    XtIsManaged(comp))
    {

     
  • Paul Gevers

    Paul Gevers - 2009-09-30

    We've applied your patch to the sources in CVS.

     
  • Paul Gevers

    Paul Gevers - 2009-09-30
    • status: open --> open-fixed
     
  • Paul Gevers

    Paul Gevers - 2009-11-28
    • status: open-fixed --> closed-fixed
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks