unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Disable Mouse Wheel Progressive Speed Patch
@ 2017-08-25 18:56 Sarah Sunday
  2017-08-25 20:26 ` Paul Eggert
  2017-08-25 21:04 ` Stefan Monnier
  0 siblings, 2 replies; 14+ messages in thread
From: Sarah Sunday @ 2017-08-25 18:56 UTC (permalink / raw)
  To: emacs-devel@gnu.org


[-- Attachment #1.1: Type: text/plain, Size: 562 bytes --]

This patch simply changes the default value of `mouse-wheel-progressive-speed` to nil, rather than 1. This makes the initial behavior closer to that of other editors: no progressive scrolling speed. It feels like an option that should be opt-in rather than opt-out, which is why I've changed it to be off by default.


This commit was done as part of the <https://github.com/josteink/emacs-oob-reboot> emacs-oob-reboot project<https://github.com/josteink/emacs-oob-reboot>, which seeks to create a better out of the box experience for Emacs.


Thank you.

[-- Attachment #1.2: Type: text/html, Size: 3494 bytes --]

[-- Attachment #2: 0001-Disable-mouse-wheel-progressive-speed.patch --]
[-- Type: application/octet-stream, Size: 887 bytes --]

From cde1c321c7c5aa656788ee6549ba655c84d64842 Mon Sep 17 00:00:00 2001
From: Sarah Sunday <ssunday@users.noreply.github.com>
Date: Fri, 25 Aug 2017 13:28:40 -0500
Subject: [PATCH] Disable mouse wheel progressive speed

---
 lisp/mwheel.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/mwheel.el b/lisp/mwheel.el
index 1428e5f4d0..1232b636fb 100644
--- a/lisp/mwheel.el
+++ b/lisp/mwheel.el
@@ -134,7 +134,7 @@ mouse-wheel-scroll-amount
                     (float :tag "Fraction of window")))))
   :set 'mouse-wheel-change-button)
 
-(defcustom mouse-wheel-progressive-speed t
+(defcustom mouse-wheel-progressive-speed nil
   "If non-nil, the faster the user moves the wheel, the faster the scrolling.
 Note that this has no effect when `mouse-wheel-scroll-amount' specifies
 a \"near full screen\" scroll or when the mouse wheel sends key instead
-- 
2.14.1


^ permalink raw reply related	[flat|nested] 14+ messages in thread

* Re: Disable Mouse Wheel Progressive Speed Patch
  2017-08-25 18:56 Disable Mouse Wheel Progressive Speed Patch Sarah Sunday
@ 2017-08-25 20:26 ` Paul Eggert
  2017-08-25 20:49   ` John Wiegley
  2017-08-28  1:16   ` Sarah Sunday
  2017-08-25 21:04 ` Stefan Monnier
  1 sibling, 2 replies; 14+ messages in thread
From: Paul Eggert @ 2017-08-25 20:26 UTC (permalink / raw)
  To: Sarah Sunday; +Cc: Emacs development discussions

I like the idea of having Emacs defaults being closer to what non-expert 
users expect.

Have you considered filing such change requests by sending email to 
bug-gnu-emacs@gnu.org instead? That would help us track them better.



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: Disable Mouse Wheel Progressive Speed Patch
  2017-08-25 20:26 ` Paul Eggert
@ 2017-08-25 20:49   ` John Wiegley
  2017-08-26 22:36     ` Robert Weiner
  2017-08-28  1:16   ` Sarah Sunday
  1 sibling, 1 reply; 14+ messages in thread
From: John Wiegley @ 2017-08-25 20:49 UTC (permalink / raw)
  To: Paul Eggert; +Cc: Sarah Sunday, Emacs development discussions

>>>>> "PE" == Paul Eggert <eggert@cs.ucla.edu> writes:

PE> I like the idea of having Emacs defaults being closer to what non-expert
PE> users expect.

I too like the thinking behind this change.

-- 
John Wiegley                  GPG fingerprint = 4710 CF98 AF9B 327B B80F
http://newartisans.com                          60E1 46C4 BD1A 7AC1 4BA2



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: Disable Mouse Wheel Progressive Speed Patch
  2017-08-25 18:56 Disable Mouse Wheel Progressive Speed Patch Sarah Sunday
  2017-08-25 20:26 ` Paul Eggert
@ 2017-08-25 21:04 ` Stefan Monnier
  2017-08-26  7:36   ` Eli Zaretskii
  1 sibling, 1 reply; 14+ messages in thread
From: Stefan Monnier @ 2017-08-25 21:04 UTC (permalink / raw)
  To: emacs-devel

> This patch simply changes the default value of
> `mouse-wheel-progressive-speed` to nil, rather than 1. This makes the
> initial behavior closer to that of other editors: no progressive scrolling
> speed. It feels like an option that should be opt-in rather than opt-out,
> which is why I've changed it to be off by default.

This was discussed recently and the "standard" behavior is not always
the same everywhere, apparently.


        Stefan




^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: Disable Mouse Wheel Progressive Speed Patch
  2017-08-25 21:04 ` Stefan Monnier
@ 2017-08-26  7:36   ` Eli Zaretskii
  2017-08-26 10:25     ` Tak Kunihiro
  0 siblings, 1 reply; 14+ messages in thread
From: Eli Zaretskii @ 2017-08-26  7:36 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

> From: Stefan Monnier <monnier@iro.umontreal.ca>
> Date: Fri, 25 Aug 2017 17:04:35 -0400
> 
> > This patch simply changes the default value of
> > `mouse-wheel-progressive-speed` to nil, rather than 1. This makes the
> > initial behavior closer to that of other editors: no progressive scrolling
> > speed. It feels like an option that should be opt-in rather than opt-out,
> > which is why I've changed it to be off by default.
> 
> This was discussed recently and the "standard" behavior is not always
> the same everywhere, apparently.

That discussion starts here:

  http://lists.gnu.org/archive/html/emacs-devel/2017-03/msg00797.html

and continues into April 2017.



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: Disable Mouse Wheel Progressive Speed Patch
  2017-08-26  7:36   ` Eli Zaretskii
@ 2017-08-26 10:25     ` Tak Kunihiro
  2017-08-26 11:09       ` Eli Zaretskii
  2017-08-26 21:06       ` Radon Rosborough
  0 siblings, 2 replies; 14+ messages in thread
From: Tak Kunihiro @ 2017-08-26 10:25 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: Stefan Monnier, emacs-devel

>>> This patch simply changes the default value of
>>> `mouse-wheel-progressive-speed` to nil, rather than 1. This makes the
>>> initial behavior closer to that of other editors: no progressive scrolling
>>> speed. It feels like an option that should be opt-in rather than opt-out,
>>> which is why I've changed it to be off by default.
>> 
>> This was discussed recently and the "standard" behavior is not always
>> the same everywhere, apparently.
>
> That discussion starts here:
>
>   http://lists.gnu.org/archive/html/emacs-devel/2017-03/msg00797.html
>
> and continues into April 2017.

As far I remember, the discussion is still open.

  http://lists.gnu.org/archive/html/emacs-devel/2017-04/msg00040.html

>> (2) Touchpad and Trackbar
>> 
>> Using touchpad or trackbar on laptop computers, I think that defaults
>> scroll too fast, especially by trackbar.  Don't you think so?
>
> I will try that later.

Did you have a chance to try that?  If not, please try it out!



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: Disable Mouse Wheel Progressive Speed Patch
  2017-08-26 10:25     ` Tak Kunihiro
@ 2017-08-26 11:09       ` Eli Zaretskii
  2017-08-27  9:43         ` Tak Kunihiro
  2017-08-26 21:06       ` Radon Rosborough
  1 sibling, 1 reply; 14+ messages in thread
From: Eli Zaretskii @ 2017-08-26 11:09 UTC (permalink / raw)
  To: Tak Kunihiro; +Cc: monnier, emacs-devel

> From: Tak Kunihiro <homeros.misasa@gmail.com>
> Cc: Stefan Monnier <monnier@iro.umontreal.ca>,  emacs-devel@gnu.org
> Date: Sat, 26 Aug 2017 19:25:08 +0900
> 
> As far I remember, the discussion is still open.
> 
>   http://lists.gnu.org/archive/html/emacs-devel/2017-04/msg00040.html
> 
> >> (2) Touchpad and Trackbar
> >> 
> >> Using touchpad or trackbar on laptop computers, I think that defaults
> >> scroll too fast, especially by trackbar.  Don't you think so?
> >
> > I will try that later.
> 
> Did you have a chance to try that?

Given the rest of the discussion, I didn't see any need to try that.
It's not like I'm blocking the change because of that (or anything
else I said).



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: Disable Mouse Wheel Progressive Speed Patch
  2017-08-26 10:25     ` Tak Kunihiro
  2017-08-26 11:09       ` Eli Zaretskii
@ 2017-08-26 21:06       ` Radon Rosborough
  1 sibling, 0 replies; 14+ messages in thread
From: Radon Rosborough @ 2017-08-26 21:06 UTC (permalink / raw)
  To: Tak Kunihiro; +Cc: Eli Zaretskii, Stefan Monnier, emacs-devel

[-- Attachment #1: Type: text/plain, Size: 427 bytes --]

> (2) Touchpad and Trackbar
>
> Using touchpad or trackbar on laptop computers, I think that
> defaults scroll too fast, especially by trackbar. Don't you think
> so?

On my machine (macOS El Capitan), the default scroll speed is about
3,000 lines per second in windowed mode. Disabling progressive speed
reduces that to about 300 lines per second. Meanwhile in the terminal
it's around 60 lines per second... quite the range!

[-- Attachment #2: Type: text/html, Size: 983 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: Disable Mouse Wheel Progressive Speed Patch
  2017-08-25 20:49   ` John Wiegley
@ 2017-08-26 22:36     ` Robert Weiner
  0 siblings, 0 replies; 14+ messages in thread
From: Robert Weiner @ 2017-08-26 22:36 UTC (permalink / raw)
  To: Paul Eggert, Sarah Sunday, Emacs development discussions

[-- Attachment #1: Type: text/plain, Size: 1969 bytes --]

On Fri, Aug 25, 2017 at 4:49 PM, John Wiegley <jwiegley@gmail.com> wrote:

> >>>>> "PE" == Paul Eggert <eggert@cs.ucla.edu> writes:
>
> PE> I like the idea of having Emacs defaults being closer to what
> non-expert
> PE> users expect.
>
> I too like the thinking behind this change.
>

​An extensive amount of work in making Emacs and XEmacs more approachable
to new users was done years ago and was involved enough to warrant a new
name, InfoDock, essentially a turnkey version of XEmacs, though many
features were compatible with Emacs.  InfoDock also was helpful to
experienced users/software developers who wanted features automatically
exposed and easy to find when needed.  I was the primary author.

To have a look just at the extensive menubars and popup menus, see:

https://gist.github.com/rswgnu/b714e01d84f715dedc741f695dda4522
<https://gist.github.com/rswgnu/b714e01d84f715dedc741f695dda4522>

That work stopped in 1999 but the core concepts are just as usable today
and the code is still available at:

<goog_220641172>
https://sourceforge.net/projects/infodock/files/InfoDock/4.00.08/

One a tgz archive is unpacked, b​elow the infodock/id-lisp/infodock/
directory are the id-menus.el and id-menubars.el files shown in the gist
above.​

An experienced Emacs Lisp developer could get the menus of InfoDock working
as an alternative menu-UI within a week I would think.  The easymenu.el
package runs XEmacs menus pretty well, I think.  Then just update commands
as necessary.  Then people could evaluate it and see whether it is worth
adding as an option to Emacs.  The menubars themselves provide a button
that allows switching between default menus and InfoDock-style ones.  I
have assigned my Emacs code to the FSF, so the copyrights would just need
to be updated.

So instead of making small incremental changes that have small impacts,
there is a way to make major improvements fairly quickly.

Bob

[-- Attachment #2: Type: text/html, Size: 4240 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: Disable Mouse Wheel Progressive Speed Patch
  2017-08-26 11:09       ` Eli Zaretskii
@ 2017-08-27  9:43         ` Tak Kunihiro
  2017-08-27 11:56           ` Alan Third
  0 siblings, 1 reply; 14+ messages in thread
From: Tak Kunihiro @ 2017-08-27  9:43 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: Tak Kunihiro, monnier, emacs-devel

>> >> (2) Touchpad and Trackbar
>> >> 
>> >> Using touchpad or trackbar on laptop computers, I think that defaults
>> >> scroll too fast, especially by trackbar.  Don't you think so?
>> >
>> > I will try that later.
>> 
>> Did you have a chance to try that?
>
> Given the rest of the discussion, I didn't see any need to try that.
> It's not like I'm blocking the change because of that (or anything
> else I said).

Pointing devices can be divided into two, that are one with coarse
resolution and one with fine resolution.  Several mice, touchpad, and
trackbar belong to pointing devices with fine resolution.  I understand
that current default value is good for the coarse pointing devices.

However, I thought scroll by the fine pointing device such for trackbar
with current default value is too fast and unusable, but other may have
different impression.  I think Emacs should worry about existence of the
fine pointing devices.



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: Disable Mouse Wheel Progressive Speed Patch
  2017-08-27  9:43         ` Tak Kunihiro
@ 2017-08-27 11:56           ` Alan Third
  2017-08-28  1:23             ` Sarah Sunday
  0 siblings, 1 reply; 14+ messages in thread
From: Alan Third @ 2017-08-27 11:56 UTC (permalink / raw)
  To: Tak Kunihiro; +Cc: Eli Zaretskii, monnier, emacs-devel

On Sun, Aug 27, 2017 at 06:43:07PM +0900, Tak Kunihiro wrote:
> However, I thought scroll by the fine pointing device such for trackbar
> with current default value is too fast and unusable, but other may have
> different impression.  I think Emacs should worry about existence of the
> fine pointing devices.

I’m working (slowly) on multi‐touch support, which will hopefully
solve the problems with some fine pointing devices.

For example, the problem with the trackpad on macs is because macOS
sends an event for each two‐finger scroll movement, which can equate to
as little as a one pixel move. Emacs treats each of those events as a
whole line scroll.

The correct way to handle it is to add up the deltas until they’re
greater than or equal to a line‐height, then issue the scroll command.

-- 
Alan Third



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: Disable Mouse Wheel Progressive Speed Patch
  2017-08-25 20:26 ` Paul Eggert
  2017-08-25 20:49   ` John Wiegley
@ 2017-08-28  1:16   ` Sarah Sunday
  1 sibling, 0 replies; 14+ messages in thread
From: Sarah Sunday @ 2017-08-28  1:16 UTC (permalink / raw)
  To: Paul Eggert; +Cc: Emacs development discussions

I didn’t know about that email for submitting change requests. I’ll make a note of that for next time. Thanks.

> On Aug 25, 2017, at 15:26, Paul Eggert <eggert@cs.ucla.edu> wrote:
> 
> I like the idea of having Emacs defaults being closer to what non-expert users expect.
> 
> Have you considered filing such change requests by sending email to bug-gnu-emacs@gnu.org instead? That would help us track them better.


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: Disable Mouse Wheel Progressive Speed Patch
  2017-08-27 11:56           ` Alan Third
@ 2017-08-28  1:23             ` Sarah Sunday
  2017-08-28  7:03               ` Clément Pit-Claudel
  0 siblings, 1 reply; 14+ messages in thread
From: Sarah Sunday @ 2017-08-28  1:23 UTC (permalink / raw)
  To: Alan Third
  Cc: Tak Kunihiro, emacs-devel@gnu.org, Eli Zaretskii,
	monnier@iro.umontreal.ca

That’s fascinating. Until it’s been fixed, I definitely think we should disable it as, Tak said, it is really troublesome to use out of the box. 

Even after the fix has been implemented, I would say keep it default off as it really feels like something the user would need to configure to compliment their mouse resolution/set-up instead of Emacs assuming for them. But that bridge can be crossed when it gets to that point.

> On Aug 27, 2017, at 06:56, Alan Third <alan@idiocy.org> wrote:
> 
> On Sun, Aug 27, 2017 at 06:43:07PM +0900, Tak Kunihiro wrote:
>> However, I thought scroll by the fine pointing device such for trackbar
>> with current default value is too fast and unusable, but other may have
>> different impression.  I think Emacs should worry about existence of the
>> fine pointing devices.
> 
> I’m working (slowly) on multi‐touch support, which will hopefully
> solve the problems with some fine pointing devices.
> 
> For example, the problem with the trackpad on macs is because macOS
> sends an event for each two‐finger scroll movement, which can equate to
> as little as a one pixel move. Emacs treats each of those events as a
> whole line scroll.
> 
> The correct way to handle it is to add up the deltas until they’re
> greater than or equal to a line‐height, then issue the scroll command.
> 
> -- 
> Alan Third
> 


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: Disable Mouse Wheel Progressive Speed Patch
  2017-08-28  1:23             ` Sarah Sunday
@ 2017-08-28  7:03               ` Clément Pit-Claudel
  0 siblings, 0 replies; 14+ messages in thread
From: Clément Pit-Claudel @ 2017-08-28  7:03 UTC (permalink / raw)
  To: emacs-devel

On 2017-08-28 03:23, Sarah Sunday wrote:
> This makes the initial behavior closer to that of other editors: no 
> progressive scrolling speed. It feels like an option that should be 
> opt-in rather than opt-out, which is why I've changed it to be off
> by default.

> Even after the fix has been implemented, I would say keep it default
> off as it really feels like something the user would need to 
> configure to compliment their mouse resolution/set-up instead of 
> Emacs assuming for them. But that bridge can be crossed when it gets 
> to that point.

I don't strongly oppose a change of default, but wouldn't it be better to improve the scrolling implementation to make the acceleration more subtle, rather than turn it off entirely?

I'd expect most smartphone users to be familiar with accelerated scrolling, since it's a common feature in mobile browsers and and other applications.  And I'm a bit wary about requiring users to do complex adjustments to their mouse configuration.

IOW, I agree good defaults would be nice, but I'm not sure "off" is one.

Clément



^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2017-08-28  7:03 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-08-25 18:56 Disable Mouse Wheel Progressive Speed Patch Sarah Sunday
2017-08-25 20:26 ` Paul Eggert
2017-08-25 20:49   ` John Wiegley
2017-08-26 22:36     ` Robert Weiner
2017-08-28  1:16   ` Sarah Sunday
2017-08-25 21:04 ` Stefan Monnier
2017-08-26  7:36   ` Eli Zaretskii
2017-08-26 10:25     ` Tak Kunihiro
2017-08-26 11:09       ` Eli Zaretskii
2017-08-27  9:43         ` Tak Kunihiro
2017-08-27 11:56           ` Alan Third
2017-08-28  1:23             ` Sarah Sunday
2017-08-28  7:03               ` Clément Pit-Claudel
2017-08-26 21:06       ` Radon Rosborough

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).