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

Fixes #515 #519

Closed
wants to merge 1 commit into from
Closed

Fixes #515 #519

wants to merge 1 commit into from

Conversation

AlexanderLyon
Copy link

When the image is rotated it was keeping its old height and width, sometimes resulting in a distorted rectangle. This is changed to reflect the new dimensions.

@jedsmith13
Copy link
Contributor

Is there anything that needs to be done on this before it can be merged into master? I am having the same issues as @AlexanderLyon and others mentioned on rotating and then resizing a rectangle in #515.

@thedustinsmith
Copy link
Contributor

I'm not opposed to this approach (although it may seem like since I haven't responded in almost a year). Sorry for not responding to this PR earlier, just been caught up in other things.

But I do think this comment is right: #515 (comment). If someone rotates the image 180 deg, the width and height shoudln't change.

@jedsmith13
Copy link
Contributor

That makes sense. If @AlexanderLyon isn't interested or doesn't have time to make that change I can create a PR next week.

@jedsmith13
Copy link
Contributor

This can be closed in favor of #647 as these changes have all been rolled into #647.

@AlexanderLyon
Copy link
Author

Thanks @jedsmith13!

thedustinsmith pushed a commit that referenced this pull request Apr 17, 2019
* Fixes #515

* Added if for 180 as suggested by @bwakabats
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