<div dir="ltr">I actually feel the same way - but kept the old style just in case. I'll update it.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 3, 2015 at 2:36 PM, Peter Hutterer <span dir="ltr"><<a href="mailto:peter.hutterer@who-t.net" target="_blank">peter.hutterer@who-t.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Mon, Dec 22, 2014 at 11:51:05AM -0800, Dima Ryazanov wrote:<br>
> They're off by 4 because of the scroll buttons.<br>
><br>
> (However, if you test them in XWayland, they'll appear to work because<br>
> XWayland has the same bug; see<br>
> <a href="http://lists.x.org/archives/xorg-devel/2014-December/044987.html" target="_blank">http://lists.x.org/archives/xorg-devel/2014-December/044987.html</a>)<br>
><br>
> Signed-off-by: Dima Ryazanov <<a href="mailto:dima@gmail.com">dima@gmail.com</a>><br>
> ---<br>
>  src/compositor-x11.c | 5 ++++-<br>
>  1 file changed, 4 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/src/compositor-x11.c b/src/compositor-x11.c<br>
> index 29f2119..af9de6f 100644<br>
> --- a/src/compositor-x11.c<br>
> +++ b/src/compositor-x11.c<br>
> @@ -1015,7 +1015,10 @@ x11_compositor_deliver_button_event(struct x11_compositor *c,<br>
><br>
>       switch (button_event->detail) {<br>
>       default:<br>
> -             button = button_event->detail + BTN_LEFT - 1;<br>
> +             button = button_event->detail + BTN_SIDE - 8;<br>
> +             break;<br>
> +     case 1:<br>
> +             button = BTN_LEFT;<br>
>               break;<br>
>       case 2:<br>
>               button = BTN_MIDDLE;<br>
> --<br>
> 2.1.0<br>
<br>
</div></div>Reviewed-by: Peter Hutterer <<a href="mailto:peter.hutterer@who-t.net">peter.hutterer@who-t.net</a>><br>
though I'd prefer having the default at the bottom of the switch<br>
case<br>
<br>
Cheers,<br>
   Peter<br>
</blockquote></div><br></div>