Version: Comes with KDE (using KDE 4.4.2) OS: Linux KolourPaint don't respect the EXIF orientation flag : he cannot recognize when a picture has been rotated using a EXIF rotation flag. Downstream bug report : https://bugs.launchpad.net/ubuntu/+bug/644168 Reproducible: Always Steps to Reproduce: 1) Rotate a picture using Shotwell for example, and save it. 2) Open the picture with KolourPaint Actual Results: The picture displayed by KolourPaint is not rotated. Expected Results: The picture displayed by KolourPaint is rotated. OS: Linux (i686) release 2.6.32-24-generic Compiler: cc
Yes. This is due to the fact that Qt does not provide the EXIF flags; see https://bugreports.qt.nokia.com/browse/QTBUG-9414 I'd need to implement it somehow separately... gwenview uses some external lib for that I think.
Git commit a149002600d8d697fa7136488f0d4703054a3a4c by Martin Koller. Committed on 26/10/2016 at 16:51. Pushed by mkoller into branch 'master'. Respect EXIF rotation information on loading an image FIXED-IN: 16.12 M +3 -2 CMakeLists.txt M +6 -1 document/kpDocument_Open.cpp http://commits.kde.org/kolourpaint/a149002600d8d697fa7136488f0d4703054a3a4c
You may have to check the runtime Qt version, because there are versions that automatically rotate JPEG according to EXIF.
Additionally, since Qt 5.5, there is API to read the image orientation, see https://doc.qt.io/qt-5/qimageiohandler.html#Transformation-enum
... and http://doc-snapshots.qt.io/qt5-dev/qimagereader.html#setAutoTransform actually transforms (i.e. rotates) the image automatically, if set to true (this is false because of compatibility reasons). There is no reason to hardcode EXIF support.
On Tuesday 22 November 2016 22:47:31 you wrote: > https://bugs.kde.org/show_bug.cgi?id=252003 > > --- Comment #5 from Christoph Feck <cfeck@kde.org> --- > ... and http://doc-snapshots.qt.io/qt5-dev/qimagereader.html#setAutoTransform > actually transforms (i.e. rotates) the image automatically, if set to true > (this is false because of compatibility reasons). > > There is no reason to hardcode EXIF support. yes, I'll have a look
Git commit 8747c55692ac41f80ca1cce5633bc892c6b3d842 by Martin Koller. Committed on 27/11/2016 at 07:53. Pushed by mkoller into branch 'master'. get rid of KExiv dependency since Qt5.5 can already auto-rotate images M +2 -4 CMakeLists.txt M +6 -6 document/kpDocument_Open.cpp https://commits.kde.org/kolourpaint/8747c55692ac41f80ca1cce5633bc892c6b3d842
*** Bug 373011 has been marked as a duplicate of this bug. ***