Merge lp:~dyams/unity-2d/Alt-F10-Menu into lp:unity-2d

Proposed by Lohith D Shivamurthy
Status: Merged
Approved by: Gerry Boland
Approved revision: 869
Merged at revision: 871
Proposed branch: lp:~dyams/unity-2d/Alt-F10-Menu
Merge into: lp:unity-2d
Diff against target: 47 lines (+10/-4)
3 files modified
panel/app/panelmanager.cpp (+3/-3)
panel/app/panelmanager.h (+1/-1)
tests/manual-tests/panel.txt (+6/-0)
To merge this branch: bzr merge lp:~dyams/unity-2d/Alt-F10-Menu
Reviewer Review Type Date Requested Status
Gerry Boland Pending
Review via email: mp+90268@code.launchpad.net

Description of the change

[panel] Shortcut key F10 is changed to Alt-F10 for menus

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'panel/app/panelmanager.cpp'
2--- panel/app/panelmanager.cpp 2011-11-24 08:45:18 +0000
3+++ panel/app/panelmanager.cpp 2012-01-26 14:38:24 +0000
4@@ -127,8 +127,8 @@
5
6 /* A F10 keypress opens the first menu of the visible application or of the first
7 indicator on the panel */
8- Hotkey* F10 = HotkeyMonitor::instance().getHotkeyFor(Qt::Key_F10, Qt::NoModifier);
9- connect(F10, SIGNAL(released()), SLOT(onF10Pressed()));
10+ Hotkey* F10 = HotkeyMonitor::instance().getHotkeyFor(Qt::Key_F10, Qt::AltModifier);
11+ connect(F10, SIGNAL(released()), SLOT(onAltF10Pressed()));
12 }
13
14 PanelManager::~PanelManager()
15@@ -223,7 +223,7 @@
16 }
17 }
18
19-void PanelManager::onF10Pressed()
20+void PanelManager::onAltF10Pressed()
21 {
22 QDesktopWidget* desktop = QApplication::desktop();
23 int screen = desktop->screenNumber(QCursor::pos());
24
25=== modified file 'panel/app/panelmanager.h'
26--- panel/app/panelmanager.h 2011-11-18 10:15:49 +0000
27+++ panel/app/panelmanager.h 2012-01-26 14:38:24 +0000
28@@ -45,7 +45,7 @@
29
30 private Q_SLOTS:
31 void onScreenCountChanged(int newCount);
32- void onF10Pressed();
33+ void onAltF10Pressed();
34 };
35
36 #endif // PanelManager_H
37
38=== modified file 'tests/manual-tests/panel.txt'
39--- tests/manual-tests/panel.txt 2011-12-07 15:01:18 +0000
40+++ tests/manual-tests/panel.txt 2012-01-26 14:38:24 +0000
41@@ -0,0 +1,6 @@
42+----
43+ * Hit Alt-F10.
44+
45+--> Panel main menu should be activated. The shortcut key F10 is changed to Alt-F10 now (lp:878492)
46+
47+----

Subscribers

People subscribed via source and target branches