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

Images with exif-orientations are not resized properly. #3148

Closed
mficzel opened this issue Oct 27, 2020 · 3 comments · Fixed by #3147
Closed

Images with exif-orientations are not resized properly. #3148

mficzel opened this issue Oct 27, 2020 · 3 comments · Fixed by #3147

Comments

@mficzel
Copy link
Member

mficzel commented Oct 27, 2020

When images are resized that use the exif orientation feature the thumbnails and crops are flipped. Since this is used by some cameras it should be supported by neos. Imagine has an autorotate feature that can be used for that.

Steps to Reproduce

  1. Go to the media module
  2. Upload an image with exif orientation like https://github.com/recurser/exif-orientation-examples/blob/master/Landscape_6.jpg
  3. Look at the generated thumbnails

Expected behavior

Thumbnail should have the same orientation as original.

Actual behavior

Thumbnail is rotated 90°

Affected Versions

Neos: afaik all

@fleigm
Copy link

fleigm commented Jan 15, 2021

Good day everyone,

I can reproduce this issue in neos 5.3.1 and 7.0 despite the merged bugfix.
Does anyone else have this problem too?

@dlubitz
Copy link
Contributor

dlubitz commented Mar 14, 2024

I had this issue today on a customers instance with Neos 7.3 and also checked other ones (incl. 8.3). All of them show the same incorrect result, but after cropping the image.

image

I found this open issue at imagine, which seems to be related. After adding the suggested solution , it works fine for me.
php-imagine/Imagine#467

@dlubitz
Copy link
Contributor

dlubitz commented Mar 14, 2024

Also stripping the image (removing exif) solves this issue:

        $stripFilter = new \Imagine\Filter\Basic\Strip();
        $stripFilter->apply($imagineImage);

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants