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

Also bind tilt to ctrl-right-drag. #2913

Merged
merged 3 commits into from
Aug 2, 2015
Merged

Also bind tilt to ctrl-right-drag. #2913

merged 3 commits into from
Aug 2, 2015

Conversation

shunter
Copy link
Contributor

@shunter shunter commented Aug 1, 2015

On Firefox, on OS X, it is impossible to ctrl-left-click, on a laptop with a trackpad. Because OS X typically interprets ctrl-click as a secondary ("right") click, Firefox "helpfully" decides to send a right-click JS mouse event, unlike both Chrome and Safari.

This new mouse binding affects all platforms, but ctrl-right-drag wasn't bound to anything by default anyway.

On Firefox, on OS X, it is impossible to ctrl-left-click, on a laptop with a trackpad.  Because OS X typically interprets ctrl-click as a secondary ("right") click, Firefox "helpfully" decides to send a right-click JS mouse event, unlike both Chrome and Safari.

This new mouse binding affects all platforms, but ctrl-right-drag wasn't bound to anything by default anyway.
@pjcozzi
Copy link
Contributor

pjcozzi commented Aug 2, 2015

Sounds good to me. Update CHANGES.md, please (for 1.13 since we are releasing 1.12 on Monday).

Does anyone else want to look at this? @emackey?

@jimmyangel
Copy link
Contributor

I tested this on Firefox 39.0 on Mac OSX 10.10.4 and it works like a charm. Thank you!

mramato added 2 commits August 2, 2015 17:34
Forgot to stage all changes.
@mramato
Copy link
Contributor

mramato commented Aug 2, 2015

I updated changes. Given this is a no/low impact change and people have mentioned it on the forum several times in the past, I see no reason why this can't go into 1.12.

@pjcozzi any objections? I didn't want to just hit merge since you assumed this is for 1.13.

@pjcozzi
Copy link
Contributor

pjcozzi commented Aug 2, 2015

Nah, 1.12 is OK.

Please follow up on the forum.

pjcozzi added a commit that referenced this pull request Aug 2, 2015
@pjcozzi pjcozzi merged commit d9b43e8 into master Aug 2, 2015
@pjcozzi pjcozzi deleted the fixTiltOnFirefoxMac branch August 2, 2015 22:11
@pjcozzi
Copy link
Contributor

pjcozzi commented Aug 2, 2015

Thanks @shunter!

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.

4 participants