Merge lp://qastaging/~didrocks/unity/add-super-shortcut-logic-721264 into lp://qastaging/unity

Proposed by Didier Roche-Tolomelli
Status: Merged
Merged at revision: 872
Proposed branch: lp://qastaging/~didrocks/unity/add-super-shortcut-logic-721264
Merge into: lp://qastaging/unity
Diff against target: 238 lines (+119/-25)
5 files modified
src/Launcher.cpp (+95/-24)
src/Launcher.h (+2/-0)
src/LauncherModel.cpp (+16/-0)
src/LauncherModel.h (+1/-0)
src/unityshell.cpp (+5/-1)
To merge this branch: bzr merge lp://qastaging/~didrocks/unity/add-super-shortcut-logic-721264
Reviewer Review Type Date Requested Status
Jason Smith (community) Approve
Review via email: mp+50372@code.qastaging.launchpad.net

Description of the change

- Add the super shortcuts logic (LP: #721264)
- Cleanup a little bit the Keypress function

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

Please make your switch block simply convert to an int, then outside the block do the activate.

review: Needs Fixing
Revision history for this message
Jason Smith (jassmith) wrote :

nevermind last, didn't see full switch structure in diff

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.