Merge lp://staging/~hannahermsen/ubuntu/trusty/ristretto/bug-1284111-3 into lp://staging/ubuntu/trusty/ristretto

Proposed by Hanna Hermsen
Status: Needs review
Proposed branch: lp://staging/~hannahermsen/ubuntu/trusty/ristretto/bug-1284111-3
Merge into: lp://staging/ubuntu/trusty/ristretto
Diff against target: 14 lines (+7/-0)
1 file modified
debian/changelog (+7/-0)
To merge this branch: bzr merge lp://staging/~hannahermsen/ubuntu/trusty/ristretto/bug-1284111-3
Reviewer Review Type Date Requested Status
Sebastien Bacher Needs Fixing
Review via email: mp+224938@code.staging.launchpad.net

Description of the change

crysman has 3 feature requests for Ristretto Image Viewer, described here: https://bugs.launchpad.net/ubuntu/+source/ristretto/+bug/1284111

I implemented all three of them. This is the third one, making it clear for the user that the copy that will be made is losless. To do this I added this information in the tooltip of the Save Copy menu item.

As I stated in the first and second merge proposal this is my first bug fix so please tell me if I did it correctly.

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your work, you submitter 3 requests, that one only includes a changelog, seems like it's incorrect/superseeded?

review: Needs Fixing

Unmerged revisions

28. By Hanna Hermsen

The tooltip of the "File -> Save Copy" menu item now makes it clear
that a losless copy will be made. (LP: #1284111)

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

to all changes: