pympress icon indicating copy to clipboard operation
pympress copied to clipboard

add support for mouse wheel to change slide

Open nbecker opened this issue 5 years ago • 6 comments

Just a feature request, it would be nice to be able to use mouse wheel to go forward/backward

nbecker avatar Dec 02 '20 16:12 nbecker

There’s a risk of accidentally scrolling a whole lot of slides, no ? If you wrongly touch a trackpad or something like that. Right now you can wheel through slides in preview mode only, i.e. when you press G

Cimbali avatar Dec 03 '20 09:12 Cimbali

@Cimbali I use the mouse wheel a lot, wouldn't hurt to add a toggle like in preview.

Sargeanthost avatar Dec 07 '20 20:12 Sargeanthost

I've used this in other viewers, such as evince and okular, and it's always worked well. I guess depends on your mouse sensitivity.

On Mon, Dec 7, 2020 at 3:12 PM Sargeanthost [email protected] wrote:

@Cimbali https://github.com/Cimbali I use the mouse wheel a lot, wouldn't hurt to add a toggle like in preview.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Cimbali/pympress/issues/179#issuecomment-740152934, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA4VUFY2Y3EDOQJY2ZFIXLSTUZJTANCNFSM4UKX676A .

-- Those who don't understand recursion are doomed to repeat it

nbecker avatar Dec 07 '20 20:12 nbecker

Alright, let’s try it and add an option to disable it if it causes issues.

I think there’s some subtleties to fix, as the preview scrolling is based on the number display so right now:

  • up = increase page number = advance in document
  • down = decrease page number = go back in document

I think that’s counter-intuitive in normal browsing, and we’ll want to have the wheel work the same in both modes.

Cimbali avatar Dec 07 '20 23:12 Cimbali

I also see scrolling currently scrolls the annotations by default, if slide editing is not enabled.

Cimbali avatar Apr 04 '21 15:04 Cimbali

I wonder whether clicking on the slide to advance it could be considered as part of this issue.

ousia avatar Nov 04 '21 15:11 ousia