Canonical System Image

Photos are displayed as 90 degree rotated in photo roll

Bug #1541210 reported by Lorn Potter on 2016-02-03

This bug affects 3 people

	Status	Importance	Assigned to	Milestone
Canonical System Image	Fix Released	High	Bill Filler	Canonical System Image 10 "OTA10"
camera-app (Ubuntu)	Fix Released	High	Florian Boucault
gallery-app (Ubuntu)	Fix Released	High	Florian Boucault

Bug Description

On some devices where the camera is such that the Orientation exif data gets written "Orientation |Right-top", as on Turbo.

The camera app has problems showing the photo in the photo roll. Gallery also has this problem. Other apps will show it normally.
The camera app view finder shows it as it should.

When taking photo is portrait mode, the photo roll shows it rotated wrong.

When taking photo in landscape mode, the photo roll shows it properly.

The "My Photos" scope shows both portrait and landscape taken photos properly.

---

This also affects photos transferred from elsewhere to the phone and viewed in Gallery.

See original description

Related branches

lp:~fboucault/ubuntu-ui-extras/fix_exif_orientation

Merged into lp:ubuntu-ui-extras at revision 85

Arthur Mello (community): Approve on 2016-01-13

PS Jenkins bot: Approve (continuous-integration) on 2015-06-10

Ugo Riboni: Pending requested 2015-04-29

lp:~artmello/gallery-app/gallery-app-remove_image_provider

Merged into lp:gallery-app at revision 1275

Florian Boucault (community): Needs Fixing on 2016-01-15

PS Jenkins bot: Needs Fixing (continuous-integration) on 2016-01-14

Jean-Baptiste Lallement (jibel) on 2016-02-03

Changed in canonical-devices-system-image:
importance:	Undecided → High
assignee:	nobody → Bill Filler (bfiller)
milestone:	none → ww08-2016

Revision history for this message

Launchpad Janitor (janitor) wrote on 2016-02-03:

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in camera-app (Ubuntu):
status:	New → Confirmed
Changed in gallery-app (Ubuntu):
status:	New → Confirmed

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2016-02-03:

Confirming. My old photos (not taken with Ubuntu phone) transferred to Bq are rotated 90 degrees when viewed in Gallery. And the same in Camera app's photo roll if they are placed in the com.ubuntu.camera subfolder.

I believe this has not been the case always, but due to some newish change.

Bq E4.5 on OTA-9.

description:

updated

Pat McGowan (pat-mcgowan) on 2016-02-04

Changed in canonical-devices-system-image:
status:	New → Confirmed

Revision history for this message

Florian Boucault (fboucault) wrote on 2016-02-04:

@Lorn: when you say "When taking photo is portrait mode, the photo roll shows it rotated wrong." you mean taking a photo with the Ubuntu Phone in portrait?

@Lorn & @Timo: could you attach the sample photos you are testing with? thank you!

Revision history for this message

Florian Boucault (fboucault) wrote on 2016-02-04:

And are thumbnails rotated wrong too in the camera photo roll's grid view? in the gallery?

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2016-02-09:

20131220_006.jpg Edit (699.4 KiB, image/jpeg)

@Florian: Not rotated wrong in gallery. Only when opening.

Attached a photo (not taken on Ubuntu phone). I believe what Lorn may have been saying that it has happens on his Turbo device even with Ubuntu, possibly because of hardware / tagging differences.

I haven't tested this on xenial, it might be useful to see if Qt 5.5 is any different.

I guess what needs to be understood is the whole chain of possible rotations of image - camera hardware, Ubuntu software liek qtubuntu-camera adding a possible rotation bit based on some sensor data, app possibly utilising the rotation bit from jpeg, and Qt possibily utilising the rotation bit from jpeg.

Revision history for this message

Lorn Potter (lorn-potter) wrote on 2016-02-17:

Sent a private email of photos of the device to Florian detailing the problem with photoroll.

Revision history for this message

Florian Boucault (fboucault) wrote on 2016-02-17:

Very bad issue indeed!
Thankfully the following MRs fixes it:
https://code.launchpad.net/~fboucault/ubuntu-ui-extras/fix_exif_orientation/+merge/257782
https://code.launchpad.net/~artmello/gallery-app/gallery-app-remove_image_provider/+merge/282649

Silo ready here:
https://requests.ci-train.ubuntu.com/#/ticket/867

Changed in camera-app (Ubuntu):
importance:	Undecided → High
status:	Confirmed → In Progress
Changed in canonical-devices-system-image:
status:	Confirmed → In Progress
Changed in gallery-app (Ubuntu):
status:	Confirmed → In Progress
Changed in camera-app (Ubuntu):
assignee:	nobody → Florian Boucault (fboucault)
Changed in gallery-app (Ubuntu):
assignee:	nobody → Florian Boucault (fboucault)
importance:	Undecided → High

Jean-Baptiste Lallement (jibel) on 2016-02-29

Changed in canonical-devices-system-image:
status:	In Progress → Fix Committed

Florian Boucault (fboucault) on 2016-03-01

Changed in camera-app (Ubuntu):
status:	In Progress → Fix Committed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2016-03-02:

This bug was fixed in the package gallery-app - 0.0.67+16.04.20160225.1-0ubuntu1

---------------
gallery-app (0.0.67+16.04.20160225.1-0ubuntu1) xenial; urgency=medium

  [ Arthur Mello ]
  * Add a timer to make sure all taken photos are added into collection
    in a batch (LP: #1521365)
  * Add missing pkg dependency for qtdeclarative5-ubuntu-thumbnailer0.1
    (LP: #1531604)
  * Fix action button calls on AP tests
  * Following some changes on thumbnailer image provider we need to
    handle Image.Error status for videos that are unable to load
    correctly (LP: #1542083)
  * Make sure to request a thumbnail big enough to handle all the
    possible sizes that way media items will not request a new one every
    time it changes its position. (LP: #1529452)
  * Remove photo-image-provider to use the one provided by ubunut-ui-
    extras Move photo-metadata from photoeditor to photo folder (LP:
    #1541210)
  * Set albums created from PopupAlbumPicker as not new, so they will
    not be deleted after editing (LP: #1483870)
  * Show a checkered background as a background for transparent images
    (LP: #1548855)