Merge lp://qastaging/~philip.scott/pantheon-photos/correct-key into lp://qastaging/~pantheon-photos/pantheon-photos/trunk

Proposed by Felipe Escoto
Status: Rejected
Rejected by: Zisu Andrei
Proposed branch: lp://qastaging/~philip.scott/pantheon-photos/correct-key
Merge into: lp://qastaging/~pantheon-photos/pantheon-photos/trunk
Diff against target: 12 lines (+1/-1)
1 file modified
data/pantheon-photos-viewer.desktop.in.in (+1/-1)
To merge this branch: bzr merge lp://qastaging/~philip.scott/pantheon-photos/correct-key
Reviewer Review Type Date Requested Status
Zisu Andrei (community) Disapprove
Review via email: mp+301503@code.qastaging.launchpad.net

Commit message

Use %u on the Exec key

Description of the change

%u is for a single file per command whether %U is for a list of files. This should make Photos keep it's current behavior when https://code.launchpad.net/~jeremywootten/pantheon-files/fix-multiple-file-launch is merged

To post a comment you must log in.
Revision history for this message
Zisu Andrei (matzipan) wrote :

With this branch, it still opens two instances.

review: Disapprove

Unmerged revisions

2978. By Felipe Escoto

use correct Exec key

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches