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

Increase scroll speed for glium backend #702

Closed
wants to merge 1 commit into from

Conversation

akhilman
Copy link

@akhilman akhilman commented Sep 7, 2021

Each so called "line" of the MuseScrollDelta::LineDelta de-facto represents
a single scroller click which in most cases equals to 15 degrees of wheel
rotation.

Currently, one scroller click scrolls exactly one line of the text,
which is very slow.

This change multiplies the scroll rate by 5. So, with one click of the
scroller, 5 lines scrolled. This number is taken from the "iced" GUI
library and feels quite balanced.

Closes #461

Each so called "line" of the MuseScrollDelta::LineDelta de-facto represents
a single scroller click which in most cases equals to 15 degrees of wheel
rotation.

Currently, one scroller click scrolls exactly one line of the text,
which is very slow.

This change multiplies the scroll rate by 5. So, with one click of the
scroller, 5 lines scrolled. This number is taken from the "iced" GUI
library and feels quite balanced.

Fixes emilk#461
@emilk emilk closed this in 2498765 Sep 7, 2021
@emilk
Copy link
Owner

emilk commented Sep 7, 2021

Thank you! I did my own commit because I felt we should change egui_web at the same time, and update the changelog

@akhilman
Copy link
Author

akhilman commented Sep 7, 2021

egui_web was totally fine. With 50 as multiplier it scrolls for 300 pixels per click.

emilk added a commit that referenced this pull request Sep 7, 2021
@emilk
Copy link
Owner

emilk commented Sep 7, 2021

Haha 🤦 - thanks for testing @akhilman ! The change has been reverted

mankinskin pushed a commit to mankinskin/egui that referenced this pull request Sep 29, 2021
mankinskin pushed a commit to mankinskin/egui that referenced this pull request Sep 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Scroll is too slow on Gnome/Wayland
2 participants