GNU bug report logs - #51891
29.0.50; [PATCH] Pixel delta support for wheel events on X

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Po Lu <luangruo <at> yahoo.com>
Subject: bug#51891: closed (Re: bug#51891: 29.0.50; [PATCH] Pixel delta
 support for wheel events on X)
Date: Sat, 20 Nov 2021 10:30:02 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#51891: 29.0.50; [PATCH] Pixel delta support for wheel events on X

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 51891 <at> debbugs.gnu.org.

-- 
51891: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=51891
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Po Lu <luangruo <at> yahoo.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: rpluim <at> gmail.com, 51891-done <at> debbugs.gnu.org
Subject: Re: bug#51891: 29.0.50; [PATCH] Pixel delta support for wheel
 events on X
Date: Sat, 20 Nov 2021 18:28:54 +0800

Eli Zaretskii <eliz <at> gnu.org> writes:

> Sure, why not?
>
> Thanks.

Thanks, installed with the improvement to documentation you suggested in
the other message.

[Message part 3 (message/rfc822, inline)]

From: Po Lu <luangruo <at> yahoo.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 29.0.50; [PATCH] Pixel delta support for wheel events on X
Date: Tue, 16 Nov 2021 20:38:03 +0800

[Message part 4 (text/plain, inline)]

I've implemented support for exposing the pixel delta of XInput 2 wheel
events to Lisp code.  With GC turned off, utilizing a modified
pixel-scroll.el with this support results in a quite pleasant
experience, but pixel-scroll still GCs too much to be useful.

Pixel based scrolling is a popular feature in modern X applications, and
does make life a lot easier in many cases.  The existing support is
inadequate as it cannot utilize the detailed delta information available
from modern touch-based scroll wheels, which this patch enables.

The XInput 2 support isn't the best from a code duplication POV, but the
behavior of the XInput 2 code often differs in subtle ways from that of
the Core Input code, and trying to combine the two together would result
in a more catastrophic mess than leaving them apart.

Obviously, here is the original XInput 2 support patch:

[0001-Add-support-for-event-processing-via-XInput-2.patch (text/x-patch, attachment)]

[Message part 6 (text/plain, inline)]

And here is the patch that exposes pixel deltas to Lisp:

[0002-Expose-pixel-wise-wheel-events-to-Lisp.patch (text/x-patch, attachment)]

[Message part 8 (text/plain, inline)]

WDYT?  Thanks.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.