Take stock of the wheel-up/down confusion
authorStefan Monnier <monnier@iro.umontreal.ca>
Wed, 24 Jan 2024 17:57:33 +0000 (12:57 -0500)
committerStefan Monnier <monnier@iro.umontreal.ca>
Wed, 24 Jan 2024 17:57:33 +0000 (12:57 -0500)
commit1f3371b46e8a6a51f88c56785175b48af2a0bed7
tree0dfb2d28d97887f9b06a9cfee3105680be5805cc
parentc9705037e98a398d0e6e145f16e0ce8cdd4a8973
Take stock of the wheel-up/down confusion

While we're hopefully all aware of the usual confusion between
the scroll operation moving the document or moving the viewport, Emacs
has its very own instance of that confusion where the
`mouse-wheel-down-event` variable is the one that (used to) hold
the value `wheel-up` and vice versa.

Thanks for Po Lu's commit 957b4f826a4 which not only fixed my change
but brought that confusion to my attention.
This patch doesn't fix the problem, but tries to fix the other places
in the code where we did not take it into account.

* doc/lispref/commands.texi (Misc Events): Mention the
wheel-up/down confusion.

* lisp/progmodes/flymake.el (flymake--mode-line-counter-map):
* lisp/completion-preview.el (completion-preview--mouse-map):
Fix wheel-up/down confusion.

* lisp/mwheel.el (mouse-wheel-down-event, mouse-wheel-up-event):
Fix docstrings.
doc/lispref/commands.texi
lisp/completion-preview.el
lisp/mwheel.el
lisp/progmodes/flymake.el