Merge lp://qastaging/~didrocks/unity/launcher-bug-fix-fest into lp://qastaging/unity

Proposed by Didier Roche-Tolomelli
Status: Merged
Merged at revision: 1032
Proposed branch: lp://qastaging/~didrocks/unity/launcher-bug-fix-fest
Merge into: lp://qastaging/unity
Diff against target: 818 lines (+281/-128)
14 files modified
src/BamfLauncherIcon.cpp (+0/-9)
src/BamfLauncherIcon.h (+0/-1)
src/Launcher.cpp (+59/-63)
src/Launcher.h (+6/-3)
src/LauncherHoverMachine.cpp (+119/-0)
src/LauncherHoverMachine.h (+64/-0)
src/LauncherIcon.cpp (+22/-2)
src/LauncherIcon.h (+2/-1)
src/PlaceEntryRemote.cpp (+1/-1)
src/PlaceLauncherIcon.cpp (+0/-11)
src/PlaceLauncherIcon.h (+0/-1)
src/SimpleLauncherIcon.cpp (+0/-3)
src/TrashLauncherIcon.cpp (+7/-30)
src/TrashLauncherIcon.h (+1/-3)
To merge this branch: bzr merge lp://qastaging/~didrocks/unity/launcher-bug-fix-fest
Reviewer Review Type Date Requested Status
Jason Smith (community) Approve
Review via email: mp+55182@code.qastaging.launchpad.net

Description of the change

This branch intends to:
- don't show empty shortcut for places not defining a label (LP: #742985)
- use regular and not tweaked quicklist system for trashlaunchericon (LP:
  #741793)
- clean the activation, QL, opennewinstance stack, as it triggered a lot of bugs
  and is more straightforward now (LP: #744196)
- only open quicklist if there is one to open (LP: #741772)
- add an HoverMachine and ensure we don't hover when tapping super (LP: #744344)

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

+1 Awesome work didier!

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.