Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add support for mouse wheel to change slide #179

Open
nbecker opened this issue Dec 2, 2020 · 6 comments
Open

add support for mouse wheel to change slide #179

nbecker opened this issue Dec 2, 2020 · 6 comments

Comments

@nbecker
Copy link

nbecker commented Dec 2, 2020

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

@Cimbali
Copy link
Owner

Cimbali commented Dec 3, 2020

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

@Sargeanthost
Copy link

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

@nbecker
Copy link
Author

nbecker commented Dec 7, 2020 via email

@Cimbali
Copy link
Owner

Cimbali commented Dec 7, 2020

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
Copy link
Owner

Cimbali commented Apr 4, 2021

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

@ousia
Copy link
Contributor

ousia commented Nov 4, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants