GNU bug report logs - #57970
29.0.50; Create new defgroup for pixel-scroll.el

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 57970 in the body.
You can then email your comments to 57970 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 29.0.50; Create new defgroup for pixel-scroll.el
Date: Wed, 21 Sep 2022 01:02:47 -0700

Severity: wishlist

The options from pixel-scroll.el are in the `mouse' and `scrolling'
groups now.  Especially the former is very crowded as it is, so I think
we should add a separate defgroup for pixel-scroll.el.

Message #8 received at 57970 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: 57970 <at> debbugs.gnu.org
Subject: Re: bug#57970: 29.0.50; Create new defgroup for pixel-scroll.el
Date: Wed, 21 Sep 2022 13:47:49 +0200

Stefan Kangas <stefankangas <at> gmail.com> writes:

> The options from pixel-scroll.el are in the `mouse' and `scrolling'
> groups now.  Especially the former is very crowded as it is, so I think
> we should add a separate defgroup for pixel-scroll.el.

Makes sense to me.

Message #11 received at 57970 <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 57970 <at> debbugs.gnu.org, Stefan Kangas <stefankangas <at> gmail.com>
Subject: Re: bug#57970: 29.0.50; Create new defgroup for pixel-scroll.el
Date: Wed, 21 Sep 2022 21:51:03 +0800

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> Stefan Kangas <stefankangas <at> gmail.com> writes:
>
>> The options from pixel-scroll.el are in the `mouse' and `scrolling'
>> groups now.  Especially the former is very crowded as it is, so I think
>> we should add a separate defgroup for pixel-scroll.el.
>
> Makes sense to me.

BTW, doesn't this mean we should now put pixel-scroll-precision-mode in
the manual somewhere?  Because the idea was previously that people would
find pixel-scroll-precision-mode by looking at the mouse customize
group.

Message #14 received at 57970 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Po Lu <luangruo <at> yahoo.com>
Cc: 57970 <at> debbugs.gnu.org, Stefan Kangas <stefankangas <at> gmail.com>
Subject: Re: bug#57970: 29.0.50; Create new defgroup for pixel-scroll.el
Date: Wed, 21 Sep 2022 16:03:55 +0200

Po Lu <luangruo <at> yahoo.com> writes:

> BTW, doesn't this mean we should now put pixel-scroll-precision-mode in
> the manual somewhere?

Yes, I think so.

> Because the idea was previously that people would find
> pixel-scroll-precision-mode by looking at the mouse customize group.

The new group will be a sub-group under the mouse group, though, so
people can still find it that way, I think.

Message #19 received at 57970-done <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 57970-done <at> debbugs.gnu.org
Subject: Re: bug#57970: 29.0.50; Create new defgroup for pixel-scroll.el
Date: Sun, 10 Sep 2023 17:17:41 -0700

Version: 29.1

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> Stefan Kangas <stefankangas <at> gmail.com> writes:
>
>> The options from pixel-scroll.el are in the `mouse' and `scrolling'
>> groups now.  Especially the former is very crowded as it is, so I think
>> we should add a separate defgroup for pixel-scroll.el.
>
> Makes sense to me.

Now done on master (commit b6c725ba98a).

Message #22 received at 57970 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: Po Lu <luangruo <at> yahoo.com>, 57970 <at> debbugs.gnu.org
Subject: Re: bug#57970: 29.0.50; Create new defgroup for pixel-scroll.el
Date: Sun, 10 Sep 2023 17:21:08 -0700

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> Po Lu <luangruo <at> yahoo.com> writes:
>
>> BTW, doesn't this mean we should now put pixel-scroll-precision-mode in
>> the manual somewhere?
>
> Yes, I think so.

This bug was closed as fixed, but feel free to propose a patch for this
related but separate issue.

I'm still not sure what is the relationship with pixel-scroll-mode.  Eli
does, I think.  So it might be best to send any suggested changes to the
bug tracker.

Thanks in advance.

Message #25 received at 57970 <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: Lars Ingebrigtsen <larsi <at> gnus.org>, 57970 <at> debbugs.gnu.org
Subject: Re: bug#57970: 29.0.50; Create new defgroup for pixel-scroll.el
Date: Mon, 11 Sep 2023 16:37:10 +0800

Stefan Kangas <stefankangas <at> gmail.com> writes:

> I'm still not sure what is the relationship with pixel-scroll-mode.  Eli
> does, I think.  So it might be best to send any suggested changes to the
> bug tracker.

For what it's worth, the only advantage the old pixel-scroll-mode has
over p-s-p-m is that it functions with regular wheel mice by default.

pixel-scroll-precision-mode is also capable of supporting wheel mice,
but detecting them and enabling the requisite options is tricky and
subject to numerous edge cases, so those options remain off by default.

Message #28 received at 57970 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Po Lu <luangruo <at> yahoo.com>
Cc: larsi <at> gnus.org, 57970 <at> debbugs.gnu.org, stefankangas <at> gmail.com
Subject: Re: bug#57970: 29.0.50; Create new defgroup for pixel-scroll.el
Date: Mon, 11 Sep 2023 15:38:45 +0300

> Cc: Lars Ingebrigtsen <larsi <at> gnus.org>, 57970 <at> debbugs.gnu.org
> Date: Mon, 11 Sep 2023 16:37:10 +0800
> From:  Po Lu via "Bug reports for GNU Emacs,
>  the Swiss army knife of text editors" <bug-gnu-emacs <at> gnu.org>
> 
> Stefan Kangas <stefankangas <at> gmail.com> writes:
> 
> > I'm still not sure what is the relationship with pixel-scroll-mode.  Eli
> > does, I think.  So it might be best to send any suggested changes to the
> > bug tracker.
> 
> For what it's worth, the only advantage the old pixel-scroll-mode has
> over p-s-p-m is that it functions with regular wheel mice by default.
> 
> pixel-scroll-precision-mode is also capable of supporting wheel mice,
> but detecting them and enabling the requisite options is tricky and
> subject to numerous edge cases, so those options remain off by default.

So maybe we should document that one is better for wheels, the other
for touchpads?  And maybe even (gasp!) rename
pixel-scroll-precision-mode to something like touchpad-scroll-mode?

Message #31 received at 57970 <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: larsi <at> gnus.org, 57970 <at> debbugs.gnu.org, stefankangas <at> gmail.com
Subject: Re: bug#57970: 29.0.50; Create new defgroup for pixel-scroll.el
Date: Mon, 11 Sep 2023 20:45:55 +0800

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

>> Cc: Lars Ingebrigtsen <larsi <at> gnus.org>, 57970 <at> debbugs.gnu.org
>> Date: Mon, 11 Sep 2023 16:37:10 +0800
>> From:  Po Lu via "Bug reports for GNU Emacs,
>>  the Swiss army knife of text editors" <bug-gnu-emacs <at> gnu.org>
>> 
>> Stefan Kangas <stefankangas <at> gmail.com> writes:
>> 
>> > I'm still not sure what is the relationship with pixel-scroll-mode.  Eli
>> > does, I think.  So it might be best to send any suggested changes to the
>> > bug tracker.
>> 
>> For what it's worth, the only advantage the old pixel-scroll-mode has
>> over p-s-p-m is that it functions with regular wheel mice by default.
>> 
>> pixel-scroll-precision-mode is also capable of supporting wheel mice,
>> but detecting them and enabling the requisite options is tricky and
>> subject to numerous edge cases, so those options remain off by default.
>
> So maybe we should document that one is better for wheels, the other
> for touchpads?  And maybe even (gasp!) rename
> pixel-scroll-precision-mode to something like touchpad-scroll-mode?

pixel-scroll-mode isn't better for wheel mice as long as you actually
tell p-s-p-m that you use one, so that assessment isn't really correct.

pixel-scroll-mode's principal failings are that it bails out when a
wheel event arrives while scrolling is taking place, it always scrolls
by a set number of lines, and it is slow.  p-s-p-m experiences none of
these problems.

Message #34 received at 57970 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Po Lu <luangruo <at> yahoo.com>
Cc: larsi <at> gnus.org, 57970 <at> debbugs.gnu.org, stefankangas <at> gmail.com
Subject: Re: bug#57970: 29.0.50; Create new defgroup for pixel-scroll.el
Date: Mon, 11 Sep 2023 16:49:59 +0300

> From: Po Lu <luangruo <at> yahoo.com>
> Cc: stefankangas <at> gmail.com,  larsi <at> gnus.org,  57970 <at> debbugs.gnu.org
> Date: Mon, 11 Sep 2023 20:45:55 +0800
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> > So maybe we should document that one is better for wheels, the other
> > for touchpads?  And maybe even (gasp!) rename
> > pixel-scroll-precision-mode to something like touchpad-scroll-mode?
> 
> pixel-scroll-mode isn't better for wheel mice as long as you actually
> tell p-s-p-m that you use one, so that assessment isn't really correct.
> 
> pixel-scroll-mode's principal failings are that it bails out when a
> wheel event arrives while scrolling is taking place, it always scrolls
> by a set number of lines, and it is slow.  p-s-p-m experiences none of
> these problems.

It sounds like your comments actually agree with my suggestions ;-)

To clarify, I didn't mean that pixel-scroll-precision-mode _cannot_ be
used with wheels, I just said pixel-scroll-mode is _better_ there
(because it requires no tricky configuration); conversely, for
touchpads, pixel-scroll-precision-mode produces a much better UX OOTB.

This particular discussion is about two modes that have very similar
names.  I think explaining the strengths and weaknesses (even if they
are minor and can be overcome, given enough efforts) goes at least
some way towards resolving user confusion about these two, and
changing the name, even if not 110% accurate, is even better.

Message #37 received at 57970 <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: larsi <at> gnus.org, 57970 <at> debbugs.gnu.org, stefankangas <at> gmail.com
Subject: Re: bug#57970: 29.0.50; Create new defgroup for pixel-scroll.el
Date: Mon, 11 Sep 2023 22:46:03 +0800

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

> It sounds like your comments actually agree with my suggestions ;-)
>
> To clarify, I didn't mean that pixel-scroll-precision-mode _cannot_ be
> used with wheels, I just said pixel-scroll-mode is _better_ there
> (because it requires no tricky configuration); conversely, for
> touchpads, pixel-scroll-precision-mode produces a much better UX OOTB.

That is correct, but not from an objective standpoint, only from the POV
of what is better "out of the box".  p-s-p-m is worse off OOTB, because
it takes no effect at all without some configuration.

> This particular discussion is about two modes that have very similar
> names.  I think explaining the strengths and weaknesses (even if they
> are minor and can be overcome, given enough efforts) goes at least
> some way towards resolving user confusion about these two, and
> changing the name, even if not 110% accurate, is even better.

Users customarily search for terms in the vein of "pixel scroll", which
that must still be taken into account: whatever name we select for the
mode should incorporate the word "pixel".

One more data point: within the stream of feature requests that
constitued the impetus for my decision to implement this feature, not a
single request mentioned the word "touchpad", whereas "pixel" and
"precise" abounded.  At least in my recollection.

Message #40 received at 57970 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Po Lu <luangruo <at> yahoo.com>
Cc: larsi <at> gnus.org, 57970 <at> debbugs.gnu.org, stefankangas <at> gmail.com
Subject: Re: bug#57970: 29.0.50; Create new defgroup for pixel-scroll.el
Date: Mon, 11 Sep 2023 18:52:46 +0300

> From: Po Lu <luangruo <at> yahoo.com>
> Cc: stefankangas <at> gmail.com,  larsi <at> gnus.org,  57970 <at> debbugs.gnu.org
> Date: Mon, 11 Sep 2023 22:46:03 +0800
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> > This particular discussion is about two modes that have very similar
> > names.  I think explaining the strengths and weaknesses (even if they
> > are minor and can be overcome, given enough efforts) goes at least
> > some way towards resolving user confusion about these two, and
> > changing the name, even if not 110% accurate, is even better.
> 
> Users customarily search for terms in the vein of "pixel scroll", which
> that must still be taken into account: whatever name we select for the
> mode should incorporate the word "pixel".
> 
> One more data point: within the stream of feature requests that
> constitued the impetus for my decision to implement this feature, not a
> single request mentioned the word "touchpad", whereas "pixel" and
> "precise" abounded.  At least in my recollection.

We could use touchpad-pixel-scroll, for example.

Previous Next

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