[ nedit-Patches-1799906 ] Make mouse wheel scrolling compatible with lesstif

[SourceForge.net]

Patches item #1799906, was opened at 2007-09-21 19:37
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=311005&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: None
Group: None
Status: Open
Resolution: None
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)


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

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=311005&aid=1799906&group_id=11005