Unity

Merge lp:~didrocks/unity/enter-is-good-for-you into lp:unity

  1. enter-is-good-for-you
  2. Merge into trunk
Proposed by Didier Roche-Tolomelli
Status: Merged
Merged at revision: 1129
Proposed branch: lp:~didrocks/unity/enter-is-good-for-you
Merge into: lp:unity
Diff against target: 23 lines (+2/-0)
2 files modified
src/Launcher.cpp (+1/-0)
src/QuicklistView.cpp (+1/-0)
To merge this branch: bzr merge lp:~didrocks/unity/enter-is-good-for-you
Reviewer Review Type Date Requested Status
Unity Team Pending
Review via email: mp+57564@code.launchpad.net

Description of the change

support numpad Enter in Launcher and Quicklist (LP: #760003)

To post a comment you must log in.
Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

review approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/Launcher.cpp'
--- src/Launcher.cpp 2011-04-13 14:12:47 +0000
+++ src/Launcher.cpp 2011-04-13 19:03:28 +0000
@@ -3278,6 +3278,7 @@
32783278
3279 // <RETURN> (start/activate currently selected icon)3279 // <RETURN> (start/activate currently selected icon)
3280 case NUX_VK_ENTER:3280 case NUX_VK_ENTER:
3281 case NUX_KP_ENTER:
3281 {3282 {
3282 // start currently selected icon3283 // start currently selected icon
3283 it = _model->at (_current_icon_index);3284 it = _model->at (_current_icon_index);
32843285
=== modified file 'src/QuicklistView.cpp'
--- src/QuicklistView.cpp 2011-04-08 10:00:22 +0000
+++ src/QuicklistView.cpp 2011-04-13 19:03:28 +0000
@@ -224,6 +224,7 @@
224 // <SPACE>, <RETURN> (activate selected menu-item) 224 // <SPACE>, <RETURN> (activate selected menu-item)
225 case NUX_VK_SPACE:225 case NUX_VK_SPACE:
226 case NUX_VK_ENTER:226 case NUX_VK_ENTER:
227 case NUX_KP_ENTER:
227 if (_current_item_index >= 0 && _current_item_index < GetNumItems () &&228 if (_current_item_index >= 0 && _current_item_index < GetNumItems () &&
228 GetNthItems (_current_item_index)->GetEnabled ())229 GetNthItems (_current_item_index)->GetEnabled ())
229 {230 {