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

Proposed by Andrea Azzarone
Status: Merged
Merged at revision: 1618
Proposed branch: lp://qastaging/~azzar1/unity/fix-858469
Merge into: lp://qastaging/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://qastaging/~azzar1/unity/fix-858469
Reviewer Review Type Date Requested Status
Neil J. Patel (community) Approve
Review via email: mp+76894@code.qastaging.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
The diff is not available at this time. You can reload the page or download it.