Merge lp:~townsend/unity/fix-lp1267888 into lp:unity
Proposed by
Christopher Townsend
Status: | Merged |
---|---|
Approved by: | Brandon Schaefer |
Approved revision: | no longer in the source branch. |
Merged at revision: | 3622 |
Proposed branch: | lp:~townsend/unity/fix-lp1267888 |
Merge into: | lp:unity |
Diff against target: |
35 lines (+4/-10) 2 files modified
launcher/ApplicationLauncherIcon.cpp (+2/-8) tests/test_application_launcher_icon.cpp (+2/-2) |
To merge this branch: | bzr merge lp:~townsend/unity/fix-lp1267888 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Brandon Schaefer (community) | Approve | ||
Review via email: mp+201255@code.launchpad.net |
Commit message
When using the mouse scrollwheel on inactive Launcher icons with unfocused windows, the unfocused window should not be focused.
Description of the change
= Issue =
In bug #1081843, the design for using the mouse scroll wheel over Launcher icons of unfocused apps led to an inconsistent behavior. In that bug, Mark Shuttleworth said the design was wrong.
= Fix =
Do not focus any windows if the Launcher icon is not active.
= Test =
Modified the already in place unit test to account for the new, correct behavior.
To post a comment you must log in.
LGTM