Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#18 NEdit's accelerator keys not working

closed-fixed
nobody
None
5
2009-11-28
2005-07-04
Joerg Fischer
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