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: handle existing negative css transforms on resizable elements #124

Merged
merged 2 commits into from
Jul 24, 2020

Conversation

kiskadenja
Copy link
Contributor

The regex for resize wasn't working for my component when translate contained a negative value.

Copy link
Owner

@mattlewis92 mattlewis92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great, thank you! Would you mind adding a very small test for this? You can copy and adapt this one: https://github.com/mattlewis92/angular-resizable-element/blob/master/test/resizable.spec.ts#L1447-L1470

@kiskadenja
Copy link
Contributor Author

Added test for pull request changes. Thx

Copy link
Owner

@mattlewis92 mattlewis92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you! Will cut a release shortly

@mattlewis92 mattlewis92 changed the title fix: regex capture for negative translation values fix: handle existing negative css transforms on resizable elements Jul 24, 2020
@mattlewis92 mattlewis92 merged commit edf56db into mattlewis92:master Jul 24, 2020
@kiskadenja kiskadenja deleted the kiskadenja branch July 24, 2020 13:31
@kiskadenja
Copy link
Contributor Author

Thank you! 😃

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.

3 participants