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

ngf-thumbnail is breaking image orientation #881

Open
adamreisnz opened this issue Aug 19, 2020 · 2 comments
Open

ngf-thumbnail is breaking image orientation #881

adamreisnz opened this issue Aug 19, 2020 · 2 comments

Comments

@adamreisnz
Copy link

adamreisnz commented Aug 19, 2020

Using ngf-src, when taking a photo with portrait orientation on Android, it is displayed correctly in the portrait orientation.
However, when using ngf-thumbnail, the orientation is broken and the image is display turned 90 degrees.

This is consistently reproducible.

Am I missing some settings to prevent this from happening? Using ngf-fix-orientation="true" doesn't help.

@adamreisnz
Copy link
Author

adamreisnz commented Aug 19, 2020

The same is happening after an Upload.resize() on the client. The resized image is oriented incorrectly. This is despite using restoreExif: true.

Applying Upload.applyExifRotation() turns the image upside down... 😩

@adamreisnz
Copy link
Author

I ended up not using any in-browser resizing/rotation/scaling, and ended up only using ngf-src instead of ngf-thumbnail, as I couldn't find a combination of tools/hacks/fixes to ensure a correct image orientation.

Images are now displayed correctly and uploaded correctly, and the server is taking care of resizing via kraken.io.

Still a pity, as I would have liked to minimise bandwidth sent to the server by resizing images on the client before upload.

If anyone has any tips on how to fix the problems mentioned in this issue that'd be great.

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

No branches or pull requests

1 participant