Merge lp://staging/~philip.scott/screenshot-tool/dont-overwrite into lp://staging/~elementary-apps/screenshot-tool/trunk

Proposed by Felipe Escoto
Status: Merged
Approved by: Danielle Foré
Approved revision: 232
Merged at revision: 234
Proposed branch: lp://staging/~philip.scott/screenshot-tool/dont-overwrite
Merge into: lp://staging/~elementary-apps/screenshot-tool/trunk
Diff against target: 24 lines (+13/-1)
1 file modified
src/ScreenshotWindow.vala (+13/-1)
To merge this branch: bzr merge lp://staging/~philip.scott/screenshot-tool/dont-overwrite
Reviewer Review Type Date Requested Status
Danielle Foré Approve
Review via email: mp+307121@code.staging.launchpad.net

Commit message

Don't overwrite files

Description of the change

Don't overwrite files, instead append the (%d) as when you download a file

To post a comment you must log in.
Revision history for this message
Danielle Foré (danrabbit) wrote :

Just one tiny code style issue ;)

232. By Felipe Escoto

Don't overwrite files

Revision history for this message
Danielle Foré (danrabbit) wrote :

Works and looks good to me :)

review: Approve

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: