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

Proposed by Hanna Hermsen
Status: Needs review
Proposed branch: lp://staging/~hannahermsen/ubuntu/trusty/ristretto/bug-1284111-1
Merge into: lp://staging/ubuntu/trusty/ristretto
Diff against target: 37 lines (+12/-0)
2 files modified
debian/changelog (+6/-0)
src/main_window.c (+6/-0)
To merge this branch: bzr merge lp://staging/~hannahermsen/ubuntu/trusty/ristretto/bug-1284111-1
Reviewer Review Type Date Requested Status
Sebastien Bacher Needs Information
Review via email: mp+224935@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 first one, pre-filling the text box in the "File -> Save Copy" dialogue with the image file name.

As I stated elsewhere 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 :

Thanks, how is that different from https://code.launchpad.net/~hannahermsen/ubuntu/trusty/ristretto/bug-1284111-2/+merge/224936? If those are different fixes, maybe you could combine them in one update?

review: Needs Information

Unmerged revisions

28. By Hanna Hermsen

Filename is now pre-filled in the "Save Copy" dialogue. (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: