Merge lp:~fboucault/camera-app/slideshow_use_photo_not_thumbnail into lp:camera-app

Proposed by Florian Boucault
Status: Merged
Approved by: Ugo Riboni
Approved revision: 403
Merged at revision: 402
Proposed branch: lp:~fboucault/camera-app/slideshow_use_photo_not_thumbnail
Merge into: lp:camera-app
Diff against target: 12 lines (+1/-1)
1 file modified
SlideshowView.qml (+1/-1)
To merge this branch: bzr merge lp:~fboucault/camera-app/slideshow_use_photo_not_thumbnail
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing
Ubuntu Phablet Team Pending
Review via email: mp+237093@code.launchpad.net

Commit message

Photo roll's slideshow view: use the photo downscaled instead of the thumbnailer.

To post a comment you must log in.
403. By Florian Boucault

Empty commit

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ugo Riboni (uriboni) wrote :

LGTM and does what it says

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'SlideshowView.qml'
2--- SlideshowView.qml 2014-09-30 22:41:12 +0000
3+++ SlideshowView.qml 2014-10-03 15:43:43 +0000
4@@ -179,7 +179,7 @@
5 anchors.fill: parent
6 asynchronous: true
7 cache: false
8- source: slideshowView.inView ? "image://thumbnailer/" + fileURL.toString() : ""
9+ source: slideshowView.inView ? (media.isVideo ? "image://thumbnailer/" + fileURL.toString() : fileURL) : ""
10 sourceSize {
11 width: listView.maxDimension
12 height: listView.maxDimension

Subscribers

People subscribed via source and target branches