Keyboard layout change on hotkeys press instead of release and do not work well with shortcuts (reopen)
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
X.Org X server |
New
|
Unknown
|
|||
xorg (Debian) |
New
|
Unknown
|
|||
xorg-server (Ubuntu) |
Confirmed
|
Wishlist
|
Unassigned |
Bug Description
From duplicate bug 1720364 report.
Steps to reproduce:
1. Install ubuntu-mate-desktop on Ubuntu 16.04 LTS with HWE (Xorg 1.19.5), or 17.10 or 18.04 LTS.
2. Set-up two keyboard layouts - English and Russian
3. Set <Ctrl+Shift> as keyboard layout switcher in mate-control-center
4. Try to use shortcuts starting from <Ctrl+Shift>:
4.1. Open Firefox, open new tab, go to some site in it, close tab, try to click <Ctrl+Shift+T> to restore closed tab.
4.2. Open mate-terminal, try to open new tab with <Ctrl+Shift+T>, or copy (<Ctrl+Shift+C>), or paste (<Ctrl+Shift+V>).
4.3. Open pluma, write some text, try to navigate in it with <Ctrl+Shift+arrow>.
Expected results:
<Ctrl+Shift> switches keyboard layout, shortcuts starting from <Ctrl+Shift+...> work normally.
Actual results:
<Ctrl+Shift> switches keyboard layout, shortcuts starting from <Ctrl+Shift+...> do not work.
Notes:
1. Ubuntu 16.04 LTS (Xorg 1.18.4) with Marco and Compton work normally with <Ctrl+Shift> keyboard layout switcher.
2. This problem was discovered before on 13.10, 14.04 and other modern versions with GNOME desktop (Metacity and Compiz) - see bug 1245473.
-------
Original description below:
This is a reopen of bug #36812, which has reappeared in Kubuntu 17.04 Zesty Zapus.
This is a bug about shortcuts mapped to combinations which include each other.
For example, if we have Ctrl+Shift (for keyboard layout) and Ctrl+Shift+N (to open a new terminal), then we are practically unable to use the second shortcut; this is what happens:
Ctrl press (nothing happens)
Shift press (keyboard layout change)
N (a simple N appears, since a shortcut has already fired)
The expected behavior is to fire shortcuts on the release (not on press) of the special keys (ctrl,shift,alt, etc) which is also how Windows behave. This is a serious problem for bilingual layouts, typically using Alt+Shift or Ctrl+Shift for keyboard layout change.
Apparently, the bug has reappeared after the patch that used to fix the problem (208_switch_
ProblemType: Bug
DistroRelease: Ubuntu 17.04
Package: xserver-xorg-core 2:1.19.3-1ubuntu1
Uname: Linux 4.9.0-22.
NonfreeKernelMo
ApportVersion: 2.20.4-0ubuntu4
Architecture: amd64
CurrentDesktop: KDE
Date: Mon Apr 17 17:46:35 2017
InstallationDate: Installed on 2015-05-01 (716 days ago)
InstallationMedia: Kubuntu 15.04 "Vivid Vervet" - Release amd64 (20150422)
SourcePackage: xorg-server
UpgradeStatus: Upgraded to zesty on 2017-04-16 (1 days ago)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#47 |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#48 |
*** Bug 731 has been marked as a duplicate of this bug. ***
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#49 |
Are you able to reproduce this issue with a current version of xorg?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#50 |
yep, it's still a current issue. reasonably non-trivial fix, though. we could
probably hijack PKE for this.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#51 |
(In reply to comment #2)
> Are you able to reproduce this issue with a current version of xorg?
playing with current xorg 7.1 on SuSE 10.1, taken from SuSE repositories.
it still behaves the same.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#52 |
Sorry about the phenomenal bug spam, guys. Adding xorg-team@ to the QA contact so bugs don't get lost in future.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#53 |
*** Bug 10662 has been marked as a duplicate of this bug. ***
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#54 |
Reproduced on Ubuntu 7.04.
I would be happy to beta test patches were group change occurs at release, and
only if no other key combination was pressed, i.e:
in my configuration alt+shift chages group.
alt+shift+tab should go back a window and NOT change a group.
currently it changes a group and do goes a window forward!
Thanx,
nonZero
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#55 |
I have the same issue with Hebrew layout...
<Alt><Shift><Tab> changes language on <Alt><Shift> key-down and not go back window as expected.
Well... long time until I thought I should file a bug... :)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#56 |
*** Bug 12085 has been marked as a duplicate of this bug. ***
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#57 |
I use
Option "XkbOptions" "ctrl:nocaps"
and have related problems. When I want to type a keybinding like C-M-a (C = Control, M = Meta (Alt)) and type it by first pressing Control, then Alt and then "a" it's translated to C-a. The Alt is blocked when I hit Control first. xev shows this order of events:
KP-C, KP-a, KR-a, KP-M, KR-M, KR-C
where KP stands for KeyPress and KR for KeyRelease. If I comment out the Option line from above in my xorg.conf, all works fine.
If I type C-M-a by first pressing Alt, then Control and then "a" it works, too.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#58 |
This bug will touch all EX-WINDOWS users that use more then one language in their system. It was the major drawback when i found it.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#59 |
Hello Xorg developers,
Can you please look at this issue?
It is very important to fix this for regular users to be able to use this environment.
It basically effects all multi-language users, and regular users are not able to accept the keydown/keyup explanations...
Thanks!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#60 |
*** Bug 16041 has been marked as a duplicate of this bug. ***
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#61 |
Hmm, I am not that sure my report is a duplicate -- but if yes, differences:
* I don't do any layout switching, I have only single layout
* issue I described has nothing to do with pressing or releasing
* the problem is that you can assign shift key to keyboard shortcuts like shift+F1 but you cannot assign alt key (shift level3) to such combinations -- in other words, such key can generate only characters (specified in layout) but beside that it is a dead key
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#62 |
On Wed, May 21, 2008 at 01:43:08AM -0700, <email address hidden> wrote:
> Hmm, I am not that sure my report is a duplicate -- but if yes, differences:
>
> * I don't do any layout switching, I have only single layout
>
> * issue I described has nothing to do with pressing or releasing
>
> * the problem is that you can assign shift key to keyboard shortcuts like
> shift+F1 but you cannot assign alt key (shift level3) to such combinations --
> in other words, such key can generate only characters (specified in layout) but
> beside that it is a dead key
Yes, Fn keys acting differently is another bug, but one that's fixable
by changing the keyboard layout definition of Ctrl+Alt.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#63 |
Daniel, thank you for the answer. Further clarification (just in case):
a) F1 was just an example -- alt+left, alt+enter, alt+<anything not defined as character> is not working.
b) I don't use ctrl+alt, I use only alt (for getting special characters, in fact since alt is a key I cannot use anymore (*) I cannot use alt+ctrl anywhere by definition)
c) alt also is an example, in general it is shift level 3 key, it could be meta key as well
(*) sorry for using term "dead key", it has special meaning in X (other than I meant)
In short my wish is:
alt+c --> ć (character)
alt+f --> alt+f (key combo; so I could use in KDE as a keyboard shortcut)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#64 |
Hello,
Is there any news for this one?
It is very annoying issue.
Thanks!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#65 |
*** Bug 18333 has been marked as a duplicate of this bug. ***
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#66 |
this is really annoying bug to me. I can't use Eclipse IDE for programming, because all hotkeys use either ctrl+shift+
I cant switch to different (like caps lock) swith mode, since I use both linux and windows too often, and it is very inconvenient to use different methods.
Is there a way I can help coding it? any narration from where I should start in sources?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#67 |
xserver/
documentation aside from the XKB protocol spec, so you need to dig through the
code to figure out what actually happens.
The entry point for xkb is ProcessKeyboard
called when a keyboard event occurs. Anthing before doesn't matter to you.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#68 |
Does someone work on this? It's a very annoying bug, a lot of Emacs shortcuts don't work because of it.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#69 |
> --- Comment #21 from Alexander Kojevnikov <email address hidden> 2009-02-23 16:33:22 PST ---
> Does someone work on this?
Not that I know of. Any help would be much appreciated.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#70 |
Can we solve this problem?
I've made some digg about key codes, may be this can be helpful?
https:/
Does anybody know: Russian distros like AltLinux or AspLinux have this poblem? Perhaps we can take some code from there?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#71 |
--- Introduction
I'd like to discuss this problem in more detail and hopefully add some ideas. For the sake of concreteness, I'm mostly discussing the "Alt+Shift bound to layout switch and preventing other Alt+Shift keyboard shortcuts" case.
After some research, I have the feeling that the problem is in the XKB protocol itself and not in its implementation, which just correctly adheres to a deficient specification. It is the XKB protocol that needs to be extended to properly cover the requested behaviour. Seems to me that currently one can bind actions only to keypresses (please correct me if I'm wrong).
Admittedly, the fix is non-trivial. But it would be worth it; the current behaviour is clumsy and limiting and (as has been said before) e.g. disallows the use of "Alt+Shift+Tab" shortcut, commonly used to switch among open windows, when "Alt+Shift" is used to switch between layouts (that's actually how I found this bug - searching why Alt+Shift+Tab doesn't work).
--- Definition of "multi-
Definition: A "multi-
--- The desired behaviour - the simple case with two layouts
Initial state: no key is pressed on the keyboard, two layouts are set in the X server (us,cz), layout "us" selected (locked)
The user's intention: to select (lock) the "cz" layout
Action taken - a four step MKPR sequence:
1) press (and hold) Alt 2) press Shift 3) release Shift 4) release Alt
The expected behaviour/response: The release of all keys (i.e. step 4) would trigger the selection (and locking) of "cz" layout.
Current behaviour/response: The layout gets changed after step 2). Moreover, XKB then somehow forgets that Shift is still held down and acts as if only Alt was held down. This precludes even the reachability of any shortcut containing "Alt+Shift" combination.
Note on expected behaviour/response: If any key other than Alt or Shift is pressed during the time when Alt was held down, no layout change should occur. In other words, pressing any other key than "Shift" or "Alt" would completely disqualify the MKPR sequence being formed from being a candidate for layout switching.
--- Slight extension
Consider this MKPR seq.:
1) press and hold Alt 2) press Shift 3) release Alt 4) release Shift
That should switch the layout too, as well as this other two possibilities:
1) press and hold Shift 2) press Alt 3) release Alt 4) release Shift
1) press and hold Shift 2) press Alt 3) release Shift 4) release Alt
Unsure: What about the following MKPR seq?
1) press and hold Alt 2) press Shift 3) release Alt 4) press Alt 5) release Shift 6) release Alt
Should this do something special? (probably not, maybe it should not do anything at all)
--- What if there are more than two layouts (e.g. 4 layouts?)
It would be IMHO practical, if the switching among layouts with Alt+Shift could work similarly as Alt+Tab switching between windows in XFCE/GNOME/KDE etc., i.e. there would be some internal recency ...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#72 |
another good article about XKB
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#73 |
(In reply to comment #24)
> Action taken - a four step MKPR sequence:
> 1) press (and hold) Alt 2) press Shift 3) release Shift 4) release Alt
.
.
.
> It would be IMHO practical, if the switching among layouts with Alt+Shift could
> work similarly as Alt+Tab switching between windows in XFCE/GNOME/KDE etc.,
> i.e. there would be some internal recency list, enabling to quickly switch
> between two most recently used layouts. I.e. press and hold Alt and then
To make this happen, the event should be triggered following step 3.
So, basically what needed to be done is whenever a key is pressed, key combination should be matched against the current pressed keys, and if a combination is met exactly, we should only set a flag for this combination - and not actually fire the event.
Whenever a key is released, this flag is checked - if the flag was set, the event shoud be fired and the flag should be cleared.
Whenever a key is pressed, the flag should also be cleared, so:
Example A:
Step Action Flag
---- -------
1 Alt pressed 0
2 Shift Pressed - match! 1
3 Shift released - EVENT FIRED 0
4 Alt released 0
Example B:
Step Action Flag
---- -------
1 Alt pressed 0
2 Shift Pressed - match! 1
3 Tab Pressed 0
4 Tab released 0
5 Shift released 0
6 Alt released 0
Example C:
Step Action Flag
---- -------
1 Alt pressed 0
2 Shift Pressed - match! 1
3 Shift released - EVENT FIRED 0
4 Shift Pressed - match! 1
5 Shift released - EVENT FIRED 0
6 Alt released 0
Example D: (very strange case, unexpected, just for "fun")
Step Action Flag
---- -------
1 Alt pressed 0
2 Shift Pressed - match! 1
3 Tab Pressed 0
4 Tab released 0
5 Shift released 0
4 Shift Pressed - match! 1
5 Shift released - EVENT FIRED 0
6 Alt released 0
I think this is good enough for 99% of the cases.
I did not dive into the protocol, but I have a feeling that adding such a flag is not a big deal, and does not require a rewrite, but please, correct me if I am wrong.
Udi
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#74 |
> To make this happen, the event should be triggered following step 3.
>.
>.
> So, basically what needed to be done is whenever a key is pressed, key
> combination should be matched against the current pressed keys, and if a
> combination is met exactly, we should only set a flag for this combination -
> and not actually fire the event.
> Whenever a key is released, this flag is checked - if the flag was set, the
> event shoud be fired and the flag should be cleared.
>.
>.
> I think this is good enough for 99% of the cases.
Good idea with the flag, and you are probably right that your solution would solve 99% of the cases (since lot of people affected by this probably use just two layouts - US and their national one - at least that is my situation).
Anyway I would be definitely for incorporating the "recency-awareness" into the switching of keyboard layouts. It would be very handy in the case when there is need to switch among more than two layouts.
I know I'm straying quite a bit from the original problem now, since this is definitely a request for extension, but I think it's still in close relation, so let me explain the idea in more detail.
--- First introduce some shorthands
<key> down = press <key> and hold it down
<key> up = release <key>
<key> hit = press <key> and immediately release it
--- What do I mean by "recency-aware" switching and why is it a good idea
Basically it's the type of switching that is used for switching among windows in XFCE/GNOME/KDE, so let's first describe how that works. For example, let's open four applications in say XFCE - first app1, then app2, then app3 and finally app4. By that, subsequently app1 app2 app3 and app4 gets focused (i presume that when the app is opened, it gathers focus). So now app4 is focused. Suppose we want to switch to app2. What one can do is:
1) Alt down
2) Tab down - now a list of applications appears, showing the applications in this order: app4, app3(selected), app2, app1 - it's the order in which they had been lastly focused
3) Tab up
4) Tab down - now app2 gets selected in the applist
5) Tab up
6) Alt up - this release triggers the actual switch, so applist disappears and app2 gets focused; also, the applist gets reordered (internally) - the apps in it are now in this sequence: app2 (currently=most recently focused), app4, app3, app1 (focused longest time ago)
Now suppose we want to switch back from app2 to app4 (typical scenario: app2 is an editor in which one edits a webpage and app4 is a web browser used for testing that page). With recency-aware applist, this can be done easily, thanks to the clever rearrangement of apps in it:
1) Alt down
2) Tab down - list of application appears: app2, app4(selected), app3, app1
3) Tab up
4) Alt up - app4 gets focused
If the list didn't get rearranged (and was still app4,app3,
Compare this to e.g. tab switching in Firefox which, by default, is just rudimentary; no recency awareness there. Ctrl+Tab is just "switch one tab to the right", Ctrl+Shift+Tab "switch one...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#75 |
(In reply to comment #27)
> So this is my proposal for keyboard layout switch. It requires changes in the
> XKB and also in the desktop environments (to display the layout selection
> window), but the result would be well worth the effort IMO.
+1 to Adam's idea. This is how it should work.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#76 |
Five years the problem remains not solved.
Developers have no problems with switching of keyboard layouts and hotkeys?
(Sorry for my english)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#77 |
(In reply to comment #29)
> Five years the problem remains not solved.
>
> Developers have no problems with switching of keyboard layouts and hotkeys?
>
> (Sorry for my english)
>
Looks like Russian and Slavic users are mainly affected. :(
What about Adam's idea? It looks very good.
Can anybody realize it? Unfortunately I'm not an Xorg developer :(
Is it possible to make it as a patch? or specific branch?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#78 |
www.xneur.ru - switch layouts on key release!
No problem for hotkeys!
xneur - daemon
gxneur - gui for gnome
kxneur - gui on QT for KDE
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#79 |
Oleg, this blatant self-promotion doesn't make sense here, since the app can't solve the issue. I wasted my time installing it, and there is simply no way to set "Ctrl-Shift" as a combination to switch layouts in it.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#80 |
(In reply to comment #32)
> Oleg, this blatant self-promotion doesn't make sense here, since the app can't
> solve the issue. I wasted my time installing it, and there is simply no way to
> set "Ctrl-Shift" as a combination to switch layouts in it.
>
It's not my programm! And no self-promotion!
1. Delete all layoute switch settings on gnome
2. Set at xneur ctrl+shift for Rotate Layout
Result:
ctrl+shift = layout swither
ctrl+shift+... = you hotkey.
It works for me, must can work for you. try.
(sorry for my english)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#81 |
hi Oleg,
sorry for my wrong comment, if that's not your program. But
1) there is no "RotateLayout" option there. at least in 0.9.4
2) even if there was, people can't set ctrl-shift regardless of side as binding; you need to choose, left ctrl-shift or right ctrl-shift.
so, unfortunately, it doesn't seem to be a nice solution.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#82 |
(In reply to comment #34)
> hi Oleg,
>
> sorry for my wrong comment, if that's not your program. But
> 1) there is no "RotateLayout" option there. at least in 0.9.4
Oh, sorry. "RotateLayouts" The option has appeared in version 0.9.5 (It is accessible in svn, but non-stable)
> 2) even if there was, people can't set ctrl-shift regardless of side as
> binding; you need to choose, left ctrl-shift or right ctrl-shift.
>
The author now works over it
> so, unfortunately, it doesn't seem to be a nice solution.
Other variants simply are not present.
The problem is not solved five years.
I just try to help.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#83 |
Oleg: I tried xneur and it obviously solves some problem for someone, but not this problem for me. For once it doesn't play nice with X.org (if I understand correctly it takes over keyboard input completely and handles it on its own), for second it only supports a small subset of languages supported in X.org and for third its way too complicated.
Adam Purkrt discussion about the changes required, and nonZero's idea for implementation look like a very good thing to implement. I understand from Adam Purkrt's comments that this suggestion conflicts with the specification of the XKB protocol - Is that correct? and if so - can it be implemented regardless in X.org or does the X.org development process calls for the specification to first be changed?
Also regarding the recency list idea described in Adam Purkrt's last comment - I think that the idea in itself is good but the implementation, especially the part where the system should switch from responding to RELEASE event to use PRESS events instead, is way complicated and we should start by getting the "switch on release" working first so that layout switching can actually be done without interfering with shortcuts, and do optimization later.
I'm not an X.org developer, but I'm a programmer by trade and if this can be fixed in X.org (see my previous questions), I would really like to try to put in a fix according to Adam's and nonZero's suggestions. I just wouldn't like to invest a lot of work in this only to find its all for naught, so I'd appreciate some feedback from the developers first (Daniel?) if this is even feasible to do in the X.org code base (i.e. without getting into a large protocol specification process).
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#84 |
(In reply to comment #36)
> Also regarding the recency list idea described in Adam Purkrt's last comment -
> I think that the idea in itself is good but the implementation, especially the
> part where the system should switch from responding to RELEASE event to use
> PRESS events instead, is way complicated and we should start by getting the
> "switch on release" working first so that layout switching can actually be done
> without interfering with shortcuts, and do optimization later.
It's not so easy to handle release event instead of press, Xorg handles ctrl+shift combination as _one_ key, so AFAIR release means _both_ ctrl and shift released.
> I'm not an X.org developer, but I'm a programmer by trade and if this can be
> fixed in X.org (see my previous questions), I would really like to try to put
> in a fix according to Adam's and nonZero's suggestions. I just wouldn't like to
> invest a lot of work in this only to find its all for naught, so I'd appreciate
> some feedback from the developers first (Daniel?) if this is even feasible to
> do in the X.org code base (i.e. without getting into a large protocol
> specification process).
Daniel is now working on XKB2, so fixing/changing XKB1 has no sense :)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#85 |
(In reply to comment #37)
> It's not so easy to handle release event instead of press, Xorg handles
> ctrl+shift combination as _one_ key, so AFAIR release means _both_ ctrl and
> shift released.
I'm not sure why you say that - xev detects CTRL and SHIFT independently.
> Daniel is now working on XKB2, so fixing/changing XKB1 has no sense :)
Will XKB2 address this issue? I've looked for information on XKB2 on the web and there is very little information to be found except that it aims to handle input-methods better and that it won't bee in the next X.org release (after missing the last three as well).
So this is another area where I would appreciate some feed back from developers - if XKB2 realistic for X.org 1.9, then maybe I should help out with that. Otherwise, I don't see what's wrong with fixing XKB1 for 1.9 even if only for this release.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#86 |
(In reply to comment #38)
> I'm not sure why you say that - xev detects CTRL and SHIFT independently.
You did not notice ISO_Next_Group which appears when CTRL and SHIFT pressed at same time.
> Will XKB2 address this issue? I've looked for information on XKB2 on the web
> and there is very little information to be found except that it aims to handle
> input-methods better and that it won't bee in the next X.org release (after
> missing the last three as well).
I don't know :) I didn't find any info about XKB2 too.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#87 |
(In reply to comment #39)
> (In reply to comment #38)
> > I'm not sure why you say that - xev detects CTRL and SHIFT independently.
>
> You did not notice ISO_Next_Group which appears when CTRL and SHIFT pressed at
> same time.
Oh, I though you meant CTRL+SHIFT as in the actual keys, not as "the keyboard layout switching combination you currently have set" (which is not CTRL+SHIFT in my system).
Indeed it does - but that is internal to the XKB implementation and the reason for this bug and not something inherent to X.org itself (other then the XKB part). And we want to change that.
Also, notice that the current implementation looks like this:
1. hold down <1st key of combination>: you get "KeyPress <1st key>"
2. hold down <2nd key of combination>: you get "KeyPress ISO_Prev_Group" (or ISO_Next_Group. I get "prev", not sure why - maybe its a GNOME thing).
3. release <2nd key of combination>: you get "KeyRelease ISO_Prev_Group".
4. release <1st key of combination>: you get "KeyRelease <1st key>"
On the other hand, notice how the current implementation handles a different release order (take it from after step 2 above):
3. release <1st key of combination>: you get "KeyRelease <1st key>"
4. release <2nd key of combination>: you get "KeyRelease <2nd key>"
So in effect, with the alternate sequence, you get a <1st key> press and release, but only ISO_sth_Group press and only <2nd key> release. As I was trained to change layouts quickly, I cannot always guarantee that I release the keys in the order they were pressed, and releasing them out of order causes some problems like the notorious "sticky keys" issue.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#88 |
First of all I have the same problem (while switching to greek layout).
A simple solution I see on this is to act only on release events and only if the previous event was a press. Like this all the situations in comment: #26 are handled correctly.
The things needed from the implementation point of view are:
- A variable (e.g. previous_
- A way to find what combination of keys has just been released (I guess this is already there, for the "has just been pressed" case).
I tried to take a look in the xkbActions.c file (xorg-server-
Any info on this? I could try to create a patch for this but I need the correct pointers on where to look.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#89 |
(In reply to comment #41)
> I tried to take a look in the xkbActions.c file (xorg-server-
> could not find the ProcessKeyboard
> Any info on this? I could try to create a patch for this but I need the correct
> pointers on where to look.
It seems it's in xkbPrKeyEv.c now
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#90 |
> you get "KeyPress ISO_Prev_Group" (or ISO_Next_Group. I get "prev", not sure why
The xkb rules send ISO_Next_Group if you press the relevent keys in one
order and ISO_Prev_Group if you press them in the other order.
Eg, given group(ctrls_
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#91 |
(In reply to comment #43)
> > you get "KeyPress ISO_Prev_Group" (or ISO_Next_Group. I get "prev", not sure why
>
> The xkb rules send ISO_Next_Group if you press the relevent keys in one
> order and ISO_Prev_Group if you press them in the other order.
>
> Eg, given group(ctrls_
>
More likely its something to do with left/right: I have ALT+SHIFT set as the group toggle, and I get ISO_Prev_Group regardless of the order I hold down ALT and SHIFT on the left side of the keyboard, but ISO_Next_Group regardless of the order that I hold down ALT and SHIFT on the right side of the keyboard.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#92 |
Created attachment 33142
This patch is to resolve the issue; it is for "trunk" Xorg sources.
Please try to patch Xorg from git://anongit.
and comment if the patch works for you.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#93 |
Hello Ilya,
I have just used your patch towards the current Ubuntu karmic version of xorg and it works! The only "issue" was just some offsets in the file:
Applying patch 200-fix-
patching file xkb/xkbActions.c
Hunk #1 succeeded at 385 (offset 60 lines).
Hunk #2 succeeded at 1249 (offset 86 lines).
Is it ok for you if I forward the patch also to the ubuntu bug here:
https:/
until it is officially included in an xserver-xorg release?
Many many thanks!
Nick
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#94 |
Just a note, karmic's version is 1.6.4
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#95 |
(In reply to comment #46)
> Hello Ilya,
>
> Is it ok for you if I forward the patch also to the ubuntu bug here:
> https:/
> until it is officially included in an xserver-xorg release?
>
> Many many thanks!
> Nick
>
I forgot to note that "switch on release" is hard coded and cant be turned off.
So I am not sure it is good for wide using now. I think there will be the full solution and then ...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#96 |
> I forgot to note that "switch on release" is hard coded and cant be turned off.
> So I am not sure it is good for wide using now. I think there will be the full
> solution and then ...
You mean for the case that someone wants to keep the old behavior?
I may ask why to want this, but I guess the people here we are a bit biased :-P
I could propose it as a solution that can be adopted for the people to try.
Thanks again!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#97 |
(In reply to comment #49)
>
> You mean for the case that someone wants to keep the old behavior?
> I may ask why to want this, but I guess the people here we are a bit biased :-P
>
> I could propose it as a solution that can be adopted for the people to try.
>
> Thanks again!
>
Yes, I mean that.
You're welcome.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#98 |
I've installed this patch on Ubuntu 9.10, Ubuntu 10.04 (alpha) and Fedora 12. On all these the patch works great and I see no problems for my use - which is granted not overly complex: two keyboard layouts with ALT+SHIFT to switch and I'm using a lot of keyboard shortcuts in many applications.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#99 |
WORKING!
xorg-server-1.6.5
Thank you so much Ilya!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#100 |
just tested in kubuntu 9.1 using packages from PPA's: works just as I want it to work, so I am able to use eclipse IDE hotkeys, and that will not cause undesired layout change.
Thanks!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#101 |
Thank you!
How can I test this on my Ubuntu laptop? Is it packed in some packages or I need to compile it manually?
Can you provide an instruction, please?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#102 |
(In reply to comment #54)
> Thank you!
>
> How can I test this on my Ubuntu laptop? Is it packed in some packages or I
> need to compile it manually?
> Can you provide an instruction, please?
>
Take a look here:
https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#103 |
Excuse me in advance if I don't use xkb terminology. :)
Shouldn't the fix be more generic, and apply to all "modifier" keys (such as ctrl, shift, alt, super) and not just layout/group switching actions?
For example, I want to be able to map the Super key to an action (which is currently impossible, AFAIK).
The logic should be something like this:
on key release:
if the released key was a modifier, and all the pressed keys are modifiers:
Fire the event
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#104 |
(In reply to comment #56)
> Shouldn't the fix be more generic, and apply to all "modifier" keys (such as
> ctrl, shift, alt, super) and not just layout/group switching actions?
>
> For example, I want to be able to map the Super key to an action (which is
> currently impossible, AFAIK).
>
> The logic should be something like this:
>
> on key release:
> if the released key was a modifier, and all the pressed keys are modifiers:
> Fire the event
>
XKB deals with concrete, well known "actions", only. What event do you want to fire with Super?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#105 |
(In reply to comment #57)
> XKB deals with concrete, well known "actions", only. What event do you want to
> fire with Super?
>
I was thinking of the functionality in gnome-keybindin
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#106 |
On Sun, Feb 07, 2010 at 02:17:25PM -0800, <email address hidden> wrote:
> --- Comment #45 from Ilya Murav'jov <email address hidden> 2010-02-07 14:17:20 PST ---
> Created an attachment (id=33142)
> --> (http://
> This patch is to resolve the issue; it is for "trunk" Xorg sources.
>
> Please try to patch Xorg from git://anongit.
> and comment if the patch works for you.
The patch looks pretty much fine to me, except that I'd just hardcode
release-only and don't try to make it configurable. I'm not entirely
sure about the bit marked with KLUDGE, and would like to find a better
way to do it, but if that's fixed (I'll try to have a look this weekend
or next, depending on time), my only objection to merging it is that it
explicitly contradicts the spec. I'm not entirely convinced that's a
dealbreaker though.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#107 |
Ping? No new patch versions? Will this be applied?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#108 |
The patch works like a charm for me Ubuntu Karmic, 9.10. Daniel please do apply it. It makes a broken thing (shortcuts) work for multilingual users and doesn't harm others.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#109 |
I've found something wrong with the patch: the keyboard starts to type capital letters with no Shift and CapsLock after changing the layout (Ctrl+Shift). That behaviour cant be always reproduced. Like so:
яzЯzяZЯzzяzяZяz
Not good ...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#110 |
(In reply to comment #62)
> I've found something wrong with the patch: the keyboard starts to type capital
> letters with no Shift and CapsLock after changing the layout (Ctrl+Shift). That
> behaviour cant be always reproduced. Like so:
> яzЯzяZЯzzяzяZяz
>
> Not good ...
Have you managed to solve this? or discover a pattern, like under what circumstances this occurs?
If it doesn't happen too often, I'd be in favor of rolling out this patch, and closing this bug before it enters its 7th year.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#111 |
(In reply to comment #63)
> (In reply to comment #62)
> > I've found something wrong with the patch: the keyboard starts to type capital
> > letters with no Shift and CapsLock after changing the layout (Ctrl+Shift). That
> > behaviour cant be always reproduced. Like so:
> > яzЯzяZЯzzяzяZяz
> >
> > Not good ...
>
> Have you managed to solve this? or discover a pattern, like under what
> circumstances this occurs?
> If it doesn't happen too often, I'd be in favor of rolling out this patch, and
> closing this bug before it enters its 7th year.
No. Unfortunately, it is difficult to get rid of it:
- much worse repeatable under debug
- it is repeatable only on my working machine, not one I use for debugging Xorg (notebook)
At first everything was fine until I started playing with keyboard settings (gnome-
As for the patch, you can use Oded Arbel' repository to try it without building
Xorg yourself, http://
P.S. I am not a developer of Xorg so I could not commit anything. On the other hand, the devs seems to be not interested too. nobody cares :(
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#112 |
Ilya, SPASIBO for the patch and hint with http://
Patch works perfectly - no side effects, everything goes as expected on Ubuntu 10.04
For me keyboard shortcuts are vital, and I've been waiting for this fix since 2001 - when i first met linux. Thank you VERY much!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#113 |
Applied the patch to xorg-server-1.7.7 (Mandriva 2010.1). Works as a charm. Looking forward to having this patch upstream.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#114 |
Ilya, did you debug the problem with the led as you described in #64?
Also, Daniel (the responsible for the patches to get accepted if I understood correctly) raised some concerns on #59. Do you think you could address them? I believe after that the patch will get accepted.
What do you think?
Thanks a lot for you contribution
Nick
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#115 |
(In reply to comment #67)
> Ilya, did you debug the problem with the led as you described in #64?
>
> Also, Daniel (the responsible for the patches to get accepted if I understood
> correctly) raised some concerns on #59. Do you think you could address them? I
> believe after that the patch will get accepted.
>
> What do you think?
>
I didn't debug #64; I don't see something wrong with #59.
I think the opposite,- as I understand from a few conversations Daniel do not accept the patch because of some policy. So improvements don't make sense, and I am somewhat disappointed.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#116 |
(In reply to comment #67)
> Ilya, did you debug the problem with the led as you described in #64?
As for #64: I have used this gnome setting (ScrollLock to indicate layout) all the time since comment #65 and haven't experienced any issues. Except minor: scrollLock key is not able to control ScrollLock LED (who even cares?). I can't remember how it worked before patch, but pretty sure - that's Gnome behavior.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#117 |
I believe scrollLock LED is used from gnome(?) to indicate the alternative layout.
At least this is how it works on me with the ubuntu xorg (without the patch from ilya)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#118 |
After Vasa, comment I made some exploration of the scroll lock issue and in both cases (patched X or unpatched X) the behavior is exactly the same.
The led can be used to indicate the alternative layout (from gnome's settings) and it works as advertised: it switches on/off as long as the layout changes.
Even if the above functionality is disabled, still the Scroll Lock key makes no difference in gnome. I suspect that scroll lock mode never gets enabled.
I tested it in virtual consoles (Ctrl+Alt+F1) and it works like Ctrl+S (pause any console output), while in gnome (tested in xterm and gnome-terminal) the scroll lock key does absolutely nothing.
Daniel, is there anything else needed for the patch to get accepted in the official branch?
Thanks a lot.
Nick
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#119 |
After Vasa's comment, I made some exploration of the scroll lock issue and in both cases (patched X or unpatched X) the behavior is exactly the same.
The led can be used to indicate the alternative layout (from gnome's settings) and it works as advertised: it switches on/off as long as the layout changes.
Even if the above functionality is disabled, still the Scroll Lock key makes no difference in gnome. I suspect that scroll lock mode never gets enabled.
I tested it in virtual consoles (Ctrl+Alt+F1) and it works like Ctrl+S (pause any console output), while in gnome (tested in xterm and gnome-terminal) the scroll lock key does absolutely nothing.
Daniel, is there anything else needed for the patch to get accepted in the official branch?
Thanks a lot.
Nick
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#120 |
Just to confirm that Ilya's patch from comment 45 completely fixes the problem without any noticeable side-effects. I'm running X11R7.5 under FreeBSD 8.1
Daniel, could you review and eventually commit the patch? This bug is quite annoying.
Thank you!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#121 |
Hi,
Is there an update on the status of this bug please?
Kind Regards,
Dave Walker
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#122 |
(In reply to comment #74)
> Hi,
>
> Is there an update on the status of this bug please?
>
My guess is that this bug is stuck at the point where the patch needs extensive and explicit testing.
Some users tested this on their system, however they should to come up with a more rigorous description of what they have tested.
Things to test would include
1. Describe which shortcut you use to switch layouts (Alt+Shift, Shift+Shift, etc)
2. Mention outcome when using with Firefox, OpenOffice, Inkscape, some GTK+ and QT apps.
3. Is there any problem when using IBus? Check the shortcuts that activate/
4. how long you have been using your Linux with this patch applied.
5. GTK+ allows to type arbitrary Unicode characters with Ctrl+Shift+u <HEX codepoint>. Does this continue to work?
6. An important test case is to use Inkscape and switch layouts with Alt+Shift. Presumably, Alt+Shift+xyz is a type of valid shortcuts in Inkscape, and affected by this bug. Is this now solved?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#123 |
(In reply to comment #75)
> (In reply to comment #74)
> > Hi,
> >
> > Is there an update on the status of this bug please?
> >
>
> My guess is that this bug is stuck at the point where the patch needs extensive
> and explicit testing.
>
> Some users tested this on their system, however they should to come up with a
> more rigorous description of what they have tested.
>
> Things to test would include
> 1. Describe which shortcut you use to switch layouts (Alt+Shift, Shift+Shift,
> etc)
> 2. Mention outcome when using with Firefox, OpenOffice, Inkscape, some GTK+ and
> QT apps.
> 3. Is there any problem when using IBus? Check the shortcuts that
> activate/
> 4. how long you have been using your Linux with this patch applied.
> 5. GTK+ allows to type arbitrary Unicode characters with Ctrl+Shift+u <HEX
> codepoint>. Does this continue to work?
> 6. An important test case is to use Inkscape and switch layouts with Alt+Shift.
> Presumably, Alt+Shift+xyz is a type of valid shortcuts in Inkscape, and
> affected by this bug. Is this now solved?
I guess most people are just satisfied with this patch, which, obviously, works great. Therefore nobody really cares if it would go upstream or not, because, obviously, it won't (otherwise, tell me, how much longer it should take to accept this patch?).
Also, the "things to test would include" items look like the author hasn't even tested the patch, otherwise, he wouldn't have any of those concerns.
Thank you.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#124 |
(In reply to comment #76)
> I guess most people are just satisfied with this patch, which, obviously, works
> great.
It does indeed, just look as the number of comments confirming it.
> Also, the "things to test would include" items look like the author hasn't even
> tested the patch, otherwise, he wouldn't have any of those concerns.
I went ahead and did the tests grandfather mentioned:
(In reply to comment #75)
> 1. Describe which shortcut you use to switch layouts (Alt+Shift, Shift+Shift,
> etc)
Alt+Shift, I guess that's what most people use (it's a Windows default too)
> 2. Mention outcome when using with Firefox, OpenOffice, Inkscape, some GTK+ and
> QT apps.
No issues whatsoever. Tested it on ArchLinux and on FreeBSD.
> 3. Is there any problem when using IBus? Check the shortcuts that
> activate/
Is it for CJK input? Never used it.
> 4. how long you have been using your Linux with this patch applied.
Linux almost since this patch has been submitted, FreeBSD - for 3 months.
> 5. GTK+ allows to type arbitrary Unicode characters with Ctrl+Shift+u <HEX
> codepoint>. Does this continue to work?
Yes: ☺ ♫ ⊥
> 6. An important test case is to use Inkscape and switch layouts with Alt+Shift.
> Presumably, Alt+Shift+xyz is a type of valid shortcuts in Inkscape, and
> affected by this bug. Is this now solved?
Yes, also some Emacs shortcuts were affected by this bug (e.g. M-> and M-<) and now work fine.
I really hope this patch gets committed, the keyboard input *is* broken without it for users who use multiple keyboard layouts. I personally don't mind patching xorg-server manually, but I guess most users do.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#125 |
(In reply to comment #75)
> Things to test would include
> 3. Is there any problem when using IBus? Check the shortcuts that
> activate/
Input methods may be a problem as i dont think anyone who is interested in this is using IM. I dont know why - maybe IM users dont use layout switching shortcuts, maybe they don't use shortcuts at all? But that is
A good reason to apply this patch to X.org's trunk - until that is done we cant get any serious testing of this feature as the only people that encounter the patch are the people that come looking for it specifically and all those fall into a set of well defined parameters.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#126 |
The patch has been applied in Ubuntu, just now.
Today is a good day. :)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#127 |
The following comments were made by Peter Hutterer (an X.org input developer) on the corresponding bug in RedHat bugzilla ( https:/
(In reply to comment #6)
> implementing this behaviour requires guesswork that I'm not sure is safe in a
> number of setups.
...
> afaict, the desired behaviour for a ctrl+shift groupchange is:
> ctrl down → set Control modifier
> shift down → set Shift modifier
> if (other key pressed)
> send event Contrl+Shift+<other key>
> else if (ctrl || shift released)
> change group
>
> The XKB map for left control in this case is:
> key <LCTL> { [ Control_L, ISO_Next_Group ] };
> So whenever ISO_Next_Group is pressed, you still need to know which modifier to
> set in case the group action isn't executed. The XkbSA_SetMod, XkbSA_LockMod,
> etc. actions provide the modifiers set for a given key, hence why it works
> currently. This information comes from the client when the xkb map is loaded
> and is used to trigger the modifier flags for a given key. The XkbSA_LockGroup
> behaviour (which is triggered at ISO_Next_Group) does not have this field
> (adding it would break ABI), so you need to guess which modifiers to set if you
> didn't trigger this action. This is the main stumbling point that I found and
> if you look at Ilya's patch that's where he needs the big hack that I'm not
> comfortable at all with it.
>
> Now, I don't know if there are layouts where the modifier mask would be
> different on the second level as opposed to the first (and Ilya's hack or a
> similar attempt would fail completely) but there's so many layouts that it'll
> take a while to get through them all.
Ilya - this is hardly my area of expertise, so if you can address these issues - either by commenting here, on the RedHat bugzilla or by changing the patch - I would greatly appreciate that.
Thanks to all the people who are involved, and lets keep the communication channels open :-)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#128 |
(In reply to comment #80)
Ok, I try to answer here, but I should note that I don't remember full details
(because it was more than half a year ago).
> The following comments were made by Peter Hutterer (an X.org input developer)
> on the corresponding bug in RedHat bugzilla (
> https:/
> (In reply to comment #6)
> > implementing this behaviour requires guesswork that I'm not sure is safe in a
> > number of setups.
> ...
> > afaict, the desired behaviour for a ctrl+shift groupchange is:
> > ctrl down → set Control modifier
> > shift down → set Shift modifier
> > if (other key pressed)
> > send event Contrl+Shift+<other key>
> > else if (ctrl || shift released)
> > change group
> >
> > The XKB map for left control in this case is:
> > key <LCTL> { [ Control_L, ISO_Next_Group ] };
> > So whenever ISO_Next_Group is pressed, you still need to know which modifier to
> > set in case the group action isn't executed. The XkbSA_SetMod, XkbSA_LockMod,
> > etc. actions provide the modifiers set for a given key, hence why it works
> > currently. This information comes from the client when the xkb map is loaded
> > and is used to trigger the modifier flags for a given key. The XkbSA_LockGroup
> > behaviour (which is triggered at ISO_Next_Group) does not have this field
> > (adding it would break ABI), so you need to guess which modifiers to set if you
> > didn't trigger this action. This is the main stumbling point that I found and
> > if you look at Ilya's patch that's where he needs the big hack that I'm not
> > comfortable at all with it.
I do not agree. You do not need to know/guess which modifiers to set - whenever
ISO_Next_Group is pressed I just don't execute it immediately but delay it till a key release (by the means of _XkbNextFreeFil
And I want to note that where I comment ":KLUDGE:" I mean a different thing: in theory that branch of code should do the same thing as the switch in XkbHandleActions() ; but, in practice I see (and want) only XkbSA_SetMods action (so kludge here is copy-n-paste from XkbHandleActions() ).
> >
> > Now, I don't know if there are layouts where the modifier mask would be
> > different on the second level as opposed to the first (and Ilya's hack or a
> > similar attempt would fail completely) but there's so many layouts that it'll
> > take a while to get through them all.
(do not understand properly the above, sorry) The only situation the patch fails
(just behaves old way, and nothing more!) is when switching is set up as just one key like "Right Alt". That is because of the line
fake_state.mods = 0;
, mods here is 0 anyway => we can't block XkbSA_LockGroup .
Anyway, nobody wants more,- but only (de facto standard) Ctrl+Shift and Alt+Shift on release. I think this is the situation where the practice begins and the theory ends.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#129 |
I'm confused. This bug is now 7 years old and it's still not fixed? What year of desktop Linux are we talking about when basic things in Linux are largely broken?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#130 |
(In reply to comment #82)
> I'm confused. This bug is now 7 years old and it's still not fixed? What year
> of desktop Linux are we talking about when basic things in Linux are largely
> broken?
thanks. empty rhetoric is the greatest way of motivating developers. This bug just dropped to the bottom of my priority list again.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#131 |
(In reply to comment #83)
> thanks. empty rhetoric is the greatest way of motivating developers. This bug
> just dropped to the bottom of my priority list again.
Peter, I'm terribly sorry for rending the air. Please, consider resolving this bug ASAP since there are thousands of people affected by it. I won't drop another comment here ever.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#132 |
(In reply to comment #84)
> (In reply to comment #83)
> > thanks. empty rhetoric is the greatest way of motivating developers. This bug
> > just dropped to the bottom of my priority list again.
>
> Peter, I'm terribly sorry for rending the air. Please, consider resolving this
> bug ASAP since there are thousands of people affected by it. I won't drop
> another comment here ever.
You don't have to excuse. You have more rights to be pissed off than a developer who puts an obvious and already proved (by time) to work patch to the end of his TODO list.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#133 |
On Fri, Apr 08, 2011 at 01:05:34AM -0700, <email address hidden> wrote:
> --- Comment #85 from kyak <email address hidden> 2011-04-08 01:05:27 PDT ---
> (In reply to comment #84)
> > (In reply to comment #83)
> > > thanks. empty rhetoric is the greatest way of motivating developers. This bug
> > > just dropped to the bottom of my priority list again.
> >
> > Peter, I'm terribly sorry for rending the air. Please, consider resolving this
> > bug ASAP since there are thousands of people affected by it. I won't drop
> > another comment here ever.
>
> You don't have to excuse. You have more rights to be pissed off than a
> developer who puts an obvious and already proved (by time) to work patch to the
> end of his TODO list.
Already proven to break the XKB specification, yes.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#134 |
Can we pay to speed up the fixing of this bug?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#135 |
(In reply to comment #86)
> Already proven to break the XKB specification, yes.
Hi Daniel. I've seen this comment about breaking the XKB specification in several places and - I'm really not trying to be contrary - I looked at the protocol specs here http://
Now - I'm not a trained X11 developer, and I'm not even that good at reading specs, so I would really appreciate it if you can point me at the section relevant to the breakage you discuss, so I can be more informed about the issue.
Thanks in advance.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#136 |
(In reply to comment #87)
> Can we pay to speed up the fixing of this bug?
I second this motion.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#137 |
(In reply to comment #88)
> (In reply to comment #86)
> > Already proven to break the XKB specification, yes.
>
> Hi Daniel. I've seen this comment about breaking the XKB specification in
> several places and - I'm really not trying to be contrary - I looked at the
> protocol specs here http://
> and I can't understand how this behavior change contradicts the spec.
>
> Now - I'm not a trained X11 developer, and I'm not even that good at reading
> specs, so I would really appreciate it if you can point me at the section
> relevant to the breakage you discuss, so I can be more informed about the
> issue.
>
> Thanks in advance.
Hi Oded,
It is not in XKBproto but in XKBlib spec. You can see the only assertion against changing layout on release in general (and the patch in particular) in ftp://ftp.
"
1. If the XkbSA_GroupAbsolute is set in the flags field, key press events set the locked keyboard group to the group specified by the group_XXX field. Otherwise, key press events add the group specified by the group_XXX field to the locked keyboard group. In either case, the resulting locked and effective keyboard groups are brought back into range depending on the value of the groups_wrap field of the con-trols structure.
2. A key release has no effect.
"
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#138 |
I would vote for this change as well.
I understand that it is not aligned with 13 years old standard and I am sorry for that. Still I see changing group on press breaking much many things than changing it on release. Maybe it is time to modify this library specification, because there is a good reason for that. Do you see any reason not to do it - other than there exists 13 years old library specification?
Best regards
Michal Ambroz
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#139 |
For Gentoo and derivatives users, a working solution for patching is described here:
https:/
Bug report description includes guidance for how an ebuild file for automatic patching of XkbActions.c can be achieved, and an example ebuild file that applies Ilya's patch to Gentoo version of xorg-server-1.10.3 as attachment is also provided by Lance Poore.
I am using the patched version happily now without having any problems. Please consider including the patch in upstream. I am sure it would make thousands of desktop Linux users' lives easier.
Thanks Ilya! and thanks to Lance in the name of Gentoo Linux multilingual keyboard users.
Happy patching :)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#140 |
Long ago a bug was opened for this issue.
https:/
On Mon, Aug 22, 2011 at 5:14 PM, <email address hidden> wrote:
> https:/
>
> --- Comment #92 from youagree <email address hidden> 2011-08-22 07:14:34 PDT ---
> For Gentoo and derivatives users, a working solution for patching is described
> here:
>
> https:/
>
> Bug report description includes guidance for how an ebuild file for automatic
> patching of XkbActions.c can be achieved, and an example ebuild file that
> applies Ilya's patch to Gentoo version of xorg-server-1.10.3 as attachment is
> also provided by Lance Poore.
>
> I am using the patched version happily now without having any problems. Please
> consider including the patch in upstream. I am sure it would make thousands of
> desktop Linux users' lives easier.
>
> Thanks Ilya! and thanks to Lance in the name of Gentoo Linux multilingual
> keyboard users.
> Happy patching :)
>
> --
> Configure bugmail: https:/
> ------- You are receiving this mail because: -------
> You are on the CC list for the bug.
>
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#141 |
(In reply to comment #93)
> Long ago a bug was opened for this issue.
>
> https:/
>
And marked as "resolved upstream". Which is just not true, at least according to the ebuilds. Havent taken a look at the code itself, but can you currently point to any existing xorg source repository with the patch applied or where xorg server has the same functionality issue cured?
Thanks
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#142 |
Lots of CCs ... I'm gonna bring this into the tracker to see if there's
something we can eventually do about this in a way that won't break backwards
compatibility.
Daniel, do you have any thoughts about how this could be done by extending XKB
rather than breaking it?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#143 |
(In reply to comment #90)
> (In reply to comment #88)
> > (In reply to comment #86)
> > > Already proven to break the XKB specification, yes.
> > ...
>
> It is not in XKBproto but in XKBlib spec. You can see the only assertion
> against changing layout on release in general (and the patch in particular) in
> ftp://ftp.
> Types", the last item XkbSA_LockGroup, citing:
>
> "
> 1. If the XkbSA_GroupAbsolute is set in the flags field, key press events set
> ....
> 2. A key release has no effect.
> "
Thanks again Ilya!
First for providing the patch, and then for high-lighting exactly where the XKB standard made what we now know to be a bad choice. Happens. No one's perfect. Not even the wisdom of the Xfree86 crowd.
Anyone know whether / how the standard can be brought up to date, so it joins us in the new millennium?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#144 |
Deferring to 1.13 as this is functional change
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#145 |
The patch doesn't apply anymore to latest xorg. Could someone update it, please?
I tried to just blindly port this patch, but it seems to cause problems with Caps Lock (it can't be switched off once switched on). So something has changed.
Could someone with xorg knowledge have a look please?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#146 |
Sorry, it was my overlook. THe patch works just fine, just needed more attention adapting to latest sources.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#147 |
Created attachment 59741
The same patch, but based on 1.12.0.902
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#148 |
While this patch solve conflict between keyboard layout switching hotkey and other hotkeys, it doesn't fix this issue in general. For example, in one have hotkey defined for Win key (a.k.a. Super_L), and for Win+something, the first hotkey handler will always run when Win key DOWN, thus pressing Win+something will result in executing both Win and Win+something handlers.
It should be fixed in same way: process current key combination on first UP event after a sequence of DOWN events, not on first DOWN.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#149 |
Created attachment 63378
xorg-server-
Some code style modifications.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#150 |
This patch breaks the keyboard layout switcher from KDE3 and Trinity. It works correctly with the layout switcher in KDE4. Have other people tested this patch with other layout switchers? What are the results?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#151 |
(In reply to comment #103)
> This patch breaks the keyboard layout switcher from KDE3 and Trinity. It works
> correctly with the layout switcher in KDE4. Have other people tested this patch
> with other layout switchers? What are the results?
This works fine with all switchers shipped on Ubuntu (gnome, unity, kde4, etc.).
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#152 |
Oded Arbel, how did you test this patch on Ubuntu? Can you give me some link to PPA or deb files for testing?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#153 |
> how did you test this patch on Ubuntu?
It's already merged. You may just install latest Ubuntu to check this patch in action. Personally I doesn't have any troubles with this patch too (KDE4, Unity).
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#154 |
As in comment 101, the current behavior of X prevents a lot of interesting usage of shift keys as shortcuts.
A Windows-only compose key program uses the "ctrl" key as the compose key. This is apparently impossible to do in X input methods because you can only bind actions to the press of "ctrl". What is wanted is an action if "ctrl" is pressed and released without hitting any other keys.
I think this can be solved more easily. For any "shift" key, you can bind actions to them, but they are only triggered if you press & release the shift key. All other keys trigger bound actions when they are pressed. The keyboard switching is NOT a special case.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#155 |
I don't know what they have done in Ubuntu but 12.04 understands both press and release. Layout switching happens on release. If you press Alt you see the global menu and after release HUD uppears. Looks like everything has been fixed. But if you use Ctrl or Alt for layout switching you still can't use this key for anything more, it's "exclusived" :( So ridiculous.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#156 |
(In reply to comment #108)
> I don't know what they have done in Ubuntu but 12.04 understands both press
> and release. Layout switching happens on release.
Ubuntu have simply applied Ilya's patch (one of the revisions attached here) to fix the layout switching problem.
> If you press Alt you see
> the global menu and after release HUD uppears.
This is a different behavior and one that works with the pristine X.org server - the ALT key, when used without any other shift keys, fires "down" when pressed and "up" when released. The Ubuntu HUD listens for this sequence and triggers when ALT is used like that without any other shift key.
The problematic behavior (as documented in this lengthy bug report - kitates, please read the discussion), is that when you press down on the second shift key, X.org fires the keyboard layout change (problem 1) and also immediately fires the "up" event for the second shift key, even though the user is still holding the key down (problem 1).
Problem 1 means that when the user wants to use <shift1>
Problem 2 means that the actual keyboard shortcut will never actually trigger because when the user holds down <key>, even though all keys are physically held down X.org only acknowledges that <shift1> and <key> are held down.
> Looks like everything has
> been fixed. But if you use Ctrl or Alt for layout switching you still can't
> use this key for anything more, it's "exclusived" :( So ridiculous.
It shouldn't work like that - I've tested the Ubuntu built X.org (with Ilya's patch) and it worked properly when using CTRL+ALT as the keyboard switching. It was immediately after the patch got accepted (at 11.04) but as ALT+SHIFT still works fine, I don't see a reason everything shouldn't continue to work (though I don't have access to an Ubuntu machine ATM to test).
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#157 |
(In reply to comment #107)
> A Windows-only compose key program uses the "ctrl" key as the compose key.
> This is apparently impossible to do in X input methods because you can only
> bind actions to the press of "ctrl".
Of course this is possible. Compose is unrelated to actions in the XKB meaning of the term. It is not only possible, it is even implemented in the XIM compose code. You can put
<Control_L><a><e> : ae
and similar stuff in your .XCompose, and your left control key acts as a Compose key. But you need the latest libX11 for this.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#158 |
I'm really disappointed this change didn't make it to 1.13.
I'm sick and tired of applying this patch after each xorg update.
How many users (and years) do you need to finally accept this change?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#159 |
I believe the most serious objection with this request is that it violates the XKB specification (see the description of SA_LockGroup in section 6.3 of "The X Keyboard Extension: Protocol Specification").
In the same specification, in section 4.0 of appendix D ("Protocol Encoding"), we see in the description of SA_LockGroup that there are still 5 unused bits in the flags field. My proposal in to use one of these bits decide whether to lock groups on press or release. By default (bit is zero), lock groups on press as the protocol specification demands. If the flag is one, lock groups on release. So by default, we would conform to the specification, and add the alternative behaviour as a new possibility beyond the specification.
There are some usage implications. One must use 'Private' do create actions with the new flag set (until xkbcomp is updated as well), and one needs support in xkeyboard-config to make the new feature usable for non-XKB-hackers.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#160 |
(In reply to comment #112)
> I believe the most serious objection with this request is that it violates
> the XKB specification (see the description of SA_LockGroup in section 6.3 of
> "The X Keyboard Extension: Protocol Specification").
>
> In the same specification, in section 4.0 of appendix D ("Protocol
> Encoding"), we see in the description of SA_LockGroup that there are still 5
> unused bits in the flags field. My proposal in to use one of these bits
> decide whether to lock groups on press or release. By default (bit is
> zero), lock groups on press as the protocol specification demands. If the
> flag is one, lock groups on release. So by default, we would conform to the
> specification, and add the alternative behaviour as a new possibility beyond
> the specification.
>
> There are some usage implications. One must use 'Private' do create actions
> with the new flag set (until xkbcomp is updated as well), and one needs
> support in xkeyboard-config to make the new feature usable for
> non-XKB-hackers.
Thanks Andreas, your answer pretty much clarifies everything for me!
Your proposal is very correct, no doubt. Does it mean that once your proposal is implemented all 3rd-party keyboard switchers (like those in Gnome and KDE) would have to be updated to make use of this new possibility?
Anyway, as i see it, there are two ways to go:
1) The long way - making things right and according to specification. This would take from very long to forever (this is the way we've been going for the last 8 years with this bug report).
2) Take a short way - let the common sense win over specification and make everybody happy.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#161 |
> Your proposal is very correct, no doubt. Does it mean that once your
> proposal is implemented all 3rd-party keyboard switchers (like those in
> Gnome and KDE) would have to be updated to make use of this new possibility?
As far as I understand, KDE and Gnome all use xkeyboard-config, and just provide their own GUI. If this is understanding is correct, the changes to xkeyboard-config would be sufficient.
> Anyway, as i see it, there are two ways to go:
> 1) The long way - making things right and according to specification. This
> would take from very long to forever (this is the way we've been going for
> the last 8 years with this bug report).
Assuming the existing patch is correct, adding the additional check for the flag is just a few lines. The changes to xkeyboard-config would be fairly simple. Assuming we grab bit 3 for the new flag, in xkeyboard-
interpret ISO_Next_Group {
useModMap
virtualMo
action= LockGroup(
}
to
interpret ISO_Next_Group {
useModMap
virtualMo
action= Private(type=6, data[0]=16, data[1]=1);
}
(untested, of course), and similarly for ISO_Prev_Group, ISO_First_Group, and ISO_Last_Group. I do not know wether the action bound to keysyms is standardised; even if it is not, it might be a good idea to make the above redefinition conditional.
> 2) Take a short way - let the common sense win over specification and make
> everybody happy.
Believe it or not, I would be unhappy when the specification would be broken. Also remember that the attitudes in this discussion are not necessarily representative of all X users, as the users satisfied with the current behaviour do not have any reason to even know about this discussion.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#162 |
(In reply to comment #109)
> (In reply to comment #108)
> > Looks like everything has
> > been fixed. But if you use Ctrl or Alt for layout switching you still can't
> > use this key for anything more, it's "exclusived" :( So ridiculous.
>
> It shouldn't work like that - I've tested the Ubuntu built X.org (with
> Ilya's patch) and it worked properly when using CTRL+ALT as the keyboard
> switching. It was immediately after the patch got accepted (at 11.04) but as
> ALT+SHIFT still works fine, I don't see a reason everything shouldn't
> continue to work (though I don't have access to an Ubuntu machine ATM to
> test).
Oded Arbel, I repeat: Ctrl OR Alt. OR, not AND. I use right Alt for layout switching so I can't use it for any other purpose.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#163 |
(In reply to comment #115)
> > It shouldn't work like that - I've tested the Ubuntu built X.org (with
> > Ilya's patch) and it worked properly when using CTRL+ALT as the keyboard
> > switching. It was immediately after the patch got accepted (at 11.04) but as
> > ALT+SHIFT still works fine, I don't see a reason everything shouldn't
> > continue to work (though I don't have access to an Ubuntu machine ATM to
> > test).
>
> Oded Arbel, I repeat: Ctrl OR Alt. OR, not AND. I use right Alt for layout
> switching so I can't use it for any other purpose.
Hi,
Actually, the patch works for key shortcuts with two or more buttons (like ALT+SHIFT, but not Alt or Ctrl alone). It is not done intentionally.
Despite the fact that there is a possibility to improve current patch behaviour for needs like yours I don't think it should be improved for all possible cases -
there are many other things to be done to make the world a better place.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#164 |
Created attachment 69198
LockMods can lock another group
This patch follows a different route: It extends modifier locking, rather than changing how group lock works. Extending has the advantage that the previous behaviour is maintained, and the patch does not violate the X Keyboard Protocol Specification. Extending modifier locking rather than group locking has the advantage that we do not need the "Kludge" of the other patch, as we can pass the modifiers that we want to set, rather then relying on heuristics.
The disadvantage over the existing patch is that we must change the keymap.
Here are three examples. The left alt key is to switch to the next layout when it is pressed and released before any other key is pressed.
key <LALT> {
repeat= No,
type= "TWO_LEVEL",
actions[Group1]= [ Private(
};
Similarly, shifting group with Shift+Right Alt (where Shift is pressed first):
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#165 |
Similarly, shifting group with Shift+Right Alt (where Shift is pressed first):
key <RALT> {
repeat= No,
type= "TWO_LEVEL",
actions[Group1]= [ SetMods(
};
Shifting group with Shift+Left Control:
key <LCTL> {
repeat= No,
type= "TWO_LEVEL",
actions[Group1]= [ SetMods(
};
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#166 |
(In reply to comment #117)
> Similarly, shifting group with Shift+Right Alt (where Shift is pressed first):
Is this mean your patch won't work when Alt (or Ctrl) is pressed before Shift?
AFAIK most people press Ctrl, then Shift, then either release them (to switch layout) or press A-Z when they need Ctrl-Shift-
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#167 |
> Is this mean your patch won't work when Alt (or Ctrl) is pressed before
> Shift?
It does not mean that. I just restricted to three examples. There is no problem to rewrite all options that xkeyboard-config offers to switch groups to take advantage of the patch.
> AFAIK most people press Ctrl, then Shift, then either release them (to
> switch layout) or press A-Z when they need Ctrl-Shift-
In this case, one needs to remap the shift key. For the left shift key:
key <LFSH> {
repeat= No,
actions[Group1]= [ SetMods(
};
For the right shift key it works similarly. If combined with the third example above, it will make the order of Shift and Control irrelevant.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#168 |
Created attachment 69213
LockMods can lock another group
My previous patch does not properly account for absolute group specification. The revised patch corrects this.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#169 |
Just to add something as a reply on comment #114:
Ubuntu has applied this patch already since 06 Jan 2011 all versions till nowadays, as you can see in the changelogs here:
http://
Select "Ubuntu Changellog" and then search for "208_switch_
Even after almost two years, there has been noone to file a bug report that this change breaks anything. I believe this is quite a good indication that Ilya's patch is already safe enough.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#170 |
It is xorg-server 1.14, year 2013, and the bug still exists.
Do something with it already...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#171 |
I wonder, bearing in mind the fact that Ubuntu has the patch for the problem, will Mir be the first Linux display server which implemet this feature? :)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#172 |
(In reply to comment #124)
> I wonder, bearing in mind the fact that Ubuntu has the patch for the
> problem, will Mir be the first Linux display server which implement this
> feature? :)
You've wandered far off.
Both Mir and Wayland have a completely different input architecture - they simply don't have this problem from the very beginning.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#173 |
I've just installed OpenSuse 12.3 and met face to face with this bug. Very frustrating, because of it I can't open last closed tab in chromium with ctrl+shift+t.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#174 |
any news here?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#175 |
Igor Gnatenko, I partly solve my problem via KSuperKey app http://
so maybe it solve your needs too.
But I also waiting fix for this issue in xkb.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#176 |
As far as I can tell, it is already clear that XKB specifications needs to be extended for this to be cleanly solved. While Ilya's hack works for many people, and as much as I would like to see this fixed, I understand why it is unacceptable to the developers. So, trying to delve into XKB specs now; this page seems to be a good starting point to me:
https:/
Also (as suggested by the author), copying info here from https:/
Peter Hutterer 2011-01-05 22:07:57 EST
there is two problems with the "tiny" patch you mentioned. one, it's an _explicit_ violation of the XKB specification (see section 4.4). two, implementing this behaviour requires guesswork that I'm not sure is safe in a number of setups.
Peter Hutterer 2011-01-06 17:12:05 EST
(In reply to comment #4)
> But: doesn't fixing of a huge problem have a priority over preservation of a
> "holy spec"?
it's a matter of figuring out the side-effects. a specification is a behaviour promise, in this case in place for 15 years or so. a lot of users and apps rely on the promised behaviour (in general, not necessarily this specific issue) and breaking it is a dangerous thing because you may not know what else you break.
this is why we're hesitant to break the behaviour on purpose.
note that i'm not claiming that there is no problem, i'm just saying it's the balance between a known problem and introducing new bugs that potentially break current applications.
> > two,
> > implementing this behaviour requires guesswork that I'm not sure is safe in a
> > number of setups.
>
> What guesswork do you mean? Which setups can present problems?
> BTW, I'm 100% sure that Ilya Murav'jev (patch author) will be glad to
> cooperate.
afaict, the desired behaviour for a ctrl+shift groupchange is:
ctrl down → set Control modifier
shift down → set Shift modifier
if (other key pressed)
send event Contrl+Shift+<other key>
else if (ctrl || shift released)
change group
The XKB map for left control in this case is:
key <LCTL> { [ Control_L, ISO_Next_Group ] };
So whenever ISO_Next_Group is pressed, you still need to know which modifier to set in case the group action isn't executed. The XkbSA_SetMod, XkbSA_LockMod, etc. actions provide the modifiers set for a given key, hence why it works currently. This information comes from the client when the xkb map is loaded and is used to trigger the modifier flags for a given key. The XkbSA_LockGroup behaviour (which is triggered at ISO_Next_Group) does not have this field (adding it would break ABI), so you need to guess which modifiers to set if you didn't trigger this action. This is the main stumbling point that I found and if you look at Ilya's patch that's where he needs the big hack that I'm not comfortable at all with it.
Now, I don't know if there are layouts where the modifier mask would be different on the second level as opposed to the first (and Ilya's hack or a similar attempt would fail completely) but there's so many layouts that it'll take a while to get through them all.
> And leaving the design bug because of purist reason looks rea...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#177 |
(In reply to comment #129)
> As far as I can tell, it is already clear that XKB specifications needs to
> be extended for this to be cleanly solved. While Ilya's hack works for many
> people, and as much as I would like to see this fixed, I understand why it
> is unacceptable to the developers.
...
> there's two-ish ppl working on input at them moment, both are badly
> overloaded because there's a lot of bugs and plenty new features that ppl
> cry out for. so this bug has less to do with purist reasons, it's more along
> the lines of "i've got so many things to do that don't break the spec, they
> get priority".
So I understand that this issue is currently not worked on by Xorg developers.
Priorities are a good thing, and I'm currently comfortable with the pragmatic approach of the downstream distributions applying Ilya's patch in their packages (which works fine for a lot of happy users who have reported as such - I have yet to hear a report where Ilya's patch is not improving behavior for multi-language users).
Still:
1) Its a good idea to have the pragmatic patch set maintained in this bug report - even if it will never be applied to upstream Xorg, simply as a central point for downstream distributors to get access to a reasonable workaround until a "correct" solution is available.
2) It will be a real shame if this issue, that is actively discussed by the community for 10 years now and is hurting a lot of users, will be ignored for another 10 years. In the mean time XKB2 - that was supposed to be the solution to all our problems - was relegated from "being worked on" to "being thought on" to "being dreamed of" (around 2010) and now appears to have fallen to the status of "it will never be important enough".
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#178 |
xkb should be extended to be able to recognize a sequence of key presses and key releases and fire action upon it. Again, this would be an extension, not a violation of current standard.
Currently, the group switching is defined in /usr/share/
partial modifier_keys
xkb_symbols "lalt_lshift_
virtual_
key <LALT> {
};
key <LFSH> {
};
};
The following is what this part of the file would look like after appropiate changes in xkb code:
xkb_sequences "lalt_lshift_
sequence { keydown <LALT>, keydown <LFSH>, keyup <LFSH> } = ISO_Next_Group;
sequence { keydown <LFSH>, keydown <LALT>, keyup <LALT> } = ISO_Next_Group;
}
For this to work
1) http://
2) http://
3) http://
At least this is my impression after a day of investigation. I know this seems like unnecessarily big change compared to Ilya's patch. But, basically, that is what I mean by "clean solution". Unfortunatelly, it is more of a "dream solution", since I feel I will hardly be able to code this..
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#179 |
Allowing arbitrary sequences to decide which action to take would certainly be powerful, but quite some effort, also for specification (think about multiple matching sequences).
In comment #117 I suggested an extension to the specification that is restricted to the enhancement request at hand. But even this change would need a protocol bump. This is where this proposal is stuck.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#180 |
Found partial workaround today.
It _is_ possible to setup xkb so that the group (i.e. layout) switch occurs on <Shift>+<Alt> (which means press and hold Shift, then press Alt - order matters) - and still <Alt>+<
Here's what I did:
$ setxkbmap us,cz
(just load the layouts I want to the server, switching not working yet)
$ xkbcomp $DISPLAY xkbdesc
(list the xkb description from server to a file named "xkbdesc")
Now two little changes to xkbdesc:
change1:
key <LALT> { [ Alt_L, Meta_L ] };
to
key <LALT> { [ Alt_L, ISO_Next_Group ] };
change2:
key <RALT> {
};
to
key <RALT> {
};
then
$ xkbcomp xkbdesc $DISPLAY
(load kbdesc to the server; ignore the warnings)
and voila! Shift+Alt (in that order) switches us<->cz keyboard and Alt+Shift+Tab works like it should.
I was actually quite surprised when I found this.
PS: Next time you start X, you can do just the last step, supposed you keep the xkbdesc file.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#181 |
Perhaps easier variant of the above workaround pertaining Alt+Shift is to comment out the following (eight) lines in /usr/share/
first change (search for "lalt_lshift"):
partial modifier_keys
xkb_symbols "lalt_lshift_
virtual_
key <LALT> {
};
// key <LFSH> {
// type[Group1]
// symbols[Group1] = [ Shift_L, ISO_Next_Group ]
// };
};
second change (~15 lines below):
partial modifier_keys
xkb_symbols "ralt_rshift_
virtual_
key <RALT> {
};
// key <RTSH> {
// type[Group1]
// symbols[Group1] = [ Shift_R, ISO_Next_Group ]
// };
};
then load the keymap, in my case
$ setxkbmap us,cz -option grp:alt_
or set it through gui (Change layout option=Alt+Shift in XFCE), or just restart X server if you've already done so
Now <shift>+<left alt> (*in that order*, i.e. first press and hold any shift, then press left alt) switches keyboard layout
<alt>+<shift> (first alt, then shift) works as a modifier (i.e. shortcuts work), and does nothing in itself
<alt>+<shift>+<tab> works as it should
It would be nice to actually have a separate "rule" for this - titled "Shift+LAlt", available with -grp:shift_
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#182 |
Currently, Alon Bar-Lev's patch (xorg-server-
xkb_keymap {
xkb_types { include "complete" };
xkb_compat { include "complete+
xkb_symbols { include "pc+us+
};
Holding ALT+SHIFT (without releasing) immediately changes the layout.
I've build a copr package for this that can be found in copr under guss77/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#183 |
(In reply to Oded Arbel from comment #135)
> Currently, Alon Bar-Lev's patch
> (xorg-server-
> Xorg 1.17.2 from Fedora 22, but apparently has no effect
ּsorry, my bad - I was mistaken. Apparently the problem has something to do with GNOME's new layout switching handling. If I set "Modifiers-only switch to next source" to "Disabled" in the keyboard shortcut editor, then run
setxkbmap -option grp:switch,
Everything works fine with the patch.
So, we actually have two problems now, with layout switching kicking on on release - both the broken XKB protocol and the broken GNOME handling of "modifier-only" layout switching.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#184 |
Created attachment 122717
xorg-server-
Here is the patch from ubuntu, that can be merged into the current debian sid version of xorg-server (1.18.3-1).
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#185 |
Created attachment 126752
xorg-server-
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#186 |
Created attachment 126753
xorg-server-
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#187 |
Created attachment 129860
The same patch, but based on 1.19.1
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#188 |
(In reply to kyak from comment #140)
> Created attachment 129860 [details] [review]
> The same patch, but based on 1.19.1
You are missing dereference in XkbSA_LockMods case...
> case XkbSA_LockMods:
>+ filter = _XkbNextFreeFil
>+ sendEvent=
>+ break;
> case XkbSA_LockGroup:
> filter = _XkbNextFreeFil
>- *sendEvent = _XkbFilterLockS
>+ *sendEvent = _XkbFilterLockG
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#189 |
Not really.. This is something that has changed from 1.18 to 1.19.
"act" is now already passed as reference to XkbActionGetFilter.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#190 |
(In reply to kyak from comment #142)
I mean dereferencing sendEvent pointer.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#191 |
Created attachment 129861
The same patch, but based on 1.19.1 (fixed)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#192 |
You are right, thanks for pointing that out. Updated patch attached.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#193 |
Try, please, apply your patch, assign the left Ctrl as switching key and then to save something in the kate editor (or other) using Ctrl+S...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#194 |
(In reply to Serge Roussak from comment #146)
> Try, please, apply your patch, assign the left Ctrl as switching key and
> then to save something in the kate editor (or other) using Ctrl+S...
You mean to say the new patch doesn't work for you? Please be more clear.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#195 |
Yes, exactly. If I assign the left Ctrl as the switching key, then if I try to save a file in a text editor with the Ctrl+S, I got the "s" char in the file.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#196 |
(In reply to Serge Roussak from comment #148)
> Yes, exactly. If I assign the left Ctrl as the switching key, then if I try
> to save a file in a text editor with the Ctrl+S, I got the "s" char in the
> file.
And the layout switching - does it occur when you press Ctrl+s? Could you please test with Ctrl+Shift?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Lastique (andysem) wrote : | #1 |
- Dependencies.txt Edit (4.7 KiB, text/plain; charset="utf-8")
- JournalErrors.txt Edit (61.2 KiB, text/plain; charset="utf-8")
- ProcCpuinfoMinimal.txt Edit (921 bytes, text/plain; charset="utf-8")
- ProcEnviron.txt Edit (300 bytes, text/plain; charset="utf-8")
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Launchpad Janitor (janitor) wrote : | #2 |
Status changed to 'Confirmed' because the bug affects multiple users.
Changed in xorg-server (Ubuntu): | |
status: | New → Confirmed |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Timo Aaltonen (tjaalton) wrote : | #3 |
And if you had a look at the upstream bug you'd see that the patch doesn't work. It needs to get upstream first anyway.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Alex Ee (bibi4k0) wrote : | #4 |
I had the same problem.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
tlk (sarcasticskull) wrote : | #5 |
Timo Aaltonen (tjaalton) wrote 10 hours ago: #3
And if you had a look at the upstream bug you'd see that the patch doesn't work. It needs to get upstream first anyway.
-------
1) It's just one (incomplete) report that it doesn't. I'll try the patch myself soon.
2) The previous patch wasn't in upstream either, yet Ubuntu maintainers did absolutely right to accept it IMO.
3) AFAIK Xorg people rejected this change outright (that is? after some prolonged explanaitons what the actual problem was - shows that they just don't have the issue thus don't care too much).
3) Yet people shouldn't be held hostage because the proposed behavior breaks the XKB spec - without (? AFAIK) a single confirmed case of it breaking any real world X client.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Lastique (andysem) wrote : | #6 |
> And if you had a look at the upstream bug you'd see that the patch doesn't work.
I can see that someone reported it to not work for one-key combinations (e.g. Ctrl). I believe, the previous patch that was removed had the same limitation, and it was "good enough".
We need to see if it works for two-key combinations (e.g. Ctrl+Alt or Alt+Shift).
> It needs to get upstream first anyway.
It's unlikely to be accepted into XKb for the same reasons as the previous patch. This ticket is a request to add the patch to Ubuntu packages because this is a real problem that affects people.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#197 |
(In reply to Oleg from comment #149)
>
> And the layout switching - does it occur when you press Ctrl+s? Could you
> please test with Ctrl+Shift?
No, it does not. Currently I could not to test multi-keys switching combinations.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
MKorvin (guitarmaster-play) wrote : | #7 |
Same problem, waiting for fix.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
flux242 (flux242) wrote : | #8 |
please apply the old patch. Let the 17.04 work as the 16.10. Screw the upstream because it could take ages for them to decide
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#198 |
Did the old patch work for you? I mean could you switch the layout pressing Ctrl while combos like Ctrl-S also worked as expected?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
tlk (sarcasticskull) wrote : | #9 |
Thats the problem - the old patch doesn't work for the newer Xorg server.
AFAIU the upstream won't accept this anyway.
Thus it's the matter of testing the patch and getting it into Ubuntu. AFAIK Ubuntu was the only major distro that applied the old patch. Was yet another reason to use it.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#199 |
(In reply to Oleg from comment #151)
> Did the old patch work for you? I mean could you switch the layout pressing
> Ctrl while combos like Ctrl-S also worked as expected?
Which patch do you mean when you say "old"?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
MKorvin (guitarmaster-play) wrote : | #10 |
I have arch linux installed on another PC and Xorg server from https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#200 |
(In reply to Serge Roussak from comment #152)
> (In reply to Oleg from comment #151)
> > Did the old patch work for you? I mean could you switch the layout pressing
> > Ctrl while combos like Ctrl-S also worked as expected?
>
> Which patch do you mean when you say "old"?
The one that was made for Xorg versions prior to 1.19 https:/
The whole problem is that 1.19 needs a new one - this is the one provided by kyak https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#201 |
(In reply to Oleg from comment #153)
> (In reply to Serge Roussak from comment #152)
> > (In reply to Oleg from comment #151)
> > > Did the old patch work for you? I mean could you switch the layout pressing
> > > Ctrl while combos like Ctrl-S also worked as expected?
> >
> > Which patch do you mean when you say "old"?
>
> The one that was made for Xorg versions prior to 1.19
> https:/
>
> The whole problem is that 1.19 needs a new one - this is the one provided by
> kyak https:/
I have used the last one with the xorg-server v.1.19.3.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#202 |
Dear xorg developers. This is an old bug. In fact this is a 13 years old bug. And it has been at least 7 years since this bug had a patch to fix it.
Yes, existing patch breaks specification. But one would seem that the fact that patch existed for 7 years and was applied by default by popular distributions without users` complains would suggest that existing patch is a practical solution to the problem despite breaking theoretical (and as discussed here - not well thought through) specification.
There is now once more spike of activity here - probably because Ubuntu 17.04 was released recently which dropped this patch because it was not compatible with newer xorg. kyak has provided an updated patch, but it would take a lot of time for distributions to pick it up and then circle would repeat with next xorg version.
Yes, there has been a few heated discussions around this patch and XKB specification. And yes, specifications are important, but this is really annoying problem for people with more than one keyboard layout. Annoying to the point of making xorg really unusable for some groups of people. So is there any chance to get more practical with this bug?
For example, kyak's patch has a function 'xkbSwitchGroup
I feel like it would allow XKB spec to stay and also would allow users affected by this problem to solve it without recompiling xorg - which average user would struggle to do.
Looking forward to your reply.
Thanks!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#203 |
Nikolay,
First, background: I am not a xorg developer, but I develop the XKB library used by most Wayland compositors (xkbcommon). The behavior there is the same.
IMO it is worth having a discussion about the behavior. A nice thing about XKB is that it has a specification. It is easier to discuss behavioral changes against a spec than hard-to-understand X code, or 150 comments bugs. Therefore, my proposal for you (or anyone else who is interested in changing the behavior) is to provide a patch against the spec. Even if the actual spec will never change, I still think that a clear proposal, which also considers possible side effects, is the first step forward here.
The current spec and the most relevant section is here: https:/
The current source for the above spec is in the kbproto repository here: https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#204 |
The issue of fixing the spec was raised before, and the answer thn was:
(Vasily Khoruzhick from comment #37)
> Daniel is now working on XKB2, so fixing/changing XKB1 has no sense :)
As far as I know, XKB2 was never released, not even as a draft.
Are you saying that the situation has changed and changes to XKB are now welcome?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#205 |
No, I am only talking about myself, as a developer of xkbcommon; as I said, I am not a xorg developer so I cannot speak for that. I do not really mind if the spec is actually changed or not, what I am interested in is:
1. A clear and precise description of the proposed change.
2. A serious consideration of how other parts of the spec may be affected by the change.
You know how you want it to behave, so just write it down in the form a patch against the spec. Then we can discuss further, with the goal of reaching a solution in xkbcommon, at least. Let me know if you (or anyone else who wants to do it) need any help.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#206 |
Created attachment 131147
Proposed extension of the XKB protocol.
> Therefore, my proposal for you (or anyone else who is interested in changing the behavior) is to provide a patch against the spec.
That is a very good proposal. The patch formalises my proposal from comment #112. An implementation was posted it the following thread:
https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#207 |
Thanks Andreas, I had missed this in the previous discussion.
First, I must say it's a very clever hack :)
I have not looked at it deeply yet, but I have some initial questions:
- Can you describe a bit how you imagine the changes to xkeyboard-config would look with this approach?
- Do you think this can be done in a backwards compatible way? As far as xkbcommon is concerned, I am only interested in this scenario: old library (unaware of the new LockMods options) & new keymap (in textual form, modified with the new flags).
X has more compatibility concerns, xkbcomp/
BTW: xkbcommon already supports noLock/noUnlock in LockMods (unlike xkbcomp), based on your work[1]. So I hope the approach does not rely on these not already working :)
[1] https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#208 |
> - Can you describe a bit how you imagine the changes to xkeyboard-config
> would look with this approach?
We would need changes wherever we want the new behaviour. I would prefer to create new options for group switching, rather than modifying existing options. Comments #117 and #118 show examples. Basically, I imagine we use explicit action specifications. However, unless as shown in the examples, with changes to xkbcomp, one could write 'LockMods' with an 'group' option, rather than using the cryptic 'Private' actions.
> - Do you think this can be done in a backwards compatible way? As far as
> xkbcommon is concerned, I am only interested in this scenario: old library
> (unaware of the new LockMods options) & new keymap (in textual form,
> modified with the new flags).
Certainly, an old library or application would get parse errors when it encountered the new options in textual form.
If xkeyboard-config adds new options rather than modifying old ones, users could just keep using the old options until they upgraded the library/
As I far as remember, xkbcommon intentionally does not support 'Private'; otherwise, using it could help during the transition.
> X has more compatibility concerns,
Yes. Peter's main concern was that some tools might create binary forms of a layout where the two bytes that now get a meaning are not zero, but set to some random values (zero is fine, as it will have no effect in the new interpretation). His idea was to bump the protocol version, but that is beyond my capabilities, so I gave up at this point. Anyway, current xkbcomp puts the bytes to zero, so the combination of an old xkbcomp with new X-server would be no problem, ever without the precaution of a protocol version bump.
> BTW: xkbcommon already supports noLock/noUnlock in LockMods (unlike xkbcomp),
> based on your work[1]. So I hope the approach does not rely on these not
> already working :)
With my proposal, xkbcomp should be touched anyway, and this is just one of three occasions where I unsuccessfully tried to get noLock/noUnlock supported in xkbcomp.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
ainur (ainureg) wrote : | #11 |
I have it since yesterday, when i decided to update x-server packages
404 sudo apt-get install xserver-xorg-core xserver-
405 sudo apt-get install xserver-
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Andrey Vetlugin (antrew) wrote : | #12 |
This annoying bug takes my linux user experience 15 years back to 2002. Is there a way to make Ubuntu great again? Are there any known workarounds for this bug?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Vavooon (vavooon) wrote : | #13 |
Meanwhile I can suggest a partial workaround for the problem:
You can map layout switching to LEFT Ctrl+Shift and right keys will be available for using with shortcuts.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
tlk (sarcasticskull) wrote : | #14 |
Not exactly a bug, it's just how it works by default and that is what the xkb specs supposedly say. So this behavior dates way back certainly more that 15 yrs ))
And the fix was incorporated by Ubuntu only a couple of yrs ago certainly not in 2002 )))
The 'workaround' suggested above is what I use btw. And what have used on other distros.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
kolya (mar-kolya) wrote : | #15 |
That patch that fixes the problem has been applied in Ubuntu for way more than 'couple of years'
Unfortunately using right keys for modifiers doesn't work well with people who use 10 finger typing - they are used to press certain keys with certain fingers - to speed up their typing, and using wrong hand slows things down and leads to errors. Some modifier-heavy apps - like Emacs become very inconvenient if used with wrong modifier keys.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
tlk (sarcasticskull) wrote : | #16 |
well yeah more than a couple. I actually remember when I got fed up with this and went looking on the net what can be done and later found out that Ubuntu would accept the patch - I was on another distro then. just don't remember when exactly.
I do the 10 fingers typing but even if you don't it's annoying.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
ainur (ainureg) wrote : | #17 |
btw
here is solution
i can translate if anyone wants
https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
tlk (sarcasticskull) wrote : | #18 |
сАлюшн пересобрать руками с патчем? хабра такая хабра.
Changed in xorg-server (Ubuntu): | |
importance: | Undecided → Wishlist |
Changed in xorg-server: | |
importance: | Unknown → High |
status: | Unknown → In Progress |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#209 |
(In reply to Nikolay from comment #155)
> There is now once more spike of activity here - probably because Ubuntu
> 17.04 was released recently which dropped this patch because it was not
> compatible with newer xorg. kyak has provided an updated patch, but it would
> take a lot of time for distributions to pick it up and then circle would
> repeat with next xorg version.
On Ubuntu 16.04.03 LTS I just went from using Unity to Gnome. With some back and forth with lightdm vs. gdm and the screen lock issues I got things working more or less until I stumbled over what probably are the ripples of this old bug (and I couldn't agree more with Nikolay and his comment).
I had two keyboard layouts running with Unity, and as everybody seems to have, I used Alt-Shift to switch layouts, just like on my Windows laptop. And like almost everyone else, I of course have Alt-Shift-Tab set for cycling backwards through windows (after jumping through all sorts of hoops to tell the window manager not to group windows of the same app...).
I reported my issue at https:/
I haven't taken the time to understand all the details, so I apologize for the outside perspective. Yet, I am of the opinion that it should be possible to define shortcuts such that they trigger their event only upon release, not upon press, because as noted by everyone else here it makes it impossible to define "release" shortcuts that are "prefixes" of other shortcuts, and this is a severe limitation that becomes highly practically relevant for the usual keyboard layout switching shortcut that just so happens to be a prefix of the usual window backwards switching shortcut.
There have been much worse backward incompatibilities in Linux. And while I hate seeing things break by such changes, here my very personal view is that should this really present a problem that cannot be solved in a backward-compatible way (e.g., by letting users distinguish between "release" and "press" shortcut definitions) then so be it. What is this compared to deprecating X11, then xorg and at some point also Wayland...?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Artem Kovalov (artemkovalov) wrote : | #19 |
Пересобрать таки помогло) Аж выдохнул. Жаль, что мейнтейнеры ppa не обновляют оперативно билды для свежих релизов.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Anton Petrusevich (apetr) wrote : | #20 |
Please return the patch back!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Igor Kostromin (elw00d) wrote : | #21 |
manual patching recipe (thx Artem Kovalov) for Ubuntu 16.04.3
https:/
I uploaded deb to https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Pavel (spvkgn) wrote : | #22 |
I uploaded xorg-server 1.19.3 packages for 17.04 with included patch from https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Pavel (spvkgn) wrote : | #23 |
Also .deb files are available for download here https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Jacob Becker (jabecker) wrote : | #24 |
I uploaded xorg-server 1.19.3 packages for 16.04.3 i386 with included patch from https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Jacob Becker (jabecker) wrote : | #25 |
Here's my script based on the gist mentioned on comment #21
https:/
It tries to be generic. It uses a separate configuration file for each platform, which means it is extensible. It is still apt-based, though.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Vitaliy (velnik) wrote : | #26 |
I had the same problem.
Ubuntu Mate 16.04.3 LTS
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
A (zorn-v) wrote : | #27 |
My script based on Jacob's one.
https:/
It removes all packages installed for build (with prompt).
Patch downdload commented out.
Bolow this line is the cry of the soul
=======
It is IDIOTISM. It's need to recompile WHOLE X.ORG for simple behavior whose should be in some config at least.
Many dislikes and curse to upstream mainteiners. AFAIK this bug since 200X. THIS IS SHAME.
You can put your important opinion in the...
Yes, I don't respect you as you don't respect users (and distro mainteiners) of your lib.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
A (zorn-v) wrote : | #28 |
Also works on 17.10 BTW.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
A (zorn-v) wrote : | #29 |
Now I understand why "libs" like wayland appears. X.org SUCKS.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#210 |
After the next update of the package "xserver-
Distributions on which this problem was repeated:
neon-usereditio
kubuntu-
kubuntu-
Distributions in which there is no such problem:
ubuntu-
ubuntu-
When setting the switch to Ctrl-Shift, you can not quickly select text with the "Ctrl-Shift-Arrows" keys. However, you do not want to have outdated xorg packages. Extremely uncomfortable, help!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#211 |
Me too very upset about this bug: there is patch for several yers - no one cares. I suggest compromise: add autotools|meson option to enable this feature. At least every distro would be able to decide whether they want to provide nonstandard behavior.
I also see that this bug is assigned but nothing happens.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#212 |
Why don't you change your layout switcher key-combination? You can set up another key-combinations for russian and different key-combination for english keyboard,too. That's KDE's problem, that allow choose frequently used key combinations.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#213 |
With all due respect, I hope this is not the usual response to X bugs. I do not think it is unreasonable to ask that shortcuts be made available from the set of control keys, since this is a very widespread feature across operating systems. I have used Kubuntu for the past several years, and made use of the alt-shift key combination to toggle between the English and Greek keyboard layouts. After the recent KDE change I find this bug affecting my workflow, interrupting various keyboard combinations involving the use of those modifiers. I think it is appropriate to call this a bug in xkb's behaviour.
I wholeheartedly understand that X developers are more thinly stretched than most, and that this bug—despite its labeled importance—does not have as extreme a disruptive impact as many other bugs. I have no desire to complain of the amount of time this bug has spent open and demand that it be fixed. I would however request that this bug not be closed, as it is indeed a bug and the responsibility of xkb.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#214 |
The main keyword the KDE interface programming philosophy called Quick Time. Windows use message queue, and uses built togheter windows manager with the os hardly. Qt enables for programs override every reactions, "answered" the user actions. Under xcfe desktop manager main program override subprograms actions, for example, midnight commander is not closeable pressed f10 key, it override the main window. KDE close midnight commander pressing f10 key.
Xkb layouts can not to do anything with this problem.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#215 |
This bug is not exclusive to KDE. I use XFCE on a different machine as well, and encounter the bug there. Various people in this thread have reported it in other WMs. Is there a reason why the solution should not be, as the title says, that XKB kick its hotkeys on release rather than press?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#216 |
Who invite me this blog?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#217 |
(In reply to Kovács Viktor from comment #165)
> Why don't you change your layout switcher key-combination? You can set up
> another key-combinations for russian and different key-combination for
> english keyboard,too. That's KDE's problem, that allow choose frequently
> used key combinations.
This has nothing to do with KDE, but lack of functionality of xkb.
There is universal key sequence to switch layout which is common to most operating systems, once this sequence is selected, functionality is lost.
Comment#161 summaries the options to actually solve the issue, until resolved people should be able to land here to apply the patch which I use for 10 years.
Changed in xorg-server: | |
status: | In Progress → Confirmed |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#218 |
Created attachment 134843
KDE keyboard layout switcher screenshot
Sorry, I'm Hungarian.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#219 |
Original problem was, that Shishakov uses ctrl+shift keyboard layout keycombination, and it not works, because it can be used in hotkeys.
Why do not change It? It is much easier thing, I think.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#220 |
@Kovács Viktor, this report is not only about "original" problem, but all the problems affected by current implementation. I described other one with modifier for national characters, once you define such key you cannot use it as shortcut modifier.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#221 |
High, Viktor!
I'm using Ctrl+Shift for twelve years since Win XP. I use this combintaion on my win machines. So it's definetely not an option for me. Since I use Cyrillic layout I switch layouts frequently (people using Latin layouts are less affected). Imagine saving action shortcut "Ctrl+S" combintaion would be replaced with something like "Alt+R" in your favourite text editor.
I'm also sure that no one will complain if the patch will be merged. And I do not understand why should we follow the standard if this standard makes living uncomfortable.
BTW according to previous comments wayland implentations suffer from this bug too despite they do not depend on Xorg-server. Do we need separate patch for libxcb and libxcbcommon?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#222 |
I understand, that you must switch between Russian and English layout. The programs uses latin-based hotkeys, I did understand. In my opinion, in the future would be change, if the translator projects "translate" the hotkeys, for example to cyrillyc letters, too. (then won't need switches between layouts so frequently) We (Hungarians) will run into that problems on the future, when we will want to write old Hungarian texts. This is a runic-like right-to-left script. My question is, do you use Windows yet, or you uses already only Linux? You can choose switcher combination closely positioned with ctrl and shift. If you buy a new laptop, it could be happen, that the keyboard metrics not exactly same as on previous one.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#223 |
I use both Windows and Linux. BTW Many people don't have a choise of the OS on their jobs.
I may provide a patch that would make it available to switch the behaviour in the runtime (I guess a line in xorg.conf would be fine). So we don't break the standard, people don't need to recompile xorg every time and patch developers don't need to adapt the patch.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#224 |
(In reply to Yan Pas from comment #176)
> I use both Windows and Linux. BTW Many people don't have a choise of the OS
> on their jobs.
>
> I may provide a patch that would make it available to switch the behaviour
> in the runtime (I guess a line in xorg.conf would be fine). So we don't
> break the standard, people don't need to recompile xorg every time and patch
> developers don't need to adapt the patch.
You should get developers' feedback about this approach first. If they are not willing to merge the patch that works via xorg.conf, you will only waste your time.
From the other hand side, you will probably waste your time anyway.
Developers talk about the "standard", and explicitely suggest sending patches to standard instead of patching xorg (read previous comments). And when such patch for the standard gets attached to this bug report, what happens? Nothing.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#225 |
(In reply to Vitaly Shishakov from comment #0)
> I used to use Ctrl-Shift combination to switch keyboard layouts (ru <--> us)
>
> but in this case i cant use any of the Ctrl-Shift-* hotkeys in any software
> i try.
>
> I noticed, that the keyboard layout becomes swithced as soon an both keys
> are DOWN --
> pressing any other key is not treated as Ctrl-Shift-<key> combination.
>
> For example -- in Windows i also use Ctrl-Shift to switch layouts, but
> there, the layout
> becomes switched only when both SHIFT and CTRL keys are UP, and no other key
> was
> pressed while they were down -- in that case the hole combination is treates
> as
> Ctrl-Shift-<key> combination, and the layout is not changed.
>
> see also: http://
>
>
>
> I use the following lines in XF86Config:
>
> Section "InputDevice"
> Driver "Keyboard"
> Identifier "Keyboard[0]"
> Option "Protocol" "Standard"
> Option "XkbLayout" "us,ru"
> Option "XkbModel" "pc104"
> Option "XkbOptions" "grp:ctrl_
> Option "XkbRules" "xfree86"
> Option "XkbVariant" ",winkeys"
> EndSection
Kubuntu grants Control+Shift+K combination. Is it acceptable?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#226 |
(In reply to Kovács Viktor from comment #178)
> Kubuntu grants Control+Shift+K combination. Is it acceptable?
The question is not, largely speaking, about whether there is any shortcut available. The question is largely specifically about shortcuts such as Ctrl+Shift or Alt+Shift, which due to this bug will interfere with other shortcuts and applications. Users such as myself are accustomed to using these shortcuts, and they should be expected to work.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#227 |
(In reply to Zebediah Figura from comment #179)
> (In reply to Kovács Viktor from comment #178)
> > Kubuntu grants Control+Shift+K combination. Is it acceptable?
>
> The question is not, largely speaking, about whether there is any shortcut
> available. The question is largely specifically about shortcuts such as
> Ctrl+Shift or Alt+Shift, which due to this bug will interfere with other
> shortcuts and applications. Users such as myself are accustomed to using
> these shortcuts, and they should be expected to work.
Yes, I was thinking about it before you answered. I tested RCtrl+RShift combination, and it has no conflicts with hotkeys. There are possibility of XKeyboard-config, I used it before a keyboard layout: caps_switch_latch. It works when capslock pushed, but not released. I just thinking, I am in a black room. But it cannot do anything with hotkeys. I think, that problem is a miner-like. X-windows have messages keypress and release, but in my opinion, it should be adopt all windows manager. I will read the according X11 header files first, I promiss. After that I come back again, could we step to an easy way or not.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
V-ctor (radioengineer) wrote : | #30 |
- xorg-server_1.19.5-0ubuntu2.1_amd64.build Edit (723.0 KiB, text/plain)
How to make it works on 17.10?
There is no package xserver-
I did this:
1. sudo apt install devscripts
2. apt-get build-dep xorg-server
3. apt source xserver-xorg-core
4. wget https:/
5 cd xkb
6. patch < attachment.
7. debuild -us -uc
and got:
# Make sure Xvfb at least starts up
PATH=debian/
debian/
install debian/
make[1]: выход из каталога «/home/
debian/rules override_dh_install
make[1]: вход в каталог «/home/
# dh_install --fail-missing
dh_install --list-missing
dh_install: Please use dh_missing --list-
dh_install: This feature will be removed in compat 11.
/home/victor/
/home/victor/
/home/victor/
/home/victor/
/home/victor/
/home/victor/
/home/victor/
/home/victor/
/home/victor/
/home/victor/
/home/victor/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Norbert (nrbrtx) wrote : | #31 |
This bug is a duplicate of well known bug 1245473 .
tags: |
added: artful xenial removed: amd64 third-party-packages |
tags: | added: bionic |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Sebastian Bär (sebastian-ubuntu-one) wrote : | #32 |
In case you don't need to switch between keyboard layouts, you can get some shortcuts working again if you use the "Tweaks" tool for Gnome.
sudo apt install gnome-tweak-tool
Then
1. Open "Tweaks"
2. Select the tab "Keyboard & Mouse"
3. Click "Additional Layout Options"
4. Disable entries in the "Switching to another layout" dialog
In my case this gave me back the "ALT + SHIFT" combinations which are essential in the Eclipse IDE for refactoring.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#228 |
Sorry, on newer Linux you can set up hot key combination for that problem as graphical UI settings, older Linux versions will not be updated. May I close It?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#229 |
(In reply to Kovács Viktor from comment #181)
> Sorry, on newer Linux you can set up hot key combination for that problem as
> graphical UI settings, older Linux versions will not be updated. May I close
> It?
With all respect to you... Why do you want to close well described bug moreover that it has good proposed solution? This issue affects multiple users using multiple keyboard layouts and merging available patch probably won't cause any problems to the rest of people that does not use layout switchig.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#230 |
(In reply to Jan Pohanka from comment #182)
> (In reply to Kovács Viktor from comment #181)
> > Sorry, on newer Linux you can set up hot key combination for that problem as
> > graphical UI settings, older Linux versions will not be updated. May I close
> > It?
>
> With all respect to you... Why do you want to close well described bug
> moreover that it has good proposed solution? This issue affects multiple
> users using multiple keyboard layouts and merging available patch probably
> won't cause any problems to the rest of people that does not use layout
> switchig.
Sorry, but where do you see "good proposed solution"? Please, please show me that solution.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#231 |
(In reply to Kovács Viktor from comment #181)
> Sorry, on newer Linux you can set up hot key combination for that problem as
> graphical UI settings, older Linux versions will not be updated.
What exactly "newer Linux" is supposed to mean? I'm using yesterday's updated Gentoo Linux with xorg-server-1.19.5 and Fluxbox - is it counts as "newer"? Or by "newer Linux" you mean something like "modern KDE only"?
(In reply to k0fe from comment #183)
> Sorry, but where do you see "good proposed solution"? Please, please show me
> that solution.
There is a patch, working good enough for years without creating any new (practical) issues. In comment #161 is was proposed to make it configurable option to make everyone happy and let users choose between using compliant protocol or working hotkeys.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#232 |
(In reply to Alex Efros from comment #184)
> (In reply to Kovács Viktor from comment #181)
> > Sorry, on newer Linux you can set up hot key combination for that problem as
> > graphical UI settings, older Linux versions will not be updated.
>
> What exactly "newer Linux" is supposed to mean? I'm using yesterday's
> updated Gentoo Linux with xorg-server-1.19.5 and Fluxbox - is it counts as
> "newer"? Or by "newer Linux" you mean something like "modern KDE only"?
>
> (In reply to k0fe from comment #183)
> > Sorry, but where do you see "good proposed solution"? Please, please show me
> > that solution.
>
> There is a patch, working good enough for years without creating any new
> (practical) issues. In comment #161 is was proposed to make it configurable
> option to make everyone happy and let users choose between using compliant
> protocol or working hotkeys.
Where is the link to the patch? Where is the instruction like how to apply this patch? And how can this comment be used by an ordinary user (without knowledge of programming) to solve this problem?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#233 |
(In reply to k0fe from comment #185)
> (In reply to Alex Efros from comment #184)
> > There is a patch, working good enough for years without creating any new
> > (practical) issues. In comment #161 is was proposed to make it configurable
> > option to make everyone happy and let users choose between using compliant
> > protocol or working hotkeys.
>
> Where is the link to the patch? Where is the instruction like how to apply
> this patch? And how can this comment be used by an ordinary user (without
> knowledge of programming) to solve this problem?
As an ordinary Gentoo Linux user, I:
- download patch attached to this issue named "The same patch, but based on 1.19.1 (fixed)" into directory /etc/portage/
- run `emerge xorg-server` to reinstall Xorg with this patch applied
- restart X to enjoy working hotkeys :)
If users of other Linux distributions have issues with this - probably they just didn't use "newer Linux". </sarcasm>
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#234 |
(In reply to Kovács Viktor from comment #181)
> Sorry, on newer Linux you can set up hot key combination for that problem as
> graphical UI settings, older Linux versions will not be updated. May I close
> It?
Please do not close this bug. If you do not want to receive any further updates on it, you can unsubscribe by removing yourself from the Cc list.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#235 |
(In reply to Alex Efros from comment #186)
> (In reply to k0fe from comment #185)
> > (In reply to Alex Efros from comment #184)
> > > There is a patch, working good enough for years without creating any new
> > > (practical) issues. In comment #161 is was proposed to make it configurable
> > > option to make everyone happy and let users choose between using compliant
> > > protocol or working hotkeys.
> >
> > Where is the link to the patch? Where is the instruction like how to apply
> > this patch? And how can this comment be used by an ordinary user (without
> > knowledge of programming) to solve this problem?
>
> As an ordinary Gentoo Linux user, I:
> - download patch attached to this issue named "The same patch, but based on
> 1.19.1 (fixed)" into directory /etc/portage/
> - run `emerge xorg-server` to reinstall Xorg with this patch applied
> - restart X to enjoy working hotkeys :)
>
> If users of other Linux distributions have issues with this - probably they
> just didn't use "newer Linux". </sarcasm>
I'd like to thank you! And great thanks to kyak for the patch!
I confirm that it works for me on "older" Gentoo and xorg-server 1.9.5.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#236 |
There seems to have been a proposed protocol extension (comment #159 etc.) Can anyone shed light, for the outside user, as to the current status of this proposal? Thanks.
Changed in xorg-server (Ubuntu): | |
assignee: | nobody → Alex (alexvaa) |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#237 |
(In reply to Zebediah Figura from comment #189)
> Can anyone shed light, for the outside user, as to the current status of
> this proposal? Thanks.
No news since. Apart from the formal proposal, there are some old patches for its implementation:
https:/
https:/
https:/
https:/
https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#238 |
(In reply to Andreas Wettstein from comment #190)
> No news since. Apart from the formal proposal, there are some old patches
> for its implementation:
> https:/
> https:/
> https:/
> https:/
> https:/
Here's what I think we would need to do in order to not break old clients:
https:/
Another, probably better, way to do it would be to define a new flag like XkbSA_HasGroupFlags inside the XkbModAction flags field when group_flags and group_XXX are valid rather than potentially garbage. That would avoid the whole version-negotiation nightmare, as nothing appears to be too picky about extra flags being defined.
Five years later, it would also be good to have support inside libxkbcommon (which has a pretty decent test suite) and xcb-proto for the flags.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Raycho Raykov (raycho-raykov) wrote : | #33 |
I patched my Lubuntu 16.04 with the mentioned upward patch few hours ago and everything starts working perfectly. I also tested Linux Mint 18.3 with Cinnamon and everything works as expected without any patches. The problem exists in Ubuntu MATE 16.04 too. All these are 64 bit versions. The patch is created correctly: when I released first button from the sequence, the sequence is executed. This way when I switching locales and keep Shift pressed (I'm using Alt+Shift) I can start writing with capital letter. I hope this information is helpful.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#239 |
(In reply to Alex Efros from comment #184)
> There is a patch, working good enough for years without creating any new
> (practical) issues. In comment #161 is was proposed to make it configurable
> option to make everyone happy and let users choose between using compliant
> protocol or working hotkeys.
(In reply to Aliaksei Urbanski from comment #188)
> I'd like to thank you! And great thanks to kyak for the patch!
> I confirm that it works for me on "older" Gentoo and xorg-server 1.9.5.
I could to say, that this patch does not work exactly as expected. See, please, my comment #146 for explanation. So I think, this thread could not be closed.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#240 |
Another user here voting for this bug to get patched OFFICIALLY from upstream.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#241 |
Did you changed your opinion after for years of not fixing this bug?
Users still need this functionality (see https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#242 |
All current Ubuntu versions are affected and RHEL too. And nobody cares.
14 years of doing nothing. My congratulations!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#243 |
(In reply to Daniel Stone from comment #191)
> Another, probably better, way to do it would be to define a new flag like
> XkbSA_HasGroupFlags inside the XkbModAction flags field when group_flags and
> group_XXX are valid rather than potentially garbage. That would avoid the
> whole version-negotiation nightmare, as nothing appears to be too picky
> about extra flags being defined.
>
> Five years later, it would also be good to have support inside libxkbcommon
> (which has a pretty decent test suite) and xcb-proto for the flags.
This comment lays out the best way forward for anyone interested to fix this bug. It shouldn't be too difficult, but personally I haven't worked on X11 for quite some time.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#244 |
Just tested simple idea on Ubuntu 16.04.4 LTS with MATE DE.
Out-the-box it has Xorg 1.18.4 which perfectly allow user to set for example <Ctrl+Shift> keyboard shortcut for keyboard layout switching.
But when I install HWE on 16.04 LTS I get newer Xorg 1.19.5.
Debian 8 (Xorg 1.16.4) and 9 (Xorg 1.19.2) have this problem too (see https:/
So versions 1.16.4, 1.19.2 and 1.19.5 should be patched to bring <Ctrl+Shift> keyboard shortcut functionality back.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Norbert (nrbrtx) wrote : | #34 |
From duplicate bug 1720364 report.
Steps to reproduce:
1. Install ubuntu-mate-desktop on Ubuntu 16.04 LTS with HWE (Xorg 1.19.5), or 17.10 or 18.04 LTS.
2. Set-up two keyboard layouts - English and Russian
3. Set <Ctrl+Shift> as keyboard layout switcher
4. Try to use shortcuts starting from <Ctrl+Shift>:
4.1. Open Firefox, open new tab, go to some site in it, close tab, try to click <Ctrl+Shift+T> to restore closed tab.
4.2. Open mate-terminal, try to open new tab with <Ctrl+Shift+T>, or copy (<Ctrl+Shift+C>), or paste (<Ctrl+Shift+V>).
4.3. Open pluma, write some text, try to navigate in it with <Ctrl+Shift+arrow>.
Expected results:
<Ctrl+Shift> switches keyboard layout, shortcuts starting from <Ctrl+Shift+...> work normally.
Actual results:
<Ctrl+Shift> switches keyboard layout, shortcuts starting from <Ctrl+Shift+...> do not work.
Notes:
1. Ubuntu 16.04 LTS (Xorg 1.18.4) with Marco and Compton work normally with <Ctrl+Shift> keyboard layout switcher.
2. This problem was discovered before on 13.10, 14.04 and other modern versions with GNOME desktop (Metacity and Compiz) - see bug 1245473.
tags: | removed: zesty |
description: | updated |
description: | updated |
Changed in xorg (Debian): | |
status: | Unknown → New |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
A (zorn-v) wrote : | #35 |
https:/
There is a script I run if x.org updated.
Fuck X.Org contributors and FUCK they minds.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#245 |
Did somebody tested Russian rulemak keyboard layout? It is between the extra layouts. It is based on russian layout with latin letters. I' m not a russian man, but I think, just testing first is a good idea!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#246 |
Viktor, could you please elaborate how exactly that may be useful here?
I would also like to ask you to make sure you contribute useful content to this discussion, or if you cannot, resist from posting. I don’t think it helps anyone to to support flamewars in this bug report, or even make the discussion longer than necessary.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Norbert (nrbrtx) wrote : | #36 |
- Patch for Bionic Beaver 18.04 LTS, it fixes the problem Edit (3.7 KiB, text/plain)
I attached patch from ArchLinux AUR (https:/
It fixes current bug on Ubuntu Bionic Beaver 18.04 LTS.
I have tested it with <Ctrl+Shift> keyboard layout switcher on GNOME (DESKTOP_
Please kindly review it and consider applying it before official final release of 18.04 LTS.
I'm ready to test proposed packages which (I hope) you would create.
As the result Ubuntu users will be as happy as ArchLinux's users.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Ubuntu Foundations Team Bug Bot (crichton) wrote : | #37 |
The attachment "Patch for Bionic Beaver 18.04 LTS, it fixes the problem" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.
[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]
tags: | added: patch |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Anton V Zharnikov (antonprogz) wrote : | #38 |
This helped me:
https:/
Be aware of xserver version numbers.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Khurshid Alam (khurshid-alam) wrote : | #39 |
@Norbert
Can you make ppa with the patch so that everyone test it easily?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Norbert (nrbrtx) wrote : | #40 |
@khurshid-alam
I can't determine how many users want to have this bug fixed. So we can start with the complete working instruction:
1. Set Ctrl+Shift keyboard layout switcher on Ubuntu 18.04 LTS
2. Ensure that Ctrl+Shift+ interference exists
3. Enable "deb-src" sources (via `software-
cd ~/Downloads
apt-get source xserver-xorg-core
cd xorg-server-1.19.6/
wget https:/
patch xkb/xkbActions.c < freedesktop-bug-865.patch
sudo apt-get install devscripts
sudo apt-get build-dep xserver-xorg-core
debuild -uc -us
sudo dpkg -i ../x*.deb
4. Reboot and test again.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Norbert (nrbrtx) wrote : | #41 |
Dear all!
I have created PPA ( https:/
You can test them by the following commands:
sudo add-apt-repository ppa:nrbrtx/
sudo apt-get update
sudo apt-get dist-upgrade
Hope this helps.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Oded Arbel (oded-geek) wrote : | #42 |
I've installed and tested Norbert's PPA and it works great. I have ALT+SHIFT set as the layout switch shortcut and with the PPA I can now use Eclipse's ALT+SHIFT+<key> keyboard shortcuts.
Kudos Norbert!
Sorry it took so long - because the PPA is only for LTS releases, I had to wait until it was safe for me to update to the 18.04 beta (i.e. no urgent work scheduled). I hope with the upcoming release there will be more testers and maybe we can get Ubuntu maintainers to pick up the patch.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
gl4dor (gl4dor) wrote : | #43 |
@nrbrtx
It seems that the patch don't work with the french layouts. It work fine with all the other layouts i tested.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Roman Shchekin (mrqtros) wrote : | #44 |
@nrbrtx I love you, dude, thanks for your fix!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
FractalizeR (fractalizer) wrote : | #45 |
Will this patch be incorporated into the core? This new behavior is annoying ;)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Maxym (khoriam) wrote : | #46 |
@nrbrtx thanks, worked like a charm!
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Elad Hen (eladhen2) wrote : | #247 |
@Norbert
Is there a way to apply this patch to debian stretch (xorg 1.19.2)?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#248 |
FYI the bug may be temporarily fixed on Ubuntu 16.04 LTS (with HWE), Ubuntu 18.04 LTS (and Mint 19) using packages from my PPA ( https:/
And it is unbelievable that we need to patch core graphical system component by ourselves to use traditional keyboard shortcuts ...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Snaker (snaker.me) wrote : | #249 |
@Norbert
I have the same problem on Ubuntu 18.04 LTS with GNOME.
I tried your instruction from #40 without any success. Do you have an idea how to make it work?
P.S. How can I roll back changes?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Snaker (snaker.me) wrote : | #250 |
@Norbert
Update:
After cleaning up changes and applying your ppa:nrbrtx/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#251 |
-- GitLab Migration Automatic Message --
This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.
You can subscribe and participate further through the new bug through this link to our GitLab instance: https:/
Changed in xorg-server: | |
status: | Confirmed → Unknown |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#252 |
Thanks for providing here attachments of Linux given and Assignee by Xorg Project Team.
Ida,
As an assignment writer, give assistance with psychology homework - http://
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Shegon (shegon) wrote : | #253 |
Yes, I confirm the bug.
tags: |
added: cosmic disco eoan removed: artful |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#254 |
I like the helpful info you supply to your articles. I’ll bookmark your blog and take a look at once more here frequently. I am somewhat sure I will be told plenty of new stuff right right here! Good luck for the following!
https:/
https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#255 |
An extensive yet tactical take on your Career Episodes can help represent everything that makes you an asset to the Engineers Australia. Career Episodes can open those closed doors comfortably. They demonstrate the key skills, the experience, and the expertise you bring along in your engineering domain. Every one of the three episodes is required to touch base on unique projects and experiences, and your role, and strategies. At CDRReport, we have that expertise to make your Career Episodes sound valid and convincing.
https:/
https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#256 |
Hi! I have the same issue i would like to know the solution
Castro B,
gratisdatingsite.nl
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#257 |
<email address hidden>: Why did you reopen this task without any comment explaining the reason? See comment 201?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#258 |
Looking for a great sample of the philosophical essay? Try reading the Epiphenomenal Qualia Essay sample. Epiphenomenalism and the knowledge argument are distinctly consistent. However, the knowledge-gained argument negates the principle of epiphenomenalism: https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#259 |
Need some additional help with your essays Barely manage to complete all assignments in time? Stop worrying and contact the team of professional writers of the great essays and get rid of all tasks! https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#265 |
tags: | removed: cosmic |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Oded Arbel (oded-geek) wrote : | #260 |
Daniel, I understand from your change that you think this bug does not affect cosmic?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
A (zorn-v) wrote : | #261 |
Its not a bug, its a feature... of X server ))
DAMN, the little patch and so many bustles
One more time (how long wait about wayland for nvidia on KDE ? :))
https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Vlad Orlov (monsta) wrote : | #262 |
Cosmic is EOL since July 18, therefore the tag is removed
https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Norbert (nrbrtx) wrote : | #263 |
Just a silent reminder: I have created a PPA ( https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Oded Arbel (oded-geek) wrote : | #264 |
Norbert: and it is working great! I wish Canonical would just pick it up.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#266 |
At Best CDR Writing, we are providing you the best CDR and RPL reports atan affordablr cost. We will deliver you 100% pilagarism free and best quality work.
https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#268 |
Found your post interesting to read. I cant wait to see your post soon. Good Luck for the upcoming update. This article is really very interesting and effective.
Best Places to Order Personal and Business Checks Online - https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
|
#269 |
Found your post interesting to read. I cant wait to see your post soon. Good Luck with the upcoming update. This article is really very interesting and effective. take a look at the link here for Best Places to Order Personal and Business Checks Online - https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Daniel van Vugt (vanvugt) wrote : | #270 |
Is there a reason this bug is open when bug 36812 is also reopened?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Jacob Becker (jabecker) wrote : | #271 |
To whom it may concern, I have uploaded xorg-server 1.20.8 packages for Ubuntu 18.04.5 amd64 (with the patch from https:/
Generally, to get the topic issue fixed, you need to install only one of the packages: xserver-
To apply the patch and build the binaries and packages, I used my script mentioned in comment #25.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Daniel van Vugt (vanvugt) wrote : | #272 |
Please propose any fixes on the Gitlab site and link them to:
https:/
tags: | removed: disco eoan |
Changed in xorg-server: | |
importance: | High → Unknown |
tags: |
added: focal hirsute impish jammy removed: xenial |
Changed in xorg-server (Ubuntu): | |
assignee: | Alex (alexvaa) → nobody |
tags: | removed: hirsute |
Changed in xorg-server: | |
status: | Unknown → New |
tags: | removed: impish |
tags: | added: kinetic |
I used to use Ctrl-Shift combination to switch keyboard layouts (ru <--> us)
but in this case i cant use any of the Ctrl-Shift-* hotkeys in any software i try.
I noticed, that the keyboard layout becomes swithced as soon an both keys are DOWN --
pressing any other key is not treated as Ctrl-Shift-<key> combination.
For example -- in Windows i also use Ctrl-Shift to switch layouts, but there, the layout
becomes switched only when both SHIFT and CTRL keys are UP, and no other key was
pressed while they were down -- in that case the hole combination is treates as
Ctrl-Shift-<key> combination, and the layout is not changed.
see also: http:// bugs.kde. org/show_ bug.cgi? id=59303
I use the following lines in XF86Config:
Section "InputDevice" shift_toggle"
Driver "Keyboard"
Identifier "Keyboard[0]"
Option "Protocol" "Standard"
Option "XkbLayout" "us,ru"
Option "XkbModel" "pc104"
Option "XkbOptions" "grp:ctrl_
Option "XkbRules" "xfree86"
Option "XkbVariant" ",winkeys"
EndSection