After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 734844 - Portrait images are previewed in landscape mode
Portrait images are previewed in landscape mode
Status: RESOLVED FIXED
Product: gnome-photos
Classification: Applications
Component: general
3.13.x
Other All
: Normal normal
: ---
Assigned To: GNOME photos maintainer(s)
GNOME photos maintainer(s)
: 737546 748398 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2014-08-15 09:11 UTC by Pranav Kant
Modified: 2015-11-18 17:35 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
base-item: Relax an assertion (1.02 KB, patch)
2015-11-18 13:29 UTC, Debarshi Ray 		rejected Details | Review
base-item: Work around GEGL not applying the embedded orientation (8.25 KB, patch)
2015-11-18 13:29 UTC, Debarshi Ray 		rejected Details | Review
base-item: Apply the embedded orientation (7.80 KB, patch)
2015-11-18 17:34 UTC, Debarshi Ray 		committed Details | Review

Description Pranav Kant 2014-08-15 09:11:32 UTC 
This is not the case with all the portrait images but some of them. I am providing links to few of the images that should help in reproducing this bug :

1. https://www.dropbox.com/s/u6igusa63gtgyjq/IMG_5818.JPG
2. https://www.flickr.com/photos/126579960@N02/14625268550/in/pool-guadec2014
Comment 1 Pranav Kant 2014-08-15 09:13:21 UTC 
Also these portrait images looks fine (i.e in portrait mode only) in overview mode, its just the preview mode that changes their orientation.
Comment 2 Marcus Lundblad 2014-08-19 19:14:42 UTC 
Also, the image I supplied display as expected by eog (and gimp).
Comment 3 Pranav Kant 2014-10-12 14:43:48 UTC 
*** Bug 737546 has been marked as a duplicate of this bug. ***
Comment 4 Debarshi Ray 2015-01-14 17:00:16 UTC 
This needs to be fixed in GEGL. See bug 710492
Comment 5 Debarshi Ray 2015-04-24 08:30:18 UTC 
*** Bug 748398 has been marked as a duplicate of this bug. ***
Comment 6 Debarshi Ray 2015-11-18 13:29:00 UTC 
Created attachment 315824 [details] [review]
base-item: Relax an assertion
Comment 7 Debarshi Ray 2015-11-18 13:29:29 UTC 
Created attachment 315825 [details] [review]
base-item: Work around GEGL not applying the embedded orientation
Comment 8 Debarshi Ray 2015-11-18 17:34:06 UTC 
Created attachment 315848 [details] [review]
base-item: Apply the embedded orientation

Here is another approach (suggested by Bastien) where we read the orientation values from Tracker and add use a GEGL operation to apply it when loading.
Comment 9 Debarshi Ray 2015-11-18 17:35:39 UTC 
Comment on attachment 315848 [details] [review]
base-item: Apply the embedded orientation

Pushed to master.