Merge lp://staging/~brandontschaefer/unity/mouse-move-hud-buttons-focus into lp://staging/unity

Proposed by Brandon Schaefer
Status: Merged
Approved by: Tim Penhey
Approved revision: no longer in the source branch.
Merged at revision: 2663
Proposed branch: lp://staging/~brandontschaefer/unity/mouse-move-hud-buttons-focus
Merge into: lp://staging/unity
Diff against target: 322 lines (+140/-22)
4 files modified
hud/HudButton.cpp (+17/-1)
hud/HudView.cpp (+62/-1)
hud/HudView.h (+6/-0)
tests/autopilot/unity/tests/test_hud.py (+55/-20)
To merge this branch: bzr merge lp://staging/~brandontschaefer/unity/mouse-move-hud-buttons-focus
Reviewer Review Type Date Requested Status
Tim Penhey (community) Approve
Gord Allott (community) Approve
jenkins continuous-integration Pending
Review via email: mp+121966@code.staging.launchpad.net

Commit message

The mouse will now cause the HUD buttons to change selection.

Description of the change

=== Problem ===
Moving the mouse over a Hud button does not change its visual focus.

=== Fix ===
Using mouse_enter/leave/move to change the visual focus of the Hud button to follow the mouse.

=== Tests ===
AP tests

To post a comment you must log in.
Revision history for this message
Gord Allott (gordallott) wrote :

seems okay to me, +1

review: Approve
Revision history for this message
Tim Penhey (thumper) wrote :

Approving dead code removal.

review: Approve

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.