Bug 164605 - Picture frame: does not honor EXIF rotation tags
Summary: Picture frame: does not honor EXIF rotation tags
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: widget-frame (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR wishlist
Target Milestone: ---
Assignee: Anne-Marie Mahfouf
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-06-21 19:05 UTC by Frederik Himpe
Modified: 2008-11-17 11:05 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Frederik Himpe 2008-06-21 19:05:45 UTC
Version:           4.0.83 (using Devel)
Installed from:    Compiled sources

The picture frame does not rotate pictures which have the EXIF rotation tag set. This way, a picture which is shown correctly rotated (in "portrait" mode) in gwenview, is wrongly shown in "landscape" mode in the picture frame.
Comment 1 Anne-Marie Mahfouf 2008-06-21 20:07:03 UTC
Can you explain more about this EXIF rotation tag? Do other slideshows usually support that? (for ex in kde3 does the screensaver slideshow support it?)
Comment 2 Frederik Himpe 2008-06-21 20:26:16 UTC
Gwenview from KDE 4 supports it, as does The Gimp, eog and f-spot. I'm attaching an image.
Comment 3 Frederik Himpe 2008-06-21 20:36:58 UTC
The file is too large to attach it here. It can be downloaded from http://artipc10.vub.ac.be/files/109_0986.jpg

Comment 4 Anne-Marie Mahfouf 2008-06-21 20:55:24 UTC
GwenView and The Gimp are full images programs, a Plasma applet is not that. 
Having asked the Krita devel (thanks Cyrille) I am swiching this as a wish because the programming practicality is not evident (depending on libexif which is in kdegraphics). 

Can't a picture shown rightly in Gwenview be saved rightly and so we would get rid of this tag?
Comment 5 Anne-Marie Mahfouf 2008-06-21 21:05:35 UTC
My question is: if you are to show your pics to your friends (using a TV with the pics on a CD for example or any other prog, KView comes to mind) wouldn't you rotate them before? Are there any "view only" programs that will use EXIF standard?

Precision: libkexiv2 is in KDE graphics and is a KDE lib.
Comment 6 Frederik Himpe 2008-06-21 23:56:25 UTC
I've never used KView, I've always been using Gwenview, eog, Digikam and F-Spot, which work fine.

When I open the above URL in Konqueror 4.0.83, the image is shown correctly too. I guess that either Konqueror itself already support it, either Konqueror uses something like gwenview to view the picture. Could not the picture frame do something similar, such as using the gwenview viewer engine if it's available?
Comment 7 lucas 2008-06-22 06:04:30 UTC
Almost every single digital camera uses EXIF rotation to 1) Save processing power, and 2) Prevent unneeded image degradation. Having to use an image editor to manually rotate every photo taken so that it can be displayed as a slideshow in the picture frame isn't really the easiest/fastest thing to do.
Comment 8 Joe Kowalski 2008-06-22 06:52:46 UTC
This would be really nice to have. I rarely manually rotate photos any more since digikam & gwenview use the exif rotation so effectively, so it is a little jarring to see the photos not rotated. Perhaps libkexiv could be moved into kdelibs or kdesupport so it could be more freely depended on?
Comment 9 Fred Albrecht 2008-09-09 21:22:28 UTC
Nowadays, when most cameras take pictures, an embarked sensor records the position of the camera while the picture is taken (note: this does not work when taking pictures vertically, be it up or down). The data is saved as part of the EXIF block. All current image display programs rotate the images according to this data to show it right side up. 
The EXIF standard (duplicated at EXIF.org) should cover this (since pretty much everybody else manages to do it). There's a fair chance that whatever lib you use to display the pics already knows how to do this if you ask it nice.
Comment 10 Anne-Marie Mahfouf 2008-11-10 16:16:48 UTC
EXIF support in konqueror depends on kdegraphics. So I guess it can be done easily with kexiv2 but will only work with kdegraphics installed, like Konqueror does.
Comment 11 xavy94 2008-11-12 14:24:00 UTC
*** This bug has been confirmed by popular vote. ***
Comment 12 Jakub Stachowski 2008-11-17 11:05:25 UTC
SVN commit 885403 by qbast:

Autorotation based on EXIF information. Works only if libkexiv2 was found.
BUG: 164605



 M  +19 -3     CMakeLists.txt  
 M  +20 -0     picture.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=885403