gallery-app

gallery-app: Can't zoom out after zooming in

Bug #1289160 reported by Daniel van Vugt
24
This bug affects 5 people
Affects Status Importance Assigned to Milestone
gallery-app
Fix Released
High
Ugo Riboni
gallery-app 14.03
gallery-app (Ubuntu)
Fix Released
High
Ugo Riboni
Ubuntu ubuntu-14.03
gallery-app (Ubuntu RTM)
Fix Released
Undecided
Unassigned

Bug Description

gallery-app: Can't zoom out after zooming in

Test case:
  1. Open a photo.
  2. Zoom in with two fingers.
  3. Lift fingers off and drag around a little.
  4. Zoom out with two-finger pinch.

Expected: Zooms out
Observed: Nothing happens. The image always stays zoomed.

Related branches

lp:~artmello/gallery-app/gallery-app-new_header
Bill Filler (community): Approve
PS Jenkins bot: Needs Fixing (continuous-integration)
Leo Arias (community): Needs Fixing
Ugo Riboni (community): Needs Fixing
Diff: 2678 lines (+641/-1105)
31 files modified
rc/qml/AlbumEditor/AlbumEditor.qml (+9/-12)
rc/qml/AlbumViewer/AlbumViewer.qml (+37/-34)
rc/qml/AlbumsOverview.qml (+5/-6)
rc/qml/Components/MediaSelector.qml (+12/-17)
rc/qml/EventsOverview.qml (+1/-1)
rc/qml/GalleryApplication.qml (+2/-3)
rc/qml/MainScreen.qml (+2/-0)
rc/qml/MediaViewer/GalleryPhotoComponent.qml (+2/-4)
rc/qml/MediaViewer/MediaViewer.qml (+103/-149)
rc/qml/MediaViewer/PhotoViewerDelegate.qml (+0/-129)
rc/qml/MediaViewer/PopupPhotoViewer.qml (+16/-14)
rc/qml/MediaViewer/SingleMediaViewer.qml (+246/-0)
rc/qml/MediaViewer/VideoViewerDelegate.qml (+0/-125)
rc/qml/MediaViewer/ZoomablePhotoComponent.qml (+0/-409)
rc/qml/OrganicView/OrganicView.qml (+1/-1)
rc/qml/PhotosOverview.qml (+1/-1)
rc/qml/PickerScreen.qml (+15/-20)
rc/qml/Utility/PhotosToolbarActions.qml (+19/-13)
rc/qml/Utility/SelectionToolbarAction.qml (+32/-22)
tests/autopilot/gallery_app/emulators/album_view.py (+4/-6)
tests/autopilot/gallery_app/emulators/gallery_utils.py (+3/-3)
tests/autopilot/gallery_app/emulators/main_screen.py (+27/-12)
tests/autopilot/gallery_app/emulators/photo_viewer.py (+4/-5)
tests/autopilot/gallery_app/emulators/picker_screen.py (+3/-3)
tests/autopilot/gallery_app/tests/__init__.py (+6/-5)
tests/autopilot/gallery_app/tests/test_album_editor.py (+14/-18)
tests/autopilot/gallery_app/tests/test_album_view.py (+17/-26)
tests/autopilot/gallery_app/tests/test_albums_view.py (+16/-11)
tests/autopilot/gallery_app/tests/test_events_view.py (+17/-14)
tests/autopilot/gallery_app/tests/test_photo_viewer.py (+8/-23)
tests/autopilot/gallery_app/tests/test_photos_view.py (+19/-19)
lp:ubuntu/utopic-proposed/gallery-app
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Is this the right package? It's called Gallery on screen, but "gallery-app" is not an installed package.

Bill Filler (bfiller)
Changed in gallery-app:
status: New → Confirmed
importance: Undecided → High
assignee: nobody → Ugo Riboni (uriboni)
milestone: none → 14.03
Changed in gallery-app (Ubuntu):
status: New → Confirmed
importance: Undecided → High
assignee: nobody → Ugo Riboni (uriboni)
milestone: none → ubuntu-14.03
Bill Filler (bfiller)
tags: added: desktop-touch
Changed in gallery-app:
importance: High → Critical
Changed in gallery-app (Ubuntu):
importance: High → Critical
Ugo Riboni (uriboni)
Changed in gallery-app:
status: Confirmed → In Progress
Changed in gallery-app (Ubuntu):
status: Confirmed → In Progress
Revision history for this message
Bill Filler (bfiller) wrote :

NOTE: we are working on a fix that will work on touch screens only. This functionality will not be supported with track pads.

Bill Filler (bfiller)
Changed in gallery-app (Ubuntu):
importance: Critical → High
Changed in gallery-app:
importance: Critical → High
tags: added: rtm14
Revision history for this message
Selene ToyKeeper (toykeeper) wrote :

Zooming out is possible, just not by using and un-pinching gesture. Regardless, if it supports pinch it should also support un-pinch.

tags: added: flo mako u124
Pat McGowan (pat-mcgowan)
tags: added: pat
Bill Filler (bfiller)
tags: added: touch-2014-09-11
Revision history for this message
Ugo Riboni (uriboni) wrote :

This problem fix is contained in this branch: https://code.launchpad.net/~artmello/gallery-app/gallery-app-new_header/+merge/231601 (which replaces the toolbar with the new header, fixing the problem since the toolbar was creating a MouseArea on top of the pinch area causing the problem).
It will also require version 1221 of the SDK to be landed.

Revision history for this message
Launchpad Janitor (janitor) wrote :

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

---------------
gallery-app (0.0.67+14.10.20140906-0ubuntu1) utopic; urgency=low

  [ Bill Filler ]
  * Mark strings for translation

  [ Arthur Mello ]
  * Make gallery-app uses the new header (LP: #1289160)

  [ Ugo Riboni ]
  * Move the new album strings to QML to make them translatable. Remove
    some dead code. (LP: #1364310)
 -- Ubuntu daily release <email address hidden> Sat, 06 Sep 2014 21:09:12 +0000

Changed in gallery-app (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gallery-app - 0.0.67+14.10.20140906~rtm-0ubuntu1

---------------
gallery-app (0.0.67+14.10.20140906~rtm-0ubuntu1) 14.09; urgency=low

  [ Bill Filler ]
  * Mark strings for translation

  [ Arthur Mello ]
  * Make gallery-app uses the new header (LP: #1289160)

  [ Ugo Riboni ]
  * Move the new album strings to QML to make them translatable. Remove
    some dead code. (LP: #1364310)
 -- Ubuntu daily release <email address hidden> Sat, 06 Sep 2014 21:09:12 +0000

Changed in gallery-app (Ubuntu RTM):
status: New → Fix Released
Bill Filler (bfiller)
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.
Subscribing...

Other bug subscribers

Remote bug watches

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