Viewer discards EXIF orientation tag of JPEG images

Flaw · Post by *Flaw » 2011-11-04, 23:33 UTC

I think this is clearly a bug (or maybe a critically missing feature now, in the end of 2011): the integrated Viewer simply discards the EXIF orientation tag of every JPEG image. In the age where thousands of pictures are captured every year by almost every single human being, it maybe worth the effort to support this tiny little EXIF tag to allow Viewer to render the picture in the way it was meant to look.

Post by *Hacker » 2011-11-05, 00:15 UTC

[mod]Moved to TC Suggestions.

Hacker (Moderator)[/mod]

tbeu · Post by *tbeu » 2011-11-11, 12:10 UTC

History.txt wrote:07.11.11 Added: Lister: Auto-rotate jpg images loaded via GDI+ to the angle stored in the EXIF information (32/64)

Is this optional in TC8PB9?

Post by *ghisler(Author) » 2011-11-11, 13:34 UTC

No, it's not.

tbeu · Post by *tbeu » 2011-11-11, 13:37 UTC

ghisler(Author) wrote:No, it's not.

That's no good. There are images where Exif orientation data does not match the true image orientation, e.g. image was manually rotated and Exif orientation flag was not reset. Maybe Lister should get some extra hot keys for left/right rotation.

Post by *ghisler(Author) » 2011-11-11, 13:59 UTC

You can use a plugin not supporting orientation if that bothers you. The internal viewer is really just the basic solution for people not installing any plugins. It's not my problem if they have buggy images on their system, sorry.

tbeu · Post by *tbeu » 2011-11-11, 14:13 UTC

It is not basic any more if Exif orientation is evaluated. Windows Photo Viewer ignores it here.

MVV · Post by *MVV » 2011-11-11, 14:17 UTC

Christian, I think it is not so hard to provide another INI key allowing to enable/disable rotation according to EXIF.

tbeu · Post by *tbeu » 2011-11-11, 14:46 UTC

Yes, please introduce new INI key ExifRotate in [Lister] section.

Biozynotiker · Post by *Biozynotiker » 2011-11-12, 13:18 UTC

Yes, please introduce new INI key ExifRotate in [Lister] section.

Christian, I think it is not so hard to provide another INI key allowing to enable/disable rotation according to EXIF.

Full Ack. I want to view the images as it is and I don't want to forced of using any non configurable "automatic" settings.

- Please provide an option to enable/disable it
- or at least a hint in the titlebar and an hotkey to view the unrotated image in lister

EDIT:
I thought about a bit and I think, that there should be a general option to enable/disable this AND provide an hotkey (R) to manipulate the image like F,L,C etc. in lister.
This should satisfy all users.

tbeu · Post by *tbeu » 2011-11-25, 15:07 UTC

History.txt wrote:15.11.11 Added: wincmd.ini [Lister] AutoRotate=0 turns off automatic rotation of JPG images according to the EXIF orientation value (32/64)

Auto rotation is no longer working in 64bit-Lister of TC8PB10 no matter what setting for AutoRotate is. JPG image is always displayed without rotation.

Post by *ghisler(Author) » 2011-11-25, 15:11 UTC

Not confirmed, I have just tested it. Maybe the file is loaded by a plugin? Please check the lister title.

You may also have *.jpg in IViewAdditionalTypes - TC then uses that instead of the internal viewer.

tbeu · Post by *tbeu » 2011-11-25, 15:29 UTC

ghisler(Author) wrote:You may also have *.jpg in IViewAdditionalTypes - TC then uses that instead of the internal viewer.

Yes, that was the point. The question is why TC8PB9 always rotates even if IViewAdditionalTypes contains *.jpg.

Auto rotation is working in TC8PB10 but in wrong direction (for left, top and right, bottom only) if compared with sgviewer or imagine plugins.

Lesmo16 · Post by *Lesmo16 » 2011-11-26, 14:58 UTC

tbeu wrote:Auto rotation is working in TC8PB10 but in wrong direction (for left, top and right, bottom only)...

Can't confirm that. Auto-rotation works fine here.

tbeu wrote:... if compared with sgviewer or imagine plugins.

SGViewer never shows/showed proper rotation, Imagine does.

tbeu · Post by *tbeu » 2011-11-26, 19:01 UTC

Please use these test pictures as a check for Exif auto rotation. You should see a letter [face=arial]F[/face] in all cases if auto rotation works correctly. However picture 5 (left, top) and 7 (right, bottom) are wrongly rotated. I checked with an empty wincmd.ini in TC8PB10 both 32 and 64-bit.
For comparison with SGViewer version 1.8.5 is at least required to see auto rotated JPG images.