Merge lp:~diegosarmentero/ubuntuone-client/ubuntuone-client-menuorder into lp:ubuntuone-client
Proposed by
Diego Sarmentero
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Diego Sarmentero | ||||
Approved revision: | 1330 | ||||
Merged at revision: | 1326 | ||||
Proposed branch: | lp:~diegosarmentero/ubuntuone-client/ubuntuone-client-menuorder | ||||
Merge into: | lp:ubuntuone-client | ||||
Prerequisite: | lp:~diegosarmentero/ubuntuone-client/ubuntuone-client-timer | ||||
Diff against target: |
92 lines (+45/-11) 2 files modified
tests/platform/sync_menu/test_linux.py (+34/-4) ubuntuone/platform/sync_menu/linux.py (+11/-7) |
||||
To merge this branch: | bzr merge lp:~diegosarmentero/ubuntuone-client/ubuntuone-client-menuorder | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Manuel de la Peña (community) | Approve | ||
dobey (community) | Approve | ||
Review via email: mp+125768@code.launchpad.net |
Commit message
- Sort items based on the written value, to prioritize the files being transferred (LP: #1052956).
To post a comment you must log in.
The implications of how this might behave, frighten me a little. A video showing this in action in an open transfers menu, which remains open for many (at least 30) seconds, would be quite helpful.
But based on the bug report, and that this code will presumably be updated many times while the menu is being examined by a user, it seems like the ordering may change. And having menu items constantly changing position would be quite disturbing for the user; even if the menu items are not clickable (as they do not seem to be).