Merge lp:~brandontschaefer/unity/hud-to-dash-fix into lp:unity
Proposed by
Brandon Schaefer
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Tim Penhey | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 2215 | ||||
Proposed branch: | lp:~brandontschaefer/unity/hud-to-dash-fix | ||||
Merge into: | lp:unity | ||||
Diff against target: |
92 lines (+30/-3) 5 files modified
plugins/unityshell/src/HudController.cpp (+2/-0) plugins/unityshell/src/LauncherController.cpp (+7/-0) plugins/unityshell/src/LauncherController.h (+2/-0) plugins/unityshell/src/unityshell.cpp (+3/-0) tests/autopilot/autopilot/tests/test_hud.py (+16/-3) |
||||
To merge this branch: | bzr merge lp:~brandontschaefer/unity/hud-to-dash-fix | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Penhey (community) | Approve | ||
Review via email: mp+100064@code.launchpad.net |
Commit message
Fixes the pinyin pop up box disappearing in the hud.
Description of the change
== Problem ==
When the Hud gets closed it doesn't emit end_key_nav, which resets the preedit box for ibus-pinyin engine. So when you reopened the Hud it wouldn't redraw the preedit window, because it didn't know it disappeared.
== Fix ==
The fix was to force Hud to lose key nav mode. This caused a problem with key focus when switching from the Hud to the Dash, this was fixed by making sure the Hud closes BEFORE the Dash opens.
== Test ==
Autopilot test
To post a comment you must log in.
At the end of test_hud_ to_dash_ has_key_ focus you shouldn't need to ensure_hidden on the dash.
If you do because it is a hud test, then change it so the test goes like this:
self. dash.ensure_ visible( ) addCleanup( self.dash. ensure_ hidden)
self.
That way if the assert fails, the dash is still hidden.
Also, PEP-8 says only one blank line between class methods.