Skip to content

#105 Fix resize with n/s handles on locked aspect ratio#106

Merged
STRML merged 2 commits intoreact-grid-layout:masterfrom
simzar:master
Oct 22, 2019
Merged

#105 Fix resize with n/s handles on locked aspect ratio#106
STRML merged 2 commits intoreact-grid-layout:masterfrom
simzar:master

Conversation

@simzar
Copy link
Copy Markdown
Contributor

@simzar simzar commented Aug 30, 2019

Fixes issue #105
Additional calculation of dimensions with locked aspect ratio starting from height change, not only width.

@STRML
Copy link
Copy Markdown
Collaborator

STRML commented Oct 22, 2019

Oh that's quite interesting. I see the issue, I wonder if there isn't a more elegant way to do this.

@simzar
Copy link
Copy Markdown
Contributor Author

simzar commented Oct 22, 2019

@STRML I agree, updated the PR with a slight local code improvement.
I've tried to refactor as little as possible, although I believe the issue could be prevented with introduction of axis specific calculations (tracking which axis was dragged).

@STRML STRML merged commit 6932b5d into react-grid-layout:master Oct 22, 2019
@STRML
Copy link
Copy Markdown
Collaborator

STRML commented Oct 22, 2019

Thanks for the contribution.

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.

2 participants