Merge lp://qastaging/~ondrej-dolejsi/shutter/ordered-urls-fix into lp://qastaging/shutter

Proposed by Ondrej-dolejsi
Status: Merged
Merged at revision: 1280
Proposed branch: lp://qastaging/~ondrej-dolejsi/shutter/ordered-urls-fix
Merge into: lp://qastaging/shutter
Diff against target: 12 lines (+1/-1)
1 file modified
share/shutter/resources/modules/Shutter/Upload/Shared.pm (+1/-1)
To merge this branch: bzr merge lp://qastaging/~ondrej-dolejsi/shutter/ordered-urls-fix
Reviewer Review Type Date Requested Status
Kyrylo Romanenko (community) code ui Approve
Mario Kemper (Romario) Pending
Review via email: mp+264952@code.qastaging.launchpad.net

Description of the change

After exporting an image to public services the returned links (with their keys descriptions) are displayed in random order. This is frustrating for users, sharing many images a day, because every time the user updates an image she has to search for the direct link, or delete link, not being able to form a muscle memory.

This branch fixes it by sorting the keys alphabetically.

To post a comment you must log in.
Revision history for this message
Kyrylo Romanenko (kromanenko) wrote :

Verified this fix by applying code modification on my "Shared.pm". It works :)

review: Approve (code ui)

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 status/vote changes: