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

prevent viewer from zooming on cmd+mousewheel (bug 1018932) #5926

Merged
merged 1 commit into from Apr 13, 2015
Merged

prevent viewer from zooming on cmd+mousewheel (bug 1018932) #5926

merged 1 commit into from Apr 13, 2015

Conversation

ghost
Copy link

@ghost ghost commented Apr 10, 2015

As reported in bug 1018932, the viewer toolbar resizes with the content when zooming with cmd+mousewheel.

This pull request adds a condition to the mousewheel handler to check if the metakey is pressed, in which case toolbar zoom is prevented.

@ghost
Copy link
Author

ghost commented Apr 10, 2015

Please note that this is my first contribution to a Mozilla project, so style or workflow feedback is appreciated.

@Snuffleupagus
Copy link
Collaborator

@browtayl Please squash the commits, see https://github.com/mozilla/pdf.js/wiki/Squashing-Commits.

@ghost
Copy link
Author

ghost commented Apr 12, 2015

@Snuffleupagus squashed.

@Snuffleupagus
Copy link
Collaborator

/botio-linux preview

@pdfjsbot
Copy link

From: Bot.io (Linux)


Received

Command cmd_preview from @Snuffleupagus received. Current queue size: 0

Live output at: http://107.21.233.14:8877/d6b1858bae97e5d/output.txt

timvandermeij added a commit that referenced this pull request Apr 13, 2015
prevent viewer from zooming on cmd+mousewheel (bug 1018932)
@timvandermeij timvandermeij merged commit e4e9a2a into mozilla:master Apr 13, 2015
@timvandermeij
Copy link
Contributor

Thank you for the patch!

@ghost ghost deleted the 1018932 branch April 14, 2015 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants