Fixed bug 1173 (No mouse wheel event on linux/x11)
authorSam Lantinga <slouken@libsdl.org>
Tue, 05 Apr 2011 09:35:56 -0700
changeset 5530 4e46a7b6773d
parent 5529 8c0d15077360
child 5531 477f550f6efd
Fixed bug 1173 (No mouse wheel event on linux/x11) Matthias 2011-03-20 23:07:02 PDT On X11, SDL 1.3 does not generate a mouse wheel event. Instead, button down/buttton up events are generated by SDL. After looking at the code in SDL_x11events.c, I assume this is due to the fact that X11 does not have a dedicated mouse wheel event. I did a little research on the behavior of mouse wheel events on X11 systems. Apparently, mouse wheel events generate a button down/button up event with the same time, i.e. with exact same timestamp. Attached you can find my changes to SDL_x11events.c, which generates SDL mouse wheel events for those button down events that have a button release event immediately following it (for the same button, and with the same timestamp). I did have to make an assumption: As standard X11 implementations know only 5 buttons, I have mapped Button4 to "wheel up" (i.e. +1), and Button5 to "wheel down" (i.e. -1). Note that if you include this patch, no SDL button down/up events will be generated on X11 platforms for mouse wheel events (which is probably a significant change for those that have programmed their code to work with them).
src/video/x11/SDL_x11events.c
--- a/src/video/x11/SDL_x11events.c	Mon Apr 04 23:38:15 2011 -0400
+++ b/src/video/x11/SDL_x11events.c	Tue Apr 05 09:35:56 2011 -0700
@@ -65,6 +65,37 @@
     return SDL_FALSE;
 }
 
+static SDL_bool X11_IsWheelEvent(Display * display,XEvent * event,int * ticks)
+{
+    XEvent peekevent;
+    if (XPending(display)) {
+        /* according to the xlib docs, no specific mouse wheel events exist.
+           however, mouse wheel events trigger a button press and a button release
+           immediately. thus, checking if the same button was released at the same
+           time as it was pressed, should be an adequate hack to derive a mouse 
+           wheel event. */
+        XPeekEvent(display,&peekevent);
+        if ((peekevent.type           == ButtonRelease) &&
+            (peekevent.xbutton.button == event->xbutton.button) &&
+            (peekevent.xbutton.time   == event->xbutton.time)) {
+
+            /* by default, X11 only knows 5 buttons. on most 3 button + wheel mouse,
+               Button4 maps to wheel up, Button5 maps to wheel down. */
+            if (event->xbutton.button == Button4) {
+                *ticks = 1;
+            }
+            else {
+                *ticks = -1;
+            }
+
+            /* remove the following release event, as this is now a wheel event */
+            XNextEvent(display,&peekevent);
+            return SDL_TRUE;
+        }
+    }
+    return SDL_FALSE;
+}
+
 static void
 X11_DispatchEvent(_THIS)
 {
@@ -316,7 +347,13 @@
         break;
 
     case ButtonPress:{
-            SDL_SendMouseButton(data->window, SDL_PRESSED, xevent.xbutton.button);
+            int ticks = 0;
+            if (X11_IsWheelEvent(display,&xevent,&ticks) == SDL_TRUE) {
+                SDL_SendMouseWheel(data->window, 0, ticks);
+            }
+            else {
+                SDL_SendMouseButton(data->window, SDL_PRESSED, xevent.xbutton.button);
+            }
         }
         break;