Merge lp://qastaging/~didrocks/unity/launcher-fix-of-the-week-04-04-11 into lp://qastaging/unity

Proposed by Didier Roche-Tolomelli
Status: Merged
Merged at revision: 1075
Proposed branch: lp://qastaging/~didrocks/unity/launcher-fix-of-the-week-04-04-11
Merge into: lp://qastaging/unity
Diff against target: 731 lines (+225/-84)
13 files modified
src/BamfLauncherIcon.cpp (+1/-6)
src/DeviceLauncherSection.cpp (+6/-1)
src/DeviceLauncherSection.h (+1/-0)
src/Launcher.cpp (+144/-46)
src/Launcher.h (+17/-2)
src/LauncherHideMachine.cpp (+30/-4)
src/LauncherHideMachine.h (+4/-1)
src/LauncherHoverMachine.cpp (+11/-11)
src/LauncherHoverMachine.h (+2/-2)
src/PanelTray.cpp (+4/-9)
src/PanelTray.h (+0/-2)
src/PlacesController.cpp (+3/-0)
src/PlacesResultsController.cpp (+2/-0)
To merge this branch: bzr merge lp://qastaging/~didrocks/unity/launcher-fix-of-the-week-04-04-11
Reviewer Review Type Date Requested Status
Jason Smith (community) Approve
Review via email: mp+56415@code.qastaging.launchpad.net

Description of the change

This branch:
- fix an issue with repeated key (LP: #750535)
- reduce shortcuts label size and change some super behaviour (LP: #747812)
- additional cleanage (signals and code)
- prevents false positive "bring the dash" positive even when we are in scale
  mode (not directly handled by unity) (LP: #751102)
- Avoid spamming the launcher with state change when the state is finally the
  same. (Launcher and Hover machine). That will hopefully fix the "click
  through" bug when the Launcher wrongly stay visible (half of bug LP: #744325)
- move the position of the icon on timeout drag. Fix a potential crasher as well
  (LP: #751196)

To post a comment you must log in.
Revision history for this message
Jason Smith (jassmith) wrote :

+1 approve

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.