Merge lp:~unity-team/unity/trusty-1302885 into lp:unity/7.2

Proposed by Stephen M. Webb
Status: Rejected
Rejected by: Stephen M. Webb
Proposed branch: lp:~unity-team/unity/trusty-1302885
Merge into: lp:unity/7.2
Diff against target: 44 lines (+19/-0)
2 files modified
debian/changelog (+10/-0)
unity-shared/GnomeKeyGrabber.cpp (+9/-0)
To merge this branch: bzr merge lp:~unity-team/unity/trusty-1302885
Reviewer Review Type Date Requested Status
Unity Team Pending
Review via email: mp+244068@code.launchpad.net

Commit message

When grabbing keys, try prefixing "XF86" if the key isn't found. GNOME gives us unprefixed keys sometimes.

Description of the change

When grabbing keys, try prefixing "XF86" if the key isn't found. GNOME gives us unprefixed keys sometimes. Fixes #1302885.

This change is cherry-picked from trunk for SRUing into Ubuntu 14.04 LTS.

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 'debian/changelog'
2--- debian/changelog 2014-08-26 13:49:00 +0000
3+++ debian/changelog 2014-12-11 14:55:00 +0000
4@@ -1,3 +1,10 @@
5+unity (7.2.3+14.04.20140826-0ubuntu2) UNRELEASED; urgency=medium
6+
7+ * When grabbing keys, try prefixing "XF86" if the key isn't found. GNOME
8+ gives us unprefixed keys sometimes (lp: #1302885).
9+
10+ -- Iain Lane <iain.lane@canonical.com> Mon, 08 Dec 2014 18:03:46 -0500
11+
12 unity (7.2.3+14.04.20140826-0ubuntu1) trusty; urgency=medium
13
14 [ Andrea Azzarone ]
15@@ -2189,6 +2196,9 @@
16 unity (7.0.0daily13.05.08ubuntu.unity.next-0ubuntu1) raring; urgency=low
17
18 * Automatic snapshot from revision 3317 (ubuntu-unity/next)
19+
20+ -- Ubuntu daily release <ps-jenkins@lists.canonical.com> Fri, 31 May 2013 04:06:11 +0000
21+
22 unity (7.0.0daily13.05.01.1ubuntu.unity.next-0ubuntu1) raring; urgency=low
23
24 [ Sebastien Bacher ]
25
26=== modified file 'unity-shared/GnomeKeyGrabber.cpp'
27--- unity-shared/GnomeKeyGrabber.cpp 2014-02-28 18:52:30 +0000
28+++ unity-shared/GnomeKeyGrabber.cpp 2014-12-11 14:55:00 +0000
29@@ -200,6 +200,15 @@
30 CompAction action;
31 action.keyFromString(accelerator);
32
33+ if (action.key().toString().empty())
34+ {
35+ CompString prefixed = "XF86" + CompString(accelerator);
36+ LOG_DEBUG(logger) << "Can't grab \"" << accelerator << "\", trying \"" << prefixed << "\"";
37+ action.keyFromString(prefixed);
38+ }
39+ else
40+ LOG_DEBUG(logger) << "grabAccelerator \"" << accelerator << "\"";
41+
42 if (!isActionPostponed(action))
43 {
44 action.setState(CompAction::StateInitKey);

Subscribers

People subscribed via source and target branches

to all changes:
to status/vote changes: