Bugs item #1799906, was opened at 2007-09-21 19:37
Message generated for change (Comment added) made by duaneclark
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=111005&aid=1799906&group_id=11005

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Program
Group: None
Status: Open
Resolution: Accepted
Priority: 5
Private: No
Submitted By: Duane Clark (duaneclark)
Assigned to: Nobody/Anonymous (nobody)
Summary: Make mouse wheel scrolling compatible with lesstif

Initial Comment:
In the various dialogs (and in particular the file selection dialog is where I 
came across this), nedit implements mouse wheel scrolling by overriding the 
translations for various forms of <Btn4Down>,<Btn4Up>. Lesstif apparently 
decided to add builtin support for mouse wheel scrolling, but they did it by 
adding the translations to the variations of <Btn4Down>. The effect is that 
using the scroll wheel causes the list to scroll by the number of lines defined 
by lesstif, plus the number of lines defined by nedit. Since lesstif by default 
scrolls by a page minus one line, this makes it impossible to use the mouse 
wheel.

Changing nedit to use just the variations of <Btn4Down> causes nedit to 
completely override the lesstif defaults, returning normal operation.

I am running this on Fedora 7, with lesstif 0.95.0-20, which actually seems to 
work pretty good with nedit.

Index: util/misc.c
===================================================================
RCS file: /cvsroot/nedit/nedit/util/misc.c,v
retrieving revision 1.86
diff -u -r1.86 misc.c
--- util/misc.c 20 Jul 2007 16:09:19 -0000      1.86
+++ util/misc.c 21 Sep 2007 19:28:21 -0000
@@ -2070,12 +2070,12 @@
     if (XmIsScrolledWindow(XtParent(w))) 
     {
         static const char scrollTranslations[] =
-           "Shift<Btn4Down>,<Btn4Up>: scrolled-window-scroll-up(1)\n"
-           "Shift<Btn5Down>,<Btn5Up>: scrolled-window-scroll-down(1)\n"
-           "Ctrl<Btn4Down>,<Btn4Up>:  scrolled-window-page-up()\n"
-           "Ctrl<Btn5Down>,<Btn5Up>:  scrolled-window-page-down()\n"
-           "<Btn4Down>,<Btn4Up>:      scrolled-window-scroll-up(3)\n"
-           "<Btn5Down>,<Btn5Up>:      scrolled-window-scroll-down(3)\n";
+           "Shift<Btn4Down>: scrolled-window-scroll-up(1)\n"
+           "Shift<Btn5Down>: scrolled-window-scroll-down(1)\n"
+           "Ctrl<Btn4Down>:  scrolled-window-page-up()\n"
+           "Ctrl<Btn5Down>:  scrolled-window-page-down()\n"
+           "<Btn4Down>:      scrolled-window-scroll-up(3)\n"
+           "<Btn5Down>:      scrolled-window-scroll-down(3)\n";
         static XtTranslations trans_table = NULL;
         
         if (trans_table == NULL)


----------------------------------------------------------------------

>Comment By: Duane Clark (duaneclark)
Date: 2007-12-28 23:54

Message:
Logged In: YES 
user_id=750500
Originator: YES

Well, I guess whether it is a bug or not depends on your point of view ;)
Anyway, I see it as just an incompatibility with Lesstif. The change should
be harmless on other Motifs.

For what it is worth, I believe this change is already incorporated into
Fedora's nedit package.


----------------------------------------------------------------------

Comment By: Thorsten Haude (yooden)
Date: 2007-12-28 23:18

Message:
Logged In: YES 
user_id=119143
Originator: NO

Firstly, your patch did not survive Sourceforge.

Secondly, I think this is a bug.

Thirdly, your fix works on my Open Motif 2.2.3 and Lesstif 0.93.94. Please
everyone, check whether this works for your Motif!
File Added: mouse-translations.2007-12-29.1.diff

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=111005&aid=1799906&group_id=11005
-- 
NEdit Develop mailing list - [email protected]
http://www.nedit.org/mailman/listinfo/develop

Reply via email to