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

Fix multi touch scaling speed #2625

Merged
merged 1 commit into from
Nov 13, 2015
Merged

Fix multi touch scaling speed #2625

merged 1 commit into from
Nov 13, 2015

Conversation

asturur
Copy link
Member

@asturur asturur commented Nov 13, 2015

we need to use transformed dimensions to include effect of skew. but we need to divide by current scale and use just the initial scale plus the scaling factor of gesture.

Closes #2586
closes #2597

we need to use transformed dimensions to include effect of skew. but we need to divide by current scale and use just the initial scale plus the scaling factor of gesture.
@asturur
Copy link
Member Author

asturur commented Nov 13, 2015

it works good.
@kangax mergw this as pre 1.6.0 release. othewise multi touch scaling is broken.

@asturur asturur changed the title Update canvas_gestures.mixin.js Fix multi touch scaling speed Nov 13, 2015
kangax added a commit that referenced this pull request Nov 13, 2015
Fix multi touch scaling speed
@kangax kangax merged commit 7cb4f17 into fabricjs:master Nov 13, 2015
@asturur asturur deleted the multitouch branch November 16, 2015 00:23
@asturur asturur restored the multitouch branch January 29, 2016 23:11
@asturur asturur deleted the multitouch branch February 7, 2016 23:53
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.

Multitouch zooming is buggy on Android 1.6.0-rc1 multitouch not working with gesture
2 participants