Merge lp://qastaging/~3v1n0/unity/super+tab-shortcut-fixes into lp://qastaging/unity
Proposed by
Marco Trevisan (Treviño)
Status: | Merged |
---|---|
Approved by: | Tim Penhey |
Approved revision: | no longer in the source branch. |
Merged at revision: | 2044 |
Proposed branch: | lp://qastaging/~3v1n0/unity/super+tab-shortcut-fixes |
Merge into: | lp://qastaging/unity |
Prerequisite: | lp://qastaging/~thomir-deactivatedaccount/unity/fix-ap-test-stability |
Diff against target: |
510 lines (+304/-101) 6 files modified
manual-tests/SuperTab.txt (+1/-20) plugins/unityshell/src/Launcher.cpp (+2/-1) plugins/unityshell/src/LauncherController.cpp (+2/-2) plugins/unityshell/src/unityshell.cpp (+6/-0) tests/autopilot/autopilot/emulators/unity/launcher.py (+8/-0) tests/autopilot/autopilot/tests/test_launcher.py (+285/-78) |
To merge this branch: | bzr merge lp://qastaging/~3v1n0/unity/super+tab-shortcut-fixes |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Thomi Richards (community) | Approve | ||
Review via email:
|
This proposal supersedes a proposal from 2012-02-01.
Description of the change
The launcher icons can be shown during the Super Tab Launcher switcher.
For both the shortcut hints and the icons shortcut overlays we use the same policy:
- If they were shown before starting the Super+Tab session, we still show them
- If they were not shown before starting the Super+Tab session, they won't be ever shown
Plus, if we're pressing a valid unity shortcut key when the Launcher Switcher is active, we should terminate the switcher without doing anything.
UNBLOCK
To post a comment you must log in.
Hi Marco,
You're going to hate me for saying this, but we need autopilot tests for this. This should be very easy to test with autopilot. I believe the only thing missing is a way to tell if the shortcut overlay is showing (which would be helpful for other AP tests as well).