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(fabric.Canvas): rotation handle should take origin into account #6686

Merged
merged 1 commit into from
Nov 15, 2020

Conversation

virror
Copy link
Contributor

@virror virror commented Nov 11, 2020

Remove the fixed center origin from the code.
Default origin should depend from _shouldCenterTransform function in the _setupCurrentTransform;

close #6679

@asturur
Copy link
Member

asturur commented Nov 15, 2020

@virror a small description would have been on spot.

@asturur asturur merged commit 8bcb539 into fabricjs:master Nov 15, 2020
@asturur asturur mentioned this pull request Dec 23, 2020
shanicerae pushed a commit to shanicerae/fabric.js that referenced this pull request Jan 16, 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.

centeredRotation is not taken into account for rotation handle
3 participants