Merge lp:~azzar1/unity/fix-858469 into lp:unity

Proposed by Andrea Azzarone
Status: Merged
Merged at revision: 1618
Proposed branch: lp:~azzar1/unity/fix-858469
Merge into: lp:unity
Diff against target: 12 lines (+1/-1)
1 file modified
plugins/unityshell/src/Launcher.cpp (+1/-1)
To merge this branch: bzr merge lp:~azzar1/unity/fix-858469
Reviewer Review Type Date Requested Status
Neil J. Patel (community) Approve
Review via email: mp+76894@code.launchpad.net

Description of the change

Don't jump during launcher keynav.

To reproduce the problem:
1. With a "filled" launcher bar press Alt+F1 (make sure to move your mouse outside of the launcher bar area to not hit bug 858502)
2. Press the down key

RESULT:
* The launcher items jump up and down.

---------------------------------------------------------------

I think this is not the best solution but this is a kind of workaround. Given that we are now at an advanced stage of the release cycle, I think it is better that launcher.cpp does not change too much (Launcher.cpp is not a file, is a puzzle :P )

To post a comment you must log in.
Revision history for this message
Hernando Torque (htorque) wrote :

Tested and works fine.

Revision history for this message
Andrea Azzarone (azzar1) wrote :

We should wait that ayatana design team expresses his opinion about bug #858794 becuase this branch deactivate autoscroll during keynav.

Revision history for this message
Neil J. Patel (njpatel) wrote :

Works well

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/unityshell/src/Launcher.cpp'
2--- plugins/unityshell/src/Launcher.cpp 2011-09-22 17:03:22 +0000
3+++ plugins/unityshell/src/Launcher.cpp 2011-09-25 09:25:28 +0000
4@@ -1916,7 +1916,7 @@
5 Launcher* self = (Launcher*) data;
6 nux::Geometry geo = self->GetGeometry();
7
8- if (!self->_hovered || self->GetActionState() == ACTION_DRAG_LAUNCHER)
9+ if (self->_keynav_activated || !self->_hovered || self->GetActionState() == ACTION_DRAG_LAUNCHER)
10 return TRUE;
11
12 if (self->MouseOverTopScrollArea())