Make button numbers consistent across platforms

There's no universal consensus on how buttons are numbered.  Left, right,
middle as 0, 1, 2 seems to be the most common convention on modern vaguely
similar libraries, so I've gone with that.

The switch to zero-based indices will obviously break all current client code.
Particularly since now is the time to finish any breaking changes before a
stable release, I think that is better than only changing the middle and right
numbers, which would likely go unnoticed.

3 files changed, 14 insertions, 7 deletions

diff --git a/src/mac.m b/src/mac.m
index 5a10964..84530b4 100644
--- a/src/mac.m
+++ b/src/mac.m
@@ -448,7 +448,7 @@ handleCrossing(PuglWrapperView* view, NSEvent* event, const PuglEventType type)
     rloc.x,
     [[NSScreen mainScreen] frame].size.height - rloc.y,
     getModifiers(event),
-    (uint32_t)[event buttonNumber] + 1,
+    (uint32_t)[event buttonNumber],
   };
 
   PuglEvent pressEvent;
@@ -469,7 +469,7 @@ handleCrossing(PuglWrapperView* view, NSEvent* event, const PuglEventType type)
     rloc.x,
     [[NSScreen mainScreen] frame].size.height - rloc.y,
     getModifiers(event),
-    (uint32_t)[event buttonNumber] + 1,
+    (uint32_t)[event buttonNumber],
   };
 
   PuglEvent releaseEvent;
diff --git a/src/win.c b/src/win.c
index eb1d0d1..c7a95d5 100644
--- a/src/win.c
+++ b/src/win.c
@@ -680,22 +680,22 @@ handleMessage(PuglView* view, UINT message, WPARAM wParam, LPARAM lParam)
     view->impl->mouseTracked = false;
     break;
   case WM_LBUTTONDOWN:
-    initMouseEvent(&event, view, 1, true, lParam);
+    initMouseEvent(&event, view, 0, true, lParam);
     break;
   case WM_MBUTTONDOWN:
     initMouseEvent(&event, view, 2, true, lParam);
     break;
   case WM_RBUTTONDOWN:
-    initMouseEvent(&event, view, 3, true, lParam);
+    initMouseEvent(&event, view, 1, true, lParam);
     break;
   case WM_LBUTTONUP:
-    initMouseEvent(&event, view, 1, false, lParam);
+    initMouseEvent(&event, view, 0, false, lParam);
     break;
   case WM_MBUTTONUP:
     initMouseEvent(&event, view, 2, false, lParam);
     break;
   case WM_RBUTTONUP:
-    initMouseEvent(&event, view, 3, false, lParam);
+    initMouseEvent(&event, view, 1, false, lParam);
     break;
   case WM_MOUSEWHEEL:
     initScrollEvent(&event, view, lParam);
diff --git a/src/x11.c b/src/x11.c
index 614b7ac..ed249a8 100644
--- a/src/x11.c
+++ b/src/x11.c
@@ -741,7 +741,14 @@ translateEvent(PuglView* const view, XEvent xevent)
       event.button.xRoot  = xevent.xbutton.x_root;
       event.button.yRoot  = xevent.xbutton.y_root;
       event.button.state  = translateModifiers(xevent.xbutton.state);
-      event.button.button = xevent.xbutton.button;
+      event.button.button = xevent.xbutton.button - 1;
+      if (event.button.button == 1) {
+        event.button.button = 2;
+      } else if (event.button.button == 2) {
+        event.button.button = 1;
+      } else if (event.button.button >= 7) {
+        event.button.button -= 4;
+      }
     }
     break;
   case KeyPress: