Unity

Merge lp:~didrocks/unity/altf10 into lp:unity

  1. altf10
  2. Merge into trunk
Proposed by Didier Roche-Tolomelli
Status: Merged
Approved by: Gord Allott
Approved revision: no longer in the source branch.
Merged at revision: 2012
Proposed branch: lp:~didrocks/unity/altf10
Merge into: lp:unity
Diff against target: 12 lines (+1/-1)
1 file modified
plugins/unityshell/unityshell.xml.in (+1/-1)
To merge this branch: bzr merge lp:~didrocks/unity/altf10
Reviewer Review Type Date Requested Status
Tim Penhey (community) Approve
Review via email: mp+94330@code.launchpad.net

Commit message

Change F10 to open the first menu of top panel to Alt + F10 as per design (LP: #878492)

Description of the change

Change F10 to open the first menu of top panel to Alt + F10 as per design (LP: #878492)

Just default value change, no test needed. Didn't see any AP test triggering it and depending on F10 key.

To post a comment you must log in.
Revision history for this message
Unity Merger (unity-merger) wrote :

The Jenkins job https://jenkins.qa.ubuntu.com/job/automerge-unity/334/console reported an error when processing this lp:~didrocks/unity/altf10 branch.
Not merging it.

Revision history for this message
Tim Penhey (thumper) :
review: Approve
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

We need to update this also in UnityPanelService. The problem with that is that it uses an hardcoded value (now set to F10). It would be better to make the service to update it dynamically.

Anyway I'll fix the service part on my branch.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/unityshell/unityshell.xml.in'
2--- plugins/unityshell/unityshell.xml.in 2012-02-18 12:24:02 +0000
3+++ plugins/unityshell/unityshell.xml.in 2012-02-23 08:37:19 +0000
4@@ -94,7 +94,7 @@
5 <option name="panel_first_menu" type="key">
6 <_short>Key to open the first panel menu</_short>
7 <_long>Open the first menu on the panel, allowing keyboard navigation thereafter.</_long>
8- <default>F10</default>
9+ <default>&lt;Alt&gt;F10</default>
10 </option>
11 <option name="launcher_switcher_forward" type="key">
12 <_short>Key to start the launcher application switcher</_short>