Gallery flickers when going fullscreen

Bug #1425563 reported by Omer Akram
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
High
Bill Filler
gallery-app
Fix Released
High
Arthur Mello
gallery-app (Ubuntu)
Fix Released
Undecided
Unassigned
Vivid
Confirmed
Undecided
Unassigned

Bug Description

image 122 vivid-proposed

Open a photo in gallery and tap on it once its open to hide the frame.

What happens: screen flickers when it goes fullscreen

Tags: ww11

Related branches

Changed in gallery-app:
status: New → Confirmed
Revision history for this message
Sebastien Bacher (seb128) wrote :

confirmed on rtm 247

Changed in gallery-app:
importance: Medium → High
Bill Filler (bfiller)
Changed in gallery-app:
assignee: nobody → Arthur Mello (artmello)
tags: added: ww11
Arthur Mello (artmello)
Changed in gallery-app:
status: Confirmed → In Progress
Changed in canonical-devices-system-image:
status: New → In Progress
importance: Undecided → High
assignee: nobody → Bill Filler (bfiller)
milestone: none → ww13-ota
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

---------------
gallery-app (0.0.67+15.04.20150318-0ubuntu1) vivid; urgency=medium

  [ Arthur Mello ]
  * Do not update PhotoViewer Image sourceSize everytime the app is
    resized on devices. That way we will not flicker the image everytime
    we toggle fullscreen Animate Image resize (LP: #1425563)
  * Replace the built-in photo editor with the one from ubuntu-ui-extras

  [ CI Train Bot ]
  * New rebuild forced.

  [ Olivier Tilloy ]
  * Translation template update.

  [ Ugo Riboni ]
  * Replace the built-in photo editor with the one from ubuntu-ui-extras
 -- CI Train Bot <email address hidden> Wed, 18 Mar 2015 16:37:33 +0000

Changed in gallery-app (Ubuntu):
status: New → Fix Released
Changed in canonical-devices-system-image:
status: In Progress → Fix Released
Revision history for this message
John McAleely (john.mcaleely) wrote :

Is the status for gallery-app wrong, ie is there still a fix to land?

Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

This is clearly not fixed

Changed in canonical-devices-system-image:
milestone: ww13-ota → ww17-2015
status: Fix Released → In Progress
Changed in gallery-app (Ubuntu):
status: Fix Released → Confirmed
Changed in canonical-devices-system-image:
milestone: ww17-2015 → ww18-2015
Revision history for this message
Tim Peeters (tpeeters) wrote :

I cannot see any flickering (i.e., frames of black or some other color inbetween the rendering of the resizing image).

What I do see is the status bar and application header hiding/unhiding at the same time, but they are not synchronized. Because of this the space for the photo is changing in a non-fluent way which causes a jittery animation. Is this what the bug is referring to?

If the jittering is a problem, I think it could be solved by instead of a header that animates out, using a semi-transparent overlay header (like in the camera app photo roll). We could also link the header to the status bar so they animate together but that would be something that we cannot do quickly since they are handled by different projects (unity vs the app).

Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

Can we land the outstanding MR for this?

Changed in canonical-devices-system-image:
milestone: ww18-2015 → ww26-2015
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

Changed in gallery-app (Ubuntu Vivid):
status: New → Confirmed
Bill Filler (bfiller)
Changed in canonical-devices-system-image:
milestone: ww26-2015 → ww28-2015
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

---------------
gallery-app (0.0.67+15.10.20150629-0ubuntu1) wily; urgency=medium

  [ Arthur Mello ]
  * Add a timer before going fullscreen while on image viewer to reduce
    flickering (LP: #1425563)
  * Make sure that DateTimeDigitized is the first tag evaluated to get
    exposure time added: tests/unittests/photo-metadata/
    tests/unittests/photo-metadata/CMakeLists.txt tests/unittests/photo-
    metadata/images/ tests/unittests/photo-metadata/images/sample01.jpg
    tests/unittests/photo-metadata/tst_photo-metadata.cpp
  * Make sure that photos that are opened from scopes and are not on
    gallery library will be displayed correctly (LP: #1389514)
  * update icon (LP: #1457424)

  [ CI Train Bot ]
  * New rebuild forced.

  [ Timo Jyrinki ]
  * Allow translating the date strings in EventCard.qml, update .pot
    file (LP: #1457357) (LP: #1457357)

 -- CI Train Bot <email address hidden> Mon, 29 Jun 2015 19:04:05 +0000

Changed in gallery-app (Ubuntu):
status: Confirmed → Fix Released
Changed in canonical-devices-system-image:
status: In Progress → Fix Released
Changed in gallery-app:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.