Bug Resizable with modifier aspectRatio

taye / interact.js

JavaScript drag and drop, resizing and multi-touch gestures with inertia and snapping for modern browsers (and also IE9+)

http://interactjs.io/

MIT License

12.32k stars 783 forks source link

Bug Resizable with modifier aspectRatio #963

Closed tups closed 2 years ago

tups commented 2 years ago

Sorry, PR #945 closed due to bad push force ^^

Here is the new PR with the updated "main" branch.

The last message: Fixed the bug when creating a resizable element with a "modifier": "aspectRatio". The top right and bottom left angle works in reverse.

Here is an example: https://codepen.io/tups/pen/RwLgmLO

Fixed issue : https://github.com/taye/interact.js/issues/944

taye commented 2 years ago

Can you reword your commit to fix(modifiers/aspectRatio): ...? Then I'll merge after I write some tests

taye commented 2 years ago

Realized I can squash with a different commit message. Thanks again for your PR!