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

adopt wheel event. #70515

Closed
wants to merge 1 commit into from
Closed

Conversation

rebornix
Copy link
Member

@rebornix rebornix commented Mar 14, 2019

The events I get when scroll my MX Master by a tiny click. I'm also using the system scroll sensitivity setting.

OS Browser Event deltaMode deltaY detail wheelDeltaY wheelDelta
macOS Chrome mousewheel 0 13 0 -39 -39
macOS Chrome wheel 0 13 0 -39 -39
macOS Firefox mousewheel
macOS Firefox wheel 0 13 0
macOS Safari mousewheel 0 13 0 -39 -39
macOS Safari wheel 0 13 0 -39 -39
windows Chrome mousewheel 0 21.66 0 -26 -26
windows Chrome wheel 0 21.66 0 -26 -26
windows Firefox mousewheel
windows Firefox wheel 1 0.65 0
windows Edge mousewheel 0 12.8 0 -26 -26
windows Edge wheel 0 12 0 -24 -24
windows IE mousewheel 0 -39
windows IE wheel 0 13.64 0

@rebornix
Copy link
Member Author

rebornix commented Apr 2, 2019

Test Items

  • Windows scaling setting
  • Zooming

@UtkarshGupta-CS
Copy link
Contributor

@rebornix Any update on this? Anything I could help with this. Or is this ready?

@rebornix rebornix closed this Jul 25, 2019
@rebornix
Copy link
Member Author

It's already fixed in master. We can look into how to adopt deltaY/deltaX in the future, for now we still use wheelDeltaY/wheelDeltaX when possible.

@rebornix rebornix deleted the rebornix/wheel branch July 25, 2019 21:31
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants