Keyboard layout change on hotkeys press instead of release and do not work well with shortcuts

Bug #36812 reported by Sergey Sinitsa
628
This bug affects 121 people
Affects Status Importance Assigned to Milestone
X.Org X server
New
Unknown
Nominated for Trunk by ksn135
gnome-control-center
Unknown
Medium
gnome-control-center (Ubuntu)
Invalid
Low
Unassigned
xorg-server (Ubuntu)
Invalid
Wishlist
Unassigned

Bug Description

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.

For users being affected by this problem, the easiest solution for now is to add this PPA in your repositories:
https://launchpad.net/~oded-geek/+archive/xorg-patches

Practical summary of this bug for ubuntu developers (since reading 120 comments is impractical for most):
This problem is a really old (since 2004) issue of the xkb part of xorg; the main discussion was made upstream in freedesktop-bugs #865. There has been a patch from Ilya Murav'jov for upstream (#55), and attached here (#61).
Upstream xorg has refused to apply the patch, mainly because it "explicitly contradicts the (xkb) spec" (#84, #91).
This patch has been reported to work for many people without any problems, and there is also a PPA by Oded Arbel (#95) where he maintains a patched version of the ubuntu xorg.
The proper resolution of this bug would be to apply this patch to the upstream xorg, or at minimum to the official ubuntu xorg package.

Related branches

Revision history for this message
In , Kristian Hoegsberg (krh-bitplanet) wrote :

*** Bug 731 has been marked as a duplicate of this bug. ***

Revision history for this message
Sergey Sinitsa (sin3) wrote :

I have Ctrl+Shift set to layout switch and press ctrl+shift+N (to open new Terminal or new tab in Opera) -- as a result I have my keyboard layout switched. This happens because Ubuntu (or XOrg?) change layout on key press instead of release and do not match layout hotkeys with application shortcuts. MS Windows do. If layout change hotkeys is pressed with another key this must be treated as application shortcut and layout must not be changed. Combined with shortcut bugs in some Linux applications [Bug 2561] this makes keyboard shortcuts nearly unusable in default bilingual configuration.

Revision history for this message
In , Erik Andrén (erik-andren) wrote :

Are you able to reproduce this issue with a current version of xorg?

Revision history for this message
In , Daniel Stone (daniels) wrote :

yep, it's still a current issue. reasonably non-trivial fix, though. we could
probably hijack PKE for this.

Revision history for this message
In , Shift-cmpd2 (shift-cmpd2) wrote :

(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.

Revision history for this message
Nick Andrik (andrikos) wrote :

I can also confirm this bug.
I use Alt+Shift for language change and Alt+Shift+Tab for backwards changing windows.
Thus, I cannot change the windows backwards

Revision history for this message
Daniel Holbach (dholbach) wrote :

Thanks for the bug report.

Somebody of the team could forward this upstream.

Changed in control-center:
assignee: nobody → desktop-bugs
importance: Medium → Low
status: Unconfirmed → Confirmed
Revision history for this message
In , Daniel Stone (daniels) wrote :

Sorry about the phenomenal bug spam, guys. Adding xorg-team@ to the QA contact so bugs don't get lost in future.

Revision history for this message
In , Daniel Stone (daniels) wrote :

*** Bug 10662 has been marked as a duplicate of this bug. ***

Revision history for this message
Sebastien Bacher (seb128) wrote :
Changed in control-center:
status: Unconfirmed → Unknown
Revision history for this message
Sergey Sinitsa (sin3) wrote :

http://bugzilla.gnome.org/show_bug.cgi?id=430637
Comment #1 from Sergey V. Udaltsov (developer, points: 14)
2007-04-17 12:44 UTC [reply]
NOTGNOME. That's the way X works.

Revision history for this message
Sebastien Bacher (seb128) wrote :

closing the Ubuntu task then

Changed in control-center:
status: Confirmed → Rejected
Changed in control-center:
status: Unknown → Rejected
Revision history for this message
Sergey Sinitsa (sin3) wrote :

This is three years old XOrg bug.
https://bugs.freedesktop.org/show_bug.cgi?id=865

Changed in xorg-server:
status: Unknown → In Progress
Revision history for this message
In , nonZero (udioron) wrote :

    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

Revision history for this message
nonZero (udioron) wrote :

Personally this is a big problem for me - not allowing me to use alt-shift-tab while my change group key is alt-shift.
I belive this problem would be a major issue as more dual-language users are added to the gang.

nonZero

Revision history for this message
In , Alon Bar-Lev (alon-barlev) wrote :

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... :)

Revision history for this message
In , Daniel Stone (daniels) wrote :

*** Bug 12085 has been marked as a duplicate of this bug. ***

Revision history for this message
In , tsdh (tassilo) wrote :

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.

Revision history for this message
In , Zukoff (zukoff) wrote :

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.

Revision history for this message
In , Alon Bar-Lev (alon-barlev) wrote :

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!

Revision history for this message
In , Peter Hutterer (peter-hutterer) wrote :

*** Bug 16041 has been marked as a duplicate of this bug. ***

Revision history for this message
In , Maciej Pilichowski (bluedzins) 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

Revision history for this message
In , Daniel Stone (daniels) wrote :

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.

Revision history for this message
In , Maciej Pilichowski (bluedzins) wrote :

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)

Revision history for this message
Igor Katson (descentspb) wrote :

I can add that this happens because of GNOME also, not jsut X.org. If i set "alt-shift" in xorg.conf for layout change, and for "ctrl-shift" in gnome, the "ctrl-shift-u" or "ctrl-shift-s" etc. shortcuts DO NOT work. It just changes the layout. So it's not X.org.

Revision history for this message
In , Alon Bar-Lev (alon-barlev) wrote :

Hello,
Is there any news for this one?
It is very annoying issue.
Thanks!

Revision history for this message
In , Daniel Stone (daniels) wrote :

*** Bug 18333 has been marked as a duplicate of this bug. ***

Revision history for this message
In , Mike Ponomarenko (mponomarenko) wrote :

this is really annoying bug to me. I can't use Eclipse IDE for programming, because all hotkeys use either ctrl+shift+something or ant+shift. In other words instead of refactoring I get russian letters which replace my code.

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?

Revision history for this message
In , Peter Hutterer (peter-hutterer) wrote :

xserver/xkb/xkbActions.c is where the action handling happens. There is little
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 ProcessKeyboardEvent, it's the first function
called when a keyboard event occurs. Anthing before doesn't matter to you.

Revision history for this message
In , Alexander Kojevnikov (alexk) wrote :

Does someone work on this? It's a very annoying bug, a lot of Emacs shortcuts don't work because of it.

Revision history for this message
In , Peter Hutterer (peter-hutterer) wrote :

> --- 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.

Revision history for this message
In , Eugene (lisitsky) wrote :

Can we solve this problem?
I've made some digg about key codes, may be this can be helpful?
https://bugs.launchpad.net/ubuntu/+source/xorg-server/+bug/292260/comments/6
Does anybody know: Russian distros like AltLinux or AspLinux have this poblem? Perhaps we can take some code from there?

Revision history for this message
In , Adam Purkrt (adam-purkrt-deactivatedaccount) wrote :
Download full text (5.3 KiB)

--- 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-key-press-release sequence"

Definition: A "multi-key-press-release sequence" (MKPR seq. for short) is a sequence of key presses and key releases occuring on a keyboard. It begins and ends with all keys on keyboard up. During the sequence, there must always be at least one key down.

--- 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 ...

Read more...

Revision history for this message
In , Adam Purkrt (adam-purkrt-deactivatedaccount) wrote :

another good article about XKB

http://pascal.tsu.ru/en/xkb/setup.html

Revision history for this message
In , nonZero (udioron) wrote :

(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
---- ----------------------------- ----
                                      0
1 Alt pressed 0
2 Shift Pressed - match! 1
3 Shift released - EVENT FIRED 0
4 Alt released 0

Example B:
Step Action Flag
---- ----------------------------- ----
                                      0
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
---- ----------------------------- ----
                                      0
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
---- ----------------------------- ----
                                      0
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

Revision history for this message
In , Adam Purkrt (adam-purkrt-deactivatedaccount) wrote :
Download full text (6.2 KiB)

> 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,app2,app1) one would need to hit tab twice to get back to app4 (assuming wrap arround). But with recency awareness, it's just "down Alt + hit Tab + up Alt" et voila.

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...

Read more...

Revision history for this message
In , Vasa Maximov (mcv-geek) wrote :

(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.

Revision history for this message
In , smwed (smwed-ya) wrote :

Five years the problem remains not solved.

Developers have no problems with switching of keyboard layouts and hotkeys?

(Sorry for my english)

Revision history for this message
In , Eugene (lisitsky) wrote :

(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?

Revision history for this message
In , smwed (smwed-ya) wrote :

www.xneur.ru - switch layouts on key release!
No problem for hotkeys!

xneur - daemon
gxneur - gui for gnome
kxneur - gui on QT for KDE

Revision history for this message
In , Mahaniok (mahaniok) wrote :

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.

Revision history for this message
In , smwed (smwed-ya) wrote :

(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)

Revision history for this message
In , Mahaniok (mahaniok) wrote :

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.

Revision history for this message
In , smwed (smwed-ya) wrote :

(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.

Revision history for this message
In , Oded Arbel (oded-geek) wrote :

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).

Revision history for this message
In , anarsoul (anarsoul) wrote :

(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 :)

Revision history for this message
In , Oded Arbel (oded-geek) wrote :

(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.

Revision history for this message
In , anarsoul (anarsoul) wrote :

(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.

Revision history for this message
In , Oded Arbel (oded-geek) wrote :

(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.

Revision history for this message
In , Nick Andrik (andrikos) wrote :

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_event_type) which tells us if the previous event was a "press" one which should get updated on every event
- 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-1.7.3.902) but I could not find the ProcessKeyboardEvent function mentioned on comment: #20.
Any info on this? I could try to create a patch for this but I need the correct pointers on where to look.

Revision history for this message
In , anarsoul (anarsoul) wrote :

(In reply to comment #41)
> I tried to take a look in the xkbActions.c file (xorg-server-1.7.3.902) but I
> could not find the ProcessKeyboardEvent function mentioned on comment: #20.
> 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

Revision history for this message
Oded Arbel (oded-geek) wrote :

Re #8: When you set layout changing options in GNOME it sets the X.org XKB options - just like setting it directly in xorg.conf. This should be solved in X.org. The Freedesktop.org bugzilla has an interesting discussion about it, I hope it will eventually lead to a solution though at this point it doesn't look like there are any developer resources committed to it.

Revision history for this message
In , James H. Cloos Jr. (cloos-jhcloos) wrote :

> 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_toggle), LCtrl+RCtrl does next and RCtrl+LCtrl does prev.

Revision history for this message
In , Oded Arbel (oded-geek) wrote :

(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_toggle), LCtrl+RCtrl does next and RCtrl+LCtrl does prev.
>

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.

Revision history for this message
In , Ilya Murav'jov (muravev) wrote :

Created an attachment (id=33142)
This patch is to resolve the issue; it is for "trunk" Xorg sources.

Please try to patch Xorg from git://anongit.freedesktop.org/git/xorg/xserver
and comment if the patch works for you.

Revision history for this message
In , Nick Andrik (andrikos) wrote :

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-key-pressing
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://bugs.launchpad.net/xorg-server/+bug/36812
until it is officially included in an xserver-xorg release?

Many many thanks!
Nick

Revision history for this message
In , Nick Andrik (andrikos) wrote :

Just a note, karmic's version is 1.6.4

Revision history for this message
In , Ilya Murav'jov (muravev) wrote :

(In reply to comment #46)
> Hello Ilya,
>
> Is it ok for you if I forward the patch also to the ubuntu bug here:
> https://bugs.launchpad.net/xorg-server/+bug/36812
> 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 ...

Revision history for this message
In , Nick Andrik (andrikos) wrote :

> 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!

Revision history for this message
In , Ilya Murav'jov (muravev) wrote :

(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.

Revision history for this message
Nick Andrik (andrikos) wrote :

Ilya Murav'jov from http://bugs.freedesktop.org/show_bug.cgi?id=865 has posted the attached patch to resolve this issue.

It indeed considers the key combinations on release and not on press.
I have checked it with combinations of Alt, Shit andTab where :
Alt+Tab : next window
Alt+Shift+Tab: previous window
Alt+Shift: change keyboard layout

I have checked it against the karmic version (1.6.4-2ubuntu4.1) and it works.
The patch version can be found in my PPA here:
https://launchpad.net/~andrikos/+archive/ppa

Could someone test it against the version on lucid and update the package?

PS: Ilya has told me that the "switch on release" is hard-coded and cannot be turned off.
Is this a problem? Would there be people preferring to keep the previous "switch on press" behavior?

tags: added: patch
Revision history for this message
Oded Arbel (oded-geek) wrote :

I also just finished patching current Ubuntu 9.10 xorg (2:1.6.4-2ubuntu4.1) and it works perfectly. The patch applied cleanly with no issues, and after that my XKB shortcut (ALT+SHIFT) switches only on release and ALT+SHIFT based application shortcuts work as expected.

I'll report on my success with Lucid next.

Revision history for this message
Oded Arbel (oded-geek) wrote :

On Lucid, xserver-xorg 1.7.3.902-1ubuntu12.1, the patch applies almost cleanly - Hunk #2 needs an offset of 1 line.

I have uploaded the package to my PPA at https://launchpad.net/~oded-geek/+archive/xorg-patches .

Revision history for this message
In , Oded Arbel (oded-geek) wrote :

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.

Revision history for this message
David Fraser (davidf) wrote :

I've tried the patch and it allows me to use my Ctrl-Shift shortcuts, but the SCIM dialog still pops up when I release one of Ctrl or Shift despite having pressed keys in between. Is this expected? How can I find where the Ctrl-Shift mapping is defined (it's not in GNOME shortcuts or SCIM shortcuts AFAICS)

Revision history for this message
In , Alon Bar-Lev (alon-barlev) wrote :

WORKING!
xorg-server-1.6.5

Thank you so much Ilya!

Revision history for this message
Oded Arbel (oded-geek) wrote :

David - when I set the keyboard layout switching shortcut to CTRL+SHIFT (instead of what I normally use) then that shortcut never triggers SCIM and only changes the standard system layout.

Revision history for this message
Ilya Murav'jov (muravjov-il) wrote :

David,
How to reproduce your situation, what DE? I have never used SCIM dialogs, how to turn them on?

Revision history for this message
David Fraser (davidf) wrote :

Ah, my problem was misunderstanding the SCIM config

See http://www.scim-im.org/forums#nabble-td4554222|a4554222

Basically I found that in the SCIM Global Setup I had Shift-Control-KeyRelease-Shift_L, Shift-Control-KeyRelease-Shift_R, Shift-Control-KeyRelease-Control_L, and Shift-Control-KeyRelease-Control_R defined. Removing these stops the Control-Shift trigger.
But I missed this even looking at the SCIM dialog multiple times because it looks more complex than a Shift-Control keypress

So although the Control-Shift in scim is incompatible with Ctrl-Shift-* shortcuts, I can disable it, so this isn't a problem for me know

Revision history for this message
Oded Arbel (oded-geek) wrote :

If I may expand on it a bit, with SCIM on and XKB layout switching set to CTRL+SHIFT, there are 3 contenders for CTRL+SHIFT, and they supersede each other in this order:
1. XKB
2. SCIM
3. Application's CTRL+SHIFT-<key> shortcuts

The patch from FreeDesktop.org bug #865 fixes it so that 1 (XKB) no longer blocks 3, but it still blocks 2 because CTRL+SHIFT release is eaten by XKB and SCIM doesn't see it. I think its OK as you wouldn't want to assign the same shortcut key to both keyboard layout and SCIM.

Revision history for this message
Ilya Murav'jov (muravjov-il) wrote :

Oded,
No, my patch doesn't block anything on release: keyboard layout is switched *along* with SCIM triggering (for David Fraser).
So I assume everything is ok untill David states clearly that he has troubles with the patch.

Revision history for this message
In , Mike Ponomarenko (mponomarenko) wrote :

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!

Revision history for this message
In , Eugene (lisitsky) wrote :

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?

Revision history for this message
In , Nick Andrik (andrikos) wrote :

(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://bugs.launchpad.net/xorg-server/+bug/36812

Revision history for this message
Nick Andrik (andrikos) wrote :

Any chance to see this patch in for lucid?

Revision history for this message
Ilya Murav'jov (muravjov-il) wrote :

Nick, I talked with Peter Hutterer, maintainer of X Server, a few days ago and he promised to review the patch (and to apply it to upstream if it's good).

P.S. He can be found on Freenode IRC, #xorg-devel, as whot. I hope this is not a private information. :)

Revision history for this message
Nick Andrik (andrikos) wrote :

Many thanks for your work and info Ilya :D

Revision history for this message
Evgeny Kolesnikov (evgenyz) wrote :

Oh! I owe you beer, Il'usha, surely! And everybody else who will help this patch approval for Lycid.

Revision history for this message
Misha Bazanov (bmw-) wrote :

I suggested this bug for one hundred papercuts because that already have a patch and bug is annoying.

Revision history for this message
Eugene (lisitsky) wrote : Re: [Bug 36812] Re: Keyboard layout change on hotkeys press instead of release and do not work well with shortcuts

Очень надеюсь, что это окончательное решение этого застарелого бага.
Хотя я видел сообщения, что у кого-то не заработало.

2010/3/14 Misha Bazanov <email address hidden>

> I suggested this bug for one hundred papercuts because that already have
> a patch and bug is annoying.
>
> --
> Keyboard layout change on hotkeys press instead of release and do not work
> well with shortcuts
> https://bugs.launchpad.net/bugs/36812
> You received this bug notification because you are a direct subscriber
> of a duplicate bug.
>

--
Yours,
Eugene Lisitsky

Revision history for this message
In , Ori Avtalion (salty-horse) wrote :

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

Revision history for this message
In , Ilya Murav'jov (muravev) wrote :

(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?

Revision history for this message
In , Ori Avtalion (salty-horse) wrote :

(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-keybinding-properties. I thought its shortcomings were due to xkb, but apparently I was wrong :)

Revision history for this message
In , Daniel Stone (daniels) wrote :

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://bugs.freedesktop.org/attachment.cgi?id=33142)
> This patch is to resolve the issue; it is for "trunk" Xorg sources.
>
> Please try to patch Xorg from git://anongit.freedesktop.org/git/xorg/xserver
> 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.

Revision history for this message
In , Paulo Zanoni (pzanoni) wrote :

Ping? No new patch versions? Will this be applied?

Revision history for this message
In , Alexander Pavlov (saaska) wrote :

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.

Revision history for this message
In , Ilya Murav'jov (muravev) wrote :

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ЯzяZяzяzяzяzяZя

Not good ...

Revision history for this message
In , Ethan Shalev (shalev-ethan) wrote :

(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ЯzяZяzяzя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.

Revision history for this message
In , Ilya Murav'jov (muravev) wrote :

(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ЯzяZяzяzя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-keyboard-properties -> Layouts -> Layout Options). Wait a minute ... Ahhhh, things go wrong if "Use keyboard LED to show alternative layout"->ScrollLock is on! That is why I couldn't reproduce it on my notebook (it doesn't have keyboard LEDs). Mmm, it is much better now :)

As for the patch, you can use Oded Arbel' repository to try it without building
Xorg yourself, http://launchpad.net/~oded-geek/+archive/xorg-patches .

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 :(

Revision history for this message
In , Vasa Maximov (mcv-geek) wrote :

Ilya, SPASIBO for the patch and hint with http://launchpad.net/~oded-geek/+archive/xorg-patches

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!

Revision history for this message
Alexander Pavlov (saaska) wrote :

I talked to the people from #xorg-devel on FreeNode today about the status of Ilya's patch

<daniels> i'm inclined to not apply it, as it directly contradicts the xkb spec, and is also apparently buggy (see the upstream bug report with some people complaining that it breaks occasionally)
<saaska> and who makes the xkb spec?
<daniels> well, us. we could change it, but there's not much point in a spec which changes ...
<saaska> mm and what would cause you to look further into this?
<daniels> well, we could write the spec better for xkb2 ;)
<daniels> yeah, i'm not personally affected by it as i only use US layout
<daniels> i can see that it sucks, but otoh i don't like contradicting the spec for no good reason
<saaska> you don't notice it if you don't switch languages
<daniels> well, i notice all the affected users in my inbox :P
<saaska> i'd even go as far as to say it's a major annoyance with layout-switching people
<daniels> understandable

To me this looks like it's not gonna be fixed any time soon. I hope xkb2 spec is somewhere around the corner. Oh well...

Revision history for this message
Ilya Murav'jov (muravjov-il) wrote :

Hi, Alexander!

> <daniels> i'm inclined to not apply it, as it directly contradicts the xkb spec, and is also apparently
> buggy (see the upstream bug report with some people complaining that it breaks occasionally)

The patch is not buggy, the only issue complained was my own (see more at http://bugs.freedesktop.org/show_bug.cgi?id=865 ).
And the reason is known - LED's highlighting confilcts with the patch, 1) it can be safely avoided, 2) it is just a little tech problem which can be done.

> <daniels> well, we could write the spec better for xkb2 ;)
> <daniels> yeah, i'm not personally affected by it as i only use US layout
> <daniels> i can see that it sucks, but otoh i don't like contradicting the spec for no good reason
I start to think it is a discrimination.

No xkb2 writing is required! My first suggestion was to add my patch as configuration option; so the default installation of Xorg works as expected, and there is no spec contradiction. If a user wishes he(she) turned that option on/off as one of many others you can see in Keyboard Preferences.

xkb2 waiting is just 'when pigs fly' excuse.

Revision history for this message
Roman Vorobets (roman-vorobets) wrote :

I think enabling multilingual people to use both keyboard shortcuts and layout switching actually IS a good reason to contradict the spec. Please improve the patch and apply it!

Revision history for this message
Alexey Kozlov (alexk13) wrote :

First of all, thanks to all of you guys for your efforts and especially to Ilya fot the patch.
I'm also affected and extremely annoyed by this bug. I've moved to Linux recently and first mentioned, that Ctrl+Shift+T doesn't work in Firefox, and then it turned out, that I can't use Eclipse properly since there are many vital Ctrl+Shift shortcuts there.

And I'd like to be explicit here - IMO it's just a shame that such a STUPID bug is still not fixed after SIX years (it was reported back in 2004!). OK, I understand, it's free software and nobody gets paid for fixing bugs, but denying existing patch for some kind of puristic (not to say religious) reasons - it seems to me, well, strange. How can we claim that Ubuntu (in particular) is user-friendly and "for people" and at the same time preserve such annoying bugs? (another example I've ran into - mystical restriction to have max. of 4 KB layouts in XKB - but at least it's not so common. btw excuse is the same - "the holy spec").

I'm not sure how it aligns with existing procedures, but wouldn't it be possible to apply this patch during packaging Xorg for Ubuntu, so that Xorg packages from standard repos (and their future updates) include this fix?

After all, this bug really affects a lot of people, since there are users outside US as well and almost all of them use layout switching.

Revision history for this message
Oded Arbel (oded-geek) wrote :

It is quite possible to package the patch just for Ubuntu. I'm actually maintaining such a build in my ppa at https://launchpad.net/~oded-geek/+archive/xorg-patches.

All ubuntu devs should do is just pick up the diff from my ppa and apply it as an update to 10.04 or 9.10. I will also deliver a patched XOrg for 10.10 shortly.

Revision history for this message
Alexey Kozlov (alexk13) wrote :

Oded,

thanks, I saw patched packages in your ppa and it's great. But my question was, are there any ways to convince Ubuntu maintainers to include them in standard repos?
(more precisely, to apply Ilya's patch to each new Xorg version and put this patched version in standard repo).

I suppose, the reasons are clear: 1) new users get fixed version out-of-the-box 2) no need to reinstall package from ppa after each Xorg update (it's not just a question of 10.04/10.10, they also produce intermediate bug-fix releases of Xorg, aren't they?)

But the whole thing seems to me a little hacky, that's why I ask about procedures,

Revision history for this message
Oded Arbel (oded-geek) wrote :

@Alexey: Regarding your point (2), if I do a well enough job and release an update to the PPA immediately after Ubuntu upstream releases an X update, then users will only have to install 1 update, or at worst they'd have 1 X update that requires them to logout and another X update a day later that again requires a log out.

Regarding point (1) - Ubuntu's X11 distribution carries a lot of out of stream patches (in 10.04 I count more then 50 additional patches) that are either in process of being accepted upstream to x.org or intermediate fixed waiting for another fix from upstream or even patches that have no chance of being accepted upstream - and Ubuntu still caries them and make sure they are up to date until they have a better solution from X.org.

To get Ilya's patch into Ubuntu proper, it requires simply the intervention of Ubuntu's "X-SWAT" team ( https://launchpad.net/~ubuntu-x-swat ) and that most likely will not require a rewrite of XKB ;-) but I don't see any of these people list on this bug report, and I'm not sure what is the procedure for addressing the Ubuntu X team directly.

Revision history for this message
Nick Andrik (andrikos) wrote :

Hello Oded,

I think it is a good idea to subscribe the team in this bug report in order to have their opinion.
And, btw, we should probably move the bug to the appropriate package (it is not in the control-center it is in xorg)

Nick

Revision history for this message
Alexey Kozlov (alexk13) wrote :

Oded,

thanks for your explanations. Hope X-SWAT team will add this patch to Ubuntu X11 distro.
Is it enough to just subscribe them or should we attract their attention in some other way? :)

Revision history for this message
Nick Andrik (andrikos) wrote : Re: [Bug 36812] Re: Keyboard layout change on hotkeys press instead of release and do not work well with shortcuts

I have already subscribed the x-swat team for now.
We wait a bit to see if someone responds and if not we can also email them
to see if they think that accepting Ilya's patch in the official ubuntu
version is a good idea.

Nick

Revision history for this message
Alexey Kozlov (alexk13) wrote :

Ok, thanks a lot!

Revision history for this message
Alexander Pavlov (saaska) wrote :

Let just me stress the importance of this bug once more, cause English-speaking users may not readily appreciate it.
A large part of non-English people use at least two keyboard layouts: one for their native language and one for Latin script: web addresses, programming, whatever. A common way to switch layouts (wherever it comes from) is pressing Ctrl+Shift or Alt+Shift. If you use any of these in Ubuntu, you lose it for all shorcuts. Ctrl-shift arrows don't work, etc etc.
We are talking about huge productivity impairment - the loss of keyboard shortcuts - in all apps systemwide, for whole countries.

Revision history for this message
Alexey Kozlov (alexk13) wrote :

So, still no reaction from x-swat after almost a week. Should we send them email and ask directly?

Revision history for this message
Nick Andrik (andrikos) wrote :

Alexey, could you please send them an email and CC also this bug ( Bug 36812 <email address hidden> )?
Like this we can get their response here.

Thanks,
Nick

Revision history for this message
Alexey Kozlov (alexk13) wrote :

Nick, should I send e-mail to their maillist ubuntu-x-swat(at)lists.launchpad ?

Revision history for this message
In , Bas-bmail (bas-bmail) wrote :

Applied the patch to xorg-server-1.7.7 (Mandriva 2010.1). Works as a charm. Looking forward to having this patch upstream.

Revision history for this message
Alexey Kozlov (alexk13) wrote : Bug 36812: Applying XKB patch to Ubuntu XOrg package

Hello X-SWAT team!

I would like to draw your attention to one old and annoying XKB bug,
which has been fixed by a patch few months ago. Although this patch
seems to provide a reasonable solution for the problem, and several
people reported it worked for them without any side-effects, it wasn't
accepted upstream. The answer from Xorg developer was, that such a
behavior would contradict the spec and the issue probably will be fixed
with a new spec for XKB2. Unfortunately, the latter sounds like "when
pigs fly"... It's also worth mentioning, that corresponding bug in Xorg
bugzilla was filed 6 (!) years ago, with no notable progress until
recently, and even now when patch is available it has been not applied
upstream.

Now brief summary of the problem.

Issue: If KB layout switching is used, any keyboard shortcuts involving
keys from key combination used for switching don't work any more.
Example: If Ctrl+Shift used as layout switching shortcut (most common
case), then Ctrl+Shift+T shortcut doesn't work in Firefox, as well as
other Ctrl+Shift+<Key> shortcuts in all other programs.

Cause: It turned out, that layout switching occurs on KEY DOWN event,
thus preventing all further keys in a shortcut from being processed.

Fix: The patch provides simple and obvious solution: switching occurs on
KEY RELEASE event, which means, it will be fired only if eg Ctrl+Shift
ALONE were pressed.

Importance: It could happen, that Xorg developers who themselves use
only 1 US layout, just do not understand how serious the problem is.
Because of this bug, quite wide range of shortcuts is not accessible in
ALL applications system-wide. And since almost all people outside
English-speaking countries use multiple layouts (national + English for
web addresses, programming etc), the number of affected users is also
significant.
What makes things even worse, is the fact, that it's not quite a priori
obvious, that layout switching causes the problem. So typically user
notices, that some shortcuts are not working in some particular
application, and will be searching for solution for this specific
problem - and will most probably find nothing. So did I, googled without
any success, and only after some experiments later I've figured out,
that layout switching is actually to blame.
So it's a big time-loss and annoyance for all newcomers, who need to
reinvent the wheel by themselves.

That's why discussing this bug on Launchpad, we came to the following
suggestion: Would it be possible to include this patch in standard
Ubuntu Xorg distribution, thus saving all international users much
efforts?
It would definitely make Ubuntu distro even more user-friendly :)

You can find full discussion and further details as well as the patch
itself here:

https://bugs.launchpad.net/xorg-server/+bug/36812

Bug filed in Freedesktop bugzilla:

http://bugs.freedesktop.org/show_bug.cgi?id=865

Many thanks in advance and hope for your collaboration.

Best regards,

Alexey Kozlov

Changed in xorg-server:
importance: Unknown → Medium
Changed in gnome-control-center:
importance: Unknown → Medium
status: Invalid → Unknown
Revision history for this message
Nick Andrik (andrikos) wrote :

I believe this bug falls in the papercut category and I have nominated as such.

It satisfies all the requirements for a papercut:
    * A bug, or an unintended problem occurring within an existing piece of software, (Xorg)
    * the presence of which makes a computer more difficult or less pleasant to use, (international users cannot use the desired shortcuts)
    * that is easy to fix, (patch already available)
    * that the average user would encounter... (insternational users with multiple keyboard layouts)
    * in a default installation of Latest release of Ubuntu or Kubuntu, Desktop Edition. (Xorg is installed by default and this bug is already many years old)

Please consider applying Ilya's patch attached in this bug report.

Revision history for this message
Vadim Dmitriev (allati) wrote :

Yesterday I upgraded to maverick and today experienced layout switching bug. Not sure if it relates to the original xserver because I immediately installed patched version kindly provided by Oded Arbel.
However, here is how it looks like: at a random moment when I switch layout gnome-settings-daemon spikes to 100%, active layout is 'stuck' (i.e. it does no longer switch) and when I press and hold a key a random sequence of two characters is printed (chars from two installed layouts mapped to the same key). My second layout is cyrillic, so if I hold down "ф", printed sequence looks like "фaфффaaaaфaaaффaфaффффaaaa".
I haven't found any guaranteed way to trigger such a behavior, but is does happen several times a day at moments when I switch layouts actively (1-2 times a second).
Have anybody else faced something like this with the original xserver and/or patched version?

Revision history for this message
Oded Arbel (oded-geek) wrote :

I have seen this behavior as well in the patched X-Server, and I may have also seen this with the original Maverick X-Server - though I'm not sure about it.

I get this bug triggered only by using Synergy, but with almost 100% consistency: I use Ubuntu 10.04 as a Synergy server, and 10.10 is the client. If I move my mouse to the 10.10 screen and try to switch languages by using ALT-SHIFT on the server keyboard, I get the faulty behavior.

What happens is that gnome-settings-daemon - which is normally responsible for switching layouts when you switch applications - starts switching through all the layouts uncontrollably as if the switching keys are stuck, and takes up 100% CPU. If you start typing anywhere, you'd get characters from all layouts mixed together. At this point, if you try to use the keyboard layout indicator in the session applet, you'd see that its stuck and does not respond to requests to set the layout. Finally, sending TERM to gnome-settings-daemon will not stop it and you must use the KILL signal (-9).

I believe this is a gnome-settings-daemon, mostly as I can also cause gnome-settings-daemon to spike to 100% CPU usage simply by sending it a TERM signal while working normally. At which point the keyboard layout indicator is again stuck and does not respond to actions, but using the layout switching keys works normally and there is no cycling through the layouts.

Revision history for this message
Nick Andrik (andrikos) wrote :

I guess it is this bug:
https://bugs.launchpad.net/ubuntu/maverick/+source/gnome-settings-daemon/+bug/625793

As it seems, gnome-settings-daemon is to blame and not Oded's patch.

Is there any other annoyance with the proposed patch?
I hope we can get it in the official package some time soon, instead of having to rely to parallel unofficial repositories.

Nick

Revision history for this message
Nick Andrik (andrikos) wrote :

Small correction above: the patch is from Ilya, not from Oded

Revision history for this message
In , Nick Andrik (andrikos) wrote :

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

Revision history for this message
In , Ilya Murav'jov (muravev) wrote :

(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.

Revision history for this message
In , Vasa Maximov (mcv-geek) wrote :

(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.

Revision history for this message
In , Nick Andrik (andrikos) wrote :

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)

Revision history for this message
In , Nick Andrik (andrikos) wrote :

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

Revision history for this message
In , Nick Andrik (andrikos) wrote :

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

Revision history for this message
In , Alexander Kojevnikov (alexk) wrote :

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!

Revision history for this message
Azamat S. Kalimoulline (turtle-bazon) wrote :

Using patched xorg from Oded ppa fixes my problem. Keboard layout changes on release and works like a charm. No side-effects affects to me. When this patch will be included in official source tree and when it will be available in repositories?

Revision history for this message
AlexGenaud (alexgenaud) wrote :

(me too) confirm Alt-Shift keyboard layout switch incompatible with Alt-Shift-Tab Window switching in reverse in Ubuntu 10.10...

This bug is set to Low/Medium priority, but what are the recommended work-arounds? Alt-Shift is an obvious keyboard layout switch because it is default in Windows. Alt-Space is default on Mac OS X (I think) but that is incompatible with Gnome window menus. Various other combinations conflict (like Alt-Gr/Right Alt is nec. for many non-English layouts). How are polyglot Ubuntu users switching keyboard layouts?

Revision history for this message
Alexey Kozlov (alexk13) wrote :

Azamat S. Kalimoulline, we all here are interested, how many more years will it take, for this at last to happen.

AlexGenaud, I suppose Ctrl+Shift is a good shortuct (actually I used it in Windows as well, cause it seems to me more convenient as default Alt+Shift)

Revision history for this message
Rocko (rockorequin) wrote :

I use Shift+Caps Lock. Ctrl-Shift is a bit too easy for me to hit by accident.

Revision history for this message
Sense Egbert Hofstede (sense) wrote :

Thank you for taking the time to report this issue. However, I'm declining this as a paper cut because this bug is not trivially fixable. A paper cut is a small usability problem in default and featured applications that affect many users.

Please note that I'm marking this as Invalid for the One Hundred Paper Cuts project ONLY.
I've opened a new task for Xorg in Ubuntu—which was missing—that makes this bug show up on the radar of the people whose task it is to look after X. Previously this tasked lacked, so the Ubuntu X developers couldn't have known this bug existed. A task for xorg should have been added when this bug was marked as Invalid for control-center.

affects: hundredpapercuts → null
Changed in null:
status: New → Invalid
Nick Andrik (andrikos)
description: updated
description: updated
description: updated
description: updated
description: updated
bugbot (bugbot)
affects: xorg (Ubuntu) → xserver-xorg-input-evdev (Ubuntu)
Revision history for this message
In , Dave Walker (davewalker) wrote :

Hi,

Is there an update on the status of this bug please?

Kind Regards,
Dave Walker

Revision history for this message
Dave Walker (davewalker) wrote :

Comment, asked for an update on the upstream bug tracker.

Revision history for this message
In , Simos Xenitellis  (simosx) wrote :

(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/deactivate/switch layout in IBus.
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?

Revision history for this message
In , Bas-bmail (bas-bmail) wrote :

(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/deactivate/switch layout in IBus.
> 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.

Revision history for this message
In , Alexander Kojevnikov (alexk) wrote :

(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/deactivate/switch layout in IBus.

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.

Revision history for this message
In , Oded Arbel (oded-geek) wrote :

(In reply to comment #75)
> Things to test would include
> 3. Is there any problem when using IBus? Check the shortcuts that
> activate/deactivate/switch layout in IBus.

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.

Changed in control-center (Ubuntu):
assignee: Ubuntu Desktop Bugs (desktop-bugs) → nobody
Changed in xserver-xorg-input-evdev (Ubuntu):
status: New → Confirmed
Revision history for this message
Bryce Harrington (bryce) wrote :

@Alexey Kozlov, thanks for your well written summary of the problem.

The one warning flag is the "contradicts spec" issue (which may be what has blocked this from being taken up 'til now), however the exact words from upstream about this seem less dire:

 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.

I can understand that voiding the spec is a blocker for upstream, but at the distro level there's a little bit more flexibility, especially given the top-level direction that we'll eventually move away from X11 towards something Wayland-ish anyway.

It sounds like the next step required for this patch is to gain some more widespread testing. Since the change is behavioral, it seems low risk for causing crashes or other critical issues, so tentatively I'm willing to stick it in natty and see how it goes.

Changed in xserver-xorg-input-evdev (Ubuntu):
importance: Undecided → Wishlist
status: Confirmed → Fix Committed
Revision history for this message
In , Ilya Murav'jov (muravev) wrote :

The patch has been applied in Ubuntu, just now.
Today is a good day. :)

Revision history for this message
Timo Aaltonen (tjaalton) wrote :

uploaded to natty

affects: xserver-xorg-input-evdev (Ubuntu) → xorg-server (Ubuntu)
Changed in xorg-server (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Alexey Kozlov (alexk13) wrote :

@Bryce Harrington, thanks a lot, we all here have been waiting for this day to come! :)

Speaking about testing, here is one more feedback from me:

I'm using my maverick with this patch applied for a couple of months now. Shortcuts are working as they should in Firefox, Eclipse and Gnome terminal, no side effects observed.

Revision history for this message
Nick Andrik (andrikos) wrote :

Many many thanks!!
It is indeed a long awaited day :D

Revision history for this message
Sergey Sinitsa (sin3) wrote :

Hi.

I reported this bug here five years ago, gathered the info in 2006-2007 and watched the logs all those years. I really happy it is fixed in Ubuntu. I even wrote a historical review of this issue in my blog:

http://rulinux.net.ru/node/462

I want to say huge thanks to Ilya Murav'jov, Alexey Kozlov, Oded Arbel, Daniel Stone, Nick Andrik, Bryce Harrington, Timo Aaltonen and many many other users and developers involved to fix it!

Revision history for this message
In , Oded Arbel (oded-geek) wrote :

The following comments were made by Peter Hutterer (an X.org input developer) on the corresponding bug in RedHat bugzilla ( https://bugzilla.redhat.com/show_bug.cgi?id=660254 ):
(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 :-)

Revision history for this message
In , Ilya Murav'jov (muravev) wrote :

(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://bugzilla.redhat.com/show_bug.cgi?id=660254 ):
> (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 _XkbNextFreeFilter()/_XkbApplyFilters() ). Btw, that trick was suggested by Daniel Stone (somewhere on the mail list).

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.

Changed in xorg-server:
importance: Medium → Unknown
Revision history for this message
Oded Arbel (oded-geek) wrote :

@Bryce Harrington: This feature was apparently dropped from the latest Natty release ( 2:1.9.99.901+git20110131.be3be758-0ubuntu1 - http://changelogs.ubuntu.com/changelogs/pool/main/x/xorg-server/xorg-server_1.9.99.901+git20110131.be3be758-0ubuntu1/changelog ) by Christopher James Halse Rogers (CCed) with the following comment:

* Drop 208_switch_on_release.diff; upstream

By "upstream" I figured it meant "Debian" (the title of the change was "Merge from (unreleased) debian-experimental") and not freedesktop.org - as I know they didn't commit this fix. On the other hand, I looked everywhere and couldn't find any reference of such an upstream release from Debian - and indeed after installing the latest xorg-server, the functionality has disappeared.

As such I'm reopening this issue and I hope the patch will come back before Natty is released. Again, thanks for all the effort.

Revision history for this message
Oded Arbel (oded-geek) wrote :

Oops - apparently I can't reopen this bug. Someone with permissions - please reopen this ticket and/or resubmit the patch. Thanks in advance.

Revision history for this message
Nick Andrik (andrikos) wrote :
Changed in xorg-server:
importance: Unknown → Medium
Revision history for this message
In , T-artem (t-artem) wrote :

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?

Revision history for this message
In , Peter Hutterer (peter-hutterer) wrote :

(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.

Revision history for this message
In , T-artem (t-artem) wrote :

(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.

Revision history for this message
In , Bas-bmail (bas-bmail) wrote :

(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.

Revision history for this message
In , Daniel Stone (daniels) wrote :

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.

Revision history for this message
In , DmitryKX (alex-custov) wrote :

Can we pay to speed up the fixing of this bug?

Revision history for this message
In , Oded Arbel (oded-geek) wrote :

(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://www.x.org/releases/current/doc/kbproto/xkbproto.html 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.

Revision history for this message
In , T-artem (t-artem) wrote :

(In reply to comment #87)
> Can we pay to speed up the fixing of this bug?

I second this motion.

Changed in xorg-server:
importance: Medium → Critical
Revision history for this message
In , Ilya Murav'jov (muravev) wrote :

(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://www.x.org/releases/current/doc/kbproto/xkbproto.html
> 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.x.org/pub/X11R7.0/doc/PDF/XKBlib.pdf, "Table 16.4 Group Action Types", the last item XkbSA_LockGroup, citing:

"
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.
"

Revision history for this message
In , Michal Ambroz (rebus) wrote :

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

Revision history for this message
In , N3ocort3x (n3ocort3x) wrote :

For Gentoo and derivatives users, a working solution for patching is described here:

https://bugs.gentoo.org/show_bug.cgi?id=379827

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 :)

Revision history for this message
In , Alon Bar-Lev (alon-barlev) wrote :

Long ago a bug was opened for this issue.

https://bugs.gentoo.org/show_bug.cgi?id=304375

On Mon, Aug 22, 2011 at 5:14 PM, <email address hidden> wrote:
> https://bugs.freedesktop.org/show_bug.cgi?id=865
>
> --- 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://bugs.gentoo.org/show_bug.cgi?id=379827
>
> 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://bugs.freedesktop.org/userprefs.cgi?tab=email
> ------- You are receiving this mail because: -------
> You are on the CC list for the bug.
>

Revision history for this message
In , N3ocort3x (n3ocort3x) wrote :

(In reply to comment #93)
> Long ago a bug was opened for this issue.
>
> https://bugs.gentoo.org/show_bug.cgi?id=304375
>

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

Revision history for this message
In , Jeremy Sequoia (jeremyhu) wrote :

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?

Changed in xorg-server:
importance: Critical → High
Curtis Hovey (sinzui)
no longer affects: null
Revision history for this message
In , Aharel (aharel) wrote :

(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.x.org/pub/X11R7.0/doc/PDF/XKBlib.pdf, "Table 16.4 Group Action
> 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?

Revision history for this message
Constantine (theaspect) wrote :

Scroll-Lock indicator always shows correct layout

Revision history for this message
In , Jeremy Sequoia (jeremyhu) wrote :

Deferring to 1.13 as this is functional change

Revision history for this message
In , Bas-bmail (bas-bmail) wrote :

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?

Revision history for this message
In , Bas-bmail (bas-bmail) wrote :

Sorry, it was my overlook. THe patch works just fine, just needed more attention adapting to latest sources.

Revision history for this message
In , Bas-bmail (bas-bmail) wrote :

Created attachment 59741
The same patch, but based on 1.12.0.902

Revision history for this message
In , Alex Efros (powerman-asdf) wrote :

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.

Revision history for this message
In , Alon Bar-Lev (alon-barlev) wrote :

Created attachment 63378
xorg-server-1.12.2-xkb-switch-on-release.patch

Some code style modifications.

Revision history for this message
In , Julius Schwartzenberg (jschwart) wrote :

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?

Revision history for this message
In , Oded Arbel (oded-geek) wrote :

(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.).

Revision history for this message
In , Murz (murznn) wrote :

Oded Arbel, how did you test this patch on Ubuntu? Can you give me some link to PPA or deb files for testing?

Revision history for this message
In , RussianNeuroMancer (russianneuromancer) wrote :

> 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).

Revision history for this message
In , Spitzak-k (spitzak-k) wrote :

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.

Revision history for this message
In , kitaets (chinaman) wrote :

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.

Revision history for this message
In , Oded Arbel (oded-geek) wrote :

(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>+<shift2>+<key> as a keyboard shortcut (when "shift1" and "shift2" are the shift keys used for layout switching, for example ALT+SHIFT or CTRL+ALT), then the user will inadvertently also trigger a layout switching that wasn't supposed to happen.

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).

Revision history for this message
In , Wettstein509 (wettstein509) wrote :

(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.

Revision history for this message
In , Bas-bmail (bas-bmail) wrote :

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?

Revision history for this message
In , Wettstein509 (wettstein509) wrote :

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.

Revision history for this message
In , Bas-bmail (bas-bmail) wrote :

(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.

Revision history for this message
In , Wettstein509 (wettstein509) wrote :

> 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-config/compat/iso9995, change the current definition:

  interpret ISO_Next_Group {
      useModMapMods= level1;
      virtualModifier= AltGr;
      action= LockGroup(group=+1);
  }

to

  interpret ISO_Next_Group {
      useModMapMods= level1;
      virtualModifier= AltGr;
      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.

Revision history for this message
In , kitaets (chinaman) wrote :

(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.

Revision history for this message
In , Ilya Murav'jov (muravev) wrote :

(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.

Revision history for this message
In , Wettstein509 (wettstein509) wrote :

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",
        symbols[Group1]= [ Alt_L, Meta_L ],
 actions[Group1]= [ Private(type=3,data[0]=1,data[1]=8,data[2]=8,data[3]=0,data[4]=0,data[5]=0,data[6]=1),
                    Private(type=3,data[0]=1,data[1]=8,data[2]=8,data[3]=0,data[4]=0,data[5]=0,data[6]=1) ]
    };

Similarly, shifting group with Shift+Right Alt (where Shift is pressed first):

Revision history for this message
In , Wettstein509 (wettstein509) wrote :

Similarly, shifting group with Shift+Right Alt (where Shift is pressed first):

  key <RALT> {
        repeat= No,
        type= "TWO_LEVEL",
        symbols[Group1]= [ Alt_R, Meta_R ],
 actions[Group1]= [ SetMods(modifiers=Mod1),
                    Private(type=3,data[0]=1,data[1]=8,data[2]=8,data[3]=0,data[4]=0,data[5]=0,data[6]=1) ]
    };

Shifting group with Shift+Left Control:

    key <LCTL> {
        repeat= No,
        type= "TWO_LEVEL",
        symbols[Group1]= [ Control_L, Control_L ],
 actions[Group1]= [ SetMods(modifiers=Control),
                    Private(type=3,data[0]=1,data[1]=4,data[2]=4,data[3]=0,data[4]=0,data[5]=0,data[6]=1) ]
    };

Revision history for this message
In , Alex Efros (powerman-asdf) wrote :

(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-something hotkey.

Revision history for this message
In , Wettstein509 (wettstein509) wrote :

> 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-something hotkey.

In this case, one needs to remap the shift key. For the left shift key:

    key <LFSH> {
        repeat= No,
        type[Group1]="PC_CONTROL_LEVEL2",
        symbols[Group1]= [ Shift_L, Shift_L ],
 actions[Group1]= [ SetMods(modifiers=Shift),
                    Private(type=3,data[0]=1,data[1]=1,data[2]=1,data[3]=0,data[4]=0,data[5]=0,data[6]=1) ]
    };

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.

Revision history for this message
In , Wettstein509 (wettstein509) wrote :

Created attachment 69213
LockMods can lock another group

My previous patch does not properly account for absolute group specification. The revised patch corrects this.

Revision history for this message
In , Nick Andrik (andrikos) wrote :

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://packages.ubuntu.com/quantal/xserver-xorg-core
Select "Ubuntu Changellog" and then search for "208_switch_on_release.diff".

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.

Revision history for this message
In , Denis Zhdanov (altwazar) wrote :

I have to recompile xorg-server on ubuntu because of this patch. It may be useful for layout switching with only modifier keys, but key-on-release behavior is unreliable while typing.

Revision history for this message
In , Ilya Murav'jov (muravjov-il) wrote :

Denis,
What do you mean that "key-on-release behavior is unreliable while typing"? Please describe.

Revision history for this message
Evgeny Kolesnikov (evgenyz) wrote :

Typing so fast that keys release events happen in different order than press events? Typing with chords?

Revision history for this message
In , Denis Zhdanov (altwazar) wrote :

For example, if I press a key (letter, backspace or other) and then layout switch combination without half second pause between, then layout wont be changed.
To switch between layouts I use "caps/shift+caps" combination. It's not a "modifier only" combination, but still affected by 208_switch_on_release.diff patch. In Windows even alt+shift combination work perfectly.

My English is bad. So can I e-mail you details in Russian?

Revision history for this message
In , anatoly techtonik (techtonik) wrote :

Just my 0.02 cents. Why not to introduce "transaction" concept for key presses?

1. Transaction is finished when the last key is released
2. The returned result from the transaction is the sequence of keys pressed

This should solve all [Alt, Shift] problems.

Example:
  user presses Alt key
  user presses Shift key and releases it
  user presses Shift key
  user releases Alt key
  user releases Shift key

The returned sequence is [Alt, Shift, Shift]

Revision history for this message
Ilya Murav'jov (muravjov-il) wrote :

> My English is bad. So can I e-mail you details in Russian?
Ok, but I am far away from playing with Xorg now.

Revision history for this message
Ilya Murav'jov (muravjov-il) wrote :

> Just my 0.02 cents. Why not to introduce "transaction" concept for key presses?
You just go too far. Xorg developers don't apply the patch because _any_ such events should be sent on press, not on release. Because X specifications tell so and they will be broken down otherwise.

Revision history for this message
Ilya Murav'jov (muravjov-il) wrote :

And if X specifications are broken down then some applications may be potentionally broken down.

Revision history for this message
In , Nicola Koshak (mcmxvii) wrote :

It is xorg-server 1.14, year 2013, and the bug still exists.

Do something with it already...

Revision history for this message
In , Eugene Shalygin (eugene-shalygin) wrote :

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? :)

Revision history for this message
In , T-artem (t-artem) wrote :

(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.

Revision history for this message
In , Headcrabextra (headcrabextra) wrote :

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.

Revision history for this message
Constantine (theaspect) wrote :

Appears again in Ubuntu 13.10

Revision history for this message
Levan Chelidze (levanchelidze) wrote :

It stooped working again

Revision history for this message
Oded Arbel (oded-geek) wrote :

This issue is still solved for me - test if the fix still works by holding slowly ALT then SHIFT then some letter key, then release all one by one - if the keyboard layout has not changed, then the issue is still resolved.

Revision history for this message
Volodya (volodya) wrote :

Ubuntu 13.10 does not work. Willing to help by providing any requested information

Revision history for this message
Lisio (lisio) wrote :

Ubuntu 13.10 has exactly the same bug.

Revision history for this message
In , Igor Gnatenko (i-gnatenko-brain) wrote :

any news here?

Revision history for this message
In , Murz (murznn) wrote :

Igor Gnatenko, I partly solve my problem via KSuperKey app http://kde-apps.org/content/show.php?content=154569
so maybe it solve your needs too.

But I also waiting fix for this issue in xkb.

Revision history for this message
In , Adam Purkrt (adam-purkrt-deactivatedaccount) wrote :
Download full text (3.4 KiB)

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://wiki.archlinux.org/index.php/X_KeyBoard_extension

Also (as suggested by the author), copying info here from https://bugzilla.redhat.com/show_bug.cgi?id=660254

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...

Read more...

Revision history for this message
In , Oded Arbel (oded-geek) wrote :

(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".

Revision history for this message
In , Adam Purkrt (adam-purkrt-deactivatedaccount) wrote :

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/X11/xkb/symbols/group with parts like

partial modifier_keys
xkb_symbols "lalt_lshift_toggle" {
    virtual_modifiers Alt;
    key <LALT> {
        symbols[Group1] = [ NoSymbol, ISO_Next_Group ],
        virtualMods= Alt
    };
    key <LFSH> {
        type[Group1]="PC_ALT_LEVEL2",
        symbols[Group1] = [ Shift_L, ISO_Next_Group ]
    };
};

The following is what this part of the file would look like after appropiate changes in xkb code:

xkb_sequences "lalt_lshift_toggle" {
    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://www.x.org/docs/XKB/XKBproto.pdf need to be extended with a small chapter
2) http://cgit.freedesktop.org/xorg/xserver/tree/xkb - code the run time check for sequences; changes to xkm format required
3) http://cgit.freedesktop.org/xorg/app/xkbcomp/tree/ - code the parsing of "xkb_sequences", "keydown", and "keyup" keywords

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..

Revision history for this message
In , Wettstae (wettstae) wrote :

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.

Revision history for this message
In , Adam Purkrt (adam-purkrt-deactivatedaccount) wrote :

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>+<Shift>+<something> (first alt, then shift, then something) shortcuts work. All this without the need to recompile xserver.

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> {
          type[group1]= "TWO_LEVEL",
         type[group2]= "ONE_LEVEL",
         symbols[Group1]= [ Alt_R, Meta_R ],
         symbols[Group2]= [ ISO_Level3_Shift ]
      };
to
      key <RALT> {
          type[group1]= "TWO_LEVEL",
         type[group2]= "TWO_LEVEL",
         symbols[Group1]= [ Alt_R, ISO_Next_Group ],
         symbols[Group2]= [ ISO_Level3_Shift, ISO_Next_Group ]
      };

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.

Revision history for this message
In , Adam Purkrt (adam-purkrt-deactivatedaccount) wrote :

Perhaps easier variant of the above workaround pertaining Alt+Shift is to comment out the following (eight) lines in /usr/share/X11/xkb/symbols/group

first change (search for "lalt_lshift"):

partial modifier_keys
xkb_symbols "lalt_lshift_toggle" {
    virtual_modifiers Alt;
    key <LALT> {
        symbols[Group1] = [ NoSymbol, ISO_Next_Group ],
        virtualMods= Alt
    };
// key <LFSH> {
// type[Group1]="PC_ALT_LEVEL2",
// symbols[Group1] = [ Shift_L, ISO_Next_Group ]
// };
};

second change (~15 lines below):

partial modifier_keys
xkb_symbols "ralt_rshift_toggle" {
    virtual_modifiers Alt;
    key <RALT> {
        symbols[Group1] = [ NoSymbol, ISO_Next_Group ],
        virtualMods= Alt
    };
// key <RTSH> {
// type[Group1]="PC_ALT_LEVEL2",
// symbols[Group1] = [ Shift_R, ISO_Next_Group ]
// };
};

then load the keymap, in my case

$ setxkbmap us,cz -option grp:alt_shift_toggle

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_lalt_toggle, but unfortunatelly I don't know how to modify the rules files..

Mathew Hodson (mhodson)
affects: control-center (Ubuntu) → gnome-control-center (Ubuntu)
Revision history for this message
In , Oded Arbel (oded-geek) wrote :

Currently, Alon Bar-Lev's patch (xorg-server-1.12.2-xkb-switch-on-release.patch) applies cleanly against Xorg 1.17.2 from Fedora 22, but apparently has no effect - with `setxkbmap -print` output like this:

xkb_keymap {
        xkb_keycodes { include "evdev+aliases(qwerty)" };
        xkb_types { include "complete" };
        xkb_compat { include "complete+ledscroll(group_lock)" };
        xkb_symbols { include "pc+us+il(lyx):2+us:3+inet(evdev)+group(alt_shift_toggle)" };
        xkb_geometry { include "pc(pc105)" };
};

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/xorg-patches, if you want to test drive.

Revision history for this message
In , Oded Arbel (oded-geek) wrote :

(In reply to Oded Arbel from comment #135)
> Currently, Alon Bar-Lev's patch
> (xorg-server-1.12.2-xkb-switch-on-release.patch) applies cleanly against
> 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,grp:alt_shift_toggle

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.

Revision history for this message
Oded Arbel (oded-geek) wrote :

@orange-buffalo: the problem you describe is not this bug - its a problem with GNOME itself. This report is about the Xserver bug that was fixed by Ubuntu (though not in Xorg itself, so it is still evident in non-Ubuntu operating system).

For the GNOME shell bug, check this bug report: https://bugzilla.redhat.com/show_bug.cgi?id=1091631

Revision history for this message
In , StingX (to-f) wrote :

Created attachment 122717
xorg-server-1.18.3-xkb-switch-on-release.patch

Here is the patch from ubuntu, that can be merged into the current debian sid version of xorg-server (1.18.3-1).

Revision history for this message
In , Alon Bar-Lev (alon-barlev) wrote :

Created attachment 126752
xorg-server-1.18.4-xkb-switch-on-release.patch

Revision history for this message
In , Alon Bar-Lev (alon-barlev) wrote :

Created attachment 126753
xorg-server-1.18.4-xkb-switch-on-release.patch

Revision history for this message
In , Bas-bmail (bas-bmail) wrote :

Created attachment 129860
The same patch, but based on 1.19.1

Revision history for this message
In , Jan Pohanka (xhpohanka) wrote :

(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 = _XkbNextFreeFilter(xkbi);
>+ sendEvent=_XkbFilterLockMods(xkbi, filter, key, act);
>+ break;
> case XkbSA_LockGroup:
> filter = _XkbNextFreeFilter(xkbi);
>- *sendEvent = _XkbFilterLockState(xkbi, filter, key, act);
>+ *sendEvent = _XkbFilterLockGroup(xkbi, filter, key, act);

Revision history for this message
In , Bas-bmail (bas-bmail) wrote :

Not really.. This is something that has changed from 1.18 to 1.19.

"act" is now already passed as reference to XkbActionGetFilter.

Revision history for this message
In , Jan Pohanka (xhpohanka) wrote :

(In reply to kyak from comment #142)

I mean dereferencing sendEvent pointer.

Revision history for this message
In , Bas-bmail (bas-bmail) wrote :

Created attachment 129861
The same patch, but based on 1.19.1 (fixed)

Revision history for this message
In , Bas-bmail (bas-bmail) wrote :

You are right, thanks for pointing that out. Updated patch attached.

Revision history for this message
In , Beaux-monde-s (beaux-monde-s) wrote :

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...

Revision history for this message
tlk (sarcasticskull) wrote :

I'v just installed Ubuntu Mate 17.04 and this patch doesn't appear to work anymore - suspect it's because Xorg has moved from evdev to libinput for handling the issue of input.
Really would like to bring the old behavior back. If anyone knows of a solution I'd be grateful if you point me at it.

Revision history for this message
tlk (sarcasticskull) wrote :

I see the patch's been dropped for zesty

changelog.Debian for xserver-xorg-core says:
xorg-server (2:1.19.3-1ubuntu1) zesty; urgency=medium
  * 208_switch_on_release.diff: Dropped, doesn't work with 1.19.

There's a new patch posted to the corresponding bug on freedesktop https://bugs.freedesktop.org/show_bug.cgi?id=865 (attachment: https://bugs.freedesktop.org/attachment.cgi?id=129861) which is supposedly compatible with 1.19

Could someone look at it please?
Vanilla behavior for those who used to switching input layout with something like Ctrl-Shift is really annoying.

Revision history for this message
In , tlk (sarcasticskull) wrote :

(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.

Revision history for this message
In , Beaux-monde-s (beaux-monde-s) wrote :

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.

Revision history for this message
In , tlk (sarcasticskull) wrote :

(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?

Revision history for this message
Lastique (andysem) wrote :

Since this bug is marked as resolved for xorg-server, I've created a new one: https://bugs.launchpad.net/ubuntu/+source/xorg-server/+bug/1683383.

Revision history for this message
In , Beaux-monde-s (beaux-monde-s) wrote :

(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.

Revision history for this message
In , tlk (sarcasticskull) wrote :

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?

Revision history for this message
In , Beaux-monde-s (beaux-monde-s) wrote :

(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"?

Revision history for this message
In , tlk (sarcasticskull) wrote :

(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://bugs.freedesktop.org/attachment.cgi?id=126753

The whole problem is that 1.19 needs a new one - this is the one provided by kyak https://bugs.freedesktop.org/attachment.cgi?id=129861

Revision history for this message
In , Beaux-monde-s (beaux-monde-s) wrote :

(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://bugs.freedesktop.org/attachment.cgi?id=126753
>
> The whole problem is that 1.19 needs a new one - this is the one provided by
> kyak https://bugs.freedesktop.org/attachment.cgi?id=129861

I have used the last one with the xorg-server v.1.19.3.

Revision history for this message
In , kolya (mar-kolya) wrote :

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 'xkbSwitchGroupOnRelease' that is a stub to make new behaviour controllable by configuration. Would xorg maintainers find this patch acceptable to upstream if that function read value from some environment variable and only turned new behaviour on if it was set?

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!

Revision history for this message
In , Ran Benita (bluetech-deactivatedaccount) wrote :

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://www.x.org/releases/current/doc/kbproto/xkbproto.html#Key_Actions

The current source for the above spec is in the kbproto repository here: https://cgit.freedesktop.org/xorg/proto/kbproto/tree/specs/ch06.xml?id=kbproto-1.0.7#n586

Revision history for this message
In , Oded Arbel (oded-geek) wrote :

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?

Revision history for this message
In , Ran Benita (bluetech-deactivatedaccount) wrote :

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.

Revision history for this message
In , Wettstae (wettstae) wrote :

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://lists.x.org/archives/xorg-devel/2012-November/034427.html

Revision history for this message
In , Ran Benita (bluetech-deactivatedaccount) wrote :

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/client/server/libxkbfile/protocol/Xlib/etc, but allow me to ignore that for now.

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://github.com/xkbcommon/libxkbcommon/commit/7c5e79159b5f5cd533a078c7672705e2ffa0a798

Revision history for this message
In , Wettstae (wettstae) wrote :

> - 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/application. If the library/application parses only used stuff, that would circument transitional compatibility issues. For an old xkbcomp with a new xkeyboard-config, I believe that would work.

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.

Revision history for this message
In , Freedesktop-x (freedesktop-x) wrote :

(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://bugzilla.gnome.org/show_bug.cgi?id=786257 which obviously was the wrong place. And now I found this bug here.

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...?

Revision history for this message
Artem Kovalov (artemkovalov) wrote :

Applying a patch indicated tlk saved my day. Hope some day this issue will be resolved forever.

Revision history for this message
In , Kof-box (kof-box) wrote :

After the next update of the package "xserver-xorg-input-evdev-hwe-16.04" (with dependencies) from the version "1: 2.10.2-1ubuntu1 ~ 16.04.1" to the version "1: 2.10.5-1ubuntu1 ~ 16.04.1 ", in the KDE shell, the language switch does not happen when the keys are released, but when pressed.

Distributions on which this problem was repeated:
neon-useredition-20171012-1018-amd64
kubuntu-16.04.3-desktop-amd64
kubuntu-17.04-desktop-amd64

Distributions in which there is no such problem:
ubuntu-16.04.3-desktop-amd64
ubuntu-17.04-desktop-amd64 (but there is no such package)

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!

Revision history for this message
In , Yanpas (yanpaso) wrote :

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.

Revision history for this message
In , Kovács Viktor (kovacs.viktor.developer) wrote :

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.

Changed in xorg-server:
status: In Progress → Won't Fix
Revision history for this message
In , Ztirfe Elgnid (z-figura12) wrote :

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.

Revision history for this message
In , Kovács Viktor (kovacs.viktor.developer) wrote :

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.

Revision history for this message
In , Ztirfe Elgnid (z-figura12) wrote :

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?

Revision history for this message
In , Kovács Viktor (kovacs.viktor.developer) wrote :

Who invite me this blog?

Revision history for this message
In , Alon Bar-Lev (alon-barlev) wrote :

(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: Won't Fix → Confirmed
Revision history for this message
In , Kovács Viktor (kovacs.viktor.developer) wrote :

Created attachment 134843
KDE keyboard layout switcher screenshot

Sorry, I'm Hungarian.

Revision history for this message
In , Kovács Viktor (kovacs.viktor.developer) wrote :

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.

Revision history for this message
In , Maciej Pilichowski (bluedzins) wrote :

@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.

Revision history for this message
In , Yanpas (yanpaso) wrote :

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?

Revision history for this message
In , Kovács Viktor (kovacs.viktor.developer) wrote :

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.

Revision history for this message
In , Yanpas (yanpaso) wrote :

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.

Revision history for this message
In , Bas-bmail (bas-bmail) wrote :

(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.

Revision history for this message
In , Kovács Viktor (kovacs.viktor.developer) wrote :

(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://bugs.kde.org/show_bug.cgi?id=59303
>
>
>
> 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_shift_toggle"
> Option "XkbRules" "xfree86"
> Option "XkbVariant" ",winkeys"
> EndSection

Kubuntu grants Control+Shift+K combination. Is it acceptable?

Revision history for this message
In , Ztirfe Elgnid (z-figura12) wrote :

(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.

Revision history for this message
In , Kovács Viktor (kovacs.viktor.developer) wrote :

(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.

Revision history for this message
In , Kovács Viktor (kovacs.viktor.developer) wrote :

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?

Revision history for this message
In , Jan Pohanka (xhpohanka) wrote :

(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.

Revision history for this message
In , Kof-box (kof-box) wrote :

(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.

Revision history for this message
In , Alex Efros (powerman-asdf) wrote :

(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.

Revision history for this message
In , Kof-box (kof-box) wrote :

(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?

Revision history for this message
In , Alex Efros (powerman-asdf) wrote :

(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/patches/x11-base/xorg-server/
- 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>

Revision history for this message
In , Daniel Stone (daniels) wrote :

(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.

Revision history for this message
In , Mim-t (mim-t) wrote :

(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/patches/x11-base/xorg-server/
> - 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.

Revision history for this message
In , Ztirfe Elgnid (z-figura12) wrote :

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.

Revision history for this message
In , Wettstae (wettstae) wrote :

(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://lists.x.org/archives/xorg-devel/2012-November/034427.html
https://lists.x.org/archives/xorg-devel/2012-November/034430.html
https://lists.x.org/archives/xorg-devel/2012-November/034429.html
https://lists.x.org/archives/xorg-devel/2012-November/034431.html
https://lists.x.org/archives/xorg-devel/2012-November/034428.html

Revision history for this message
In , Daniel Stone (daniels) wrote :

(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://lists.x.org/archives/xorg-devel/2012-November/034427.html
> https://lists.x.org/archives/xorg-devel/2012-November/034430.html
> https://lists.x.org/archives/xorg-devel/2012-November/034429.html
> https://lists.x.org/archives/xorg-devel/2012-November/034431.html
> https://lists.x.org/archives/xorg-devel/2012-November/034428.html

Here's what I think we would need to do in order to not break old clients:
https://lists.freedesktop.org/archives/xorg-devel/2013-January/035049.html

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.

Revision history for this message
In , Beaux-monde-s (beaux-monde-s) wrote :

(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.

Revision history for this message
In , David-cortes-rivera (david-cortes-rivera) wrote :

Another user here voting for this bug to get patched OFFICIALLY from upstream.

Revision history for this message
In , Norbert (nrbrtx) wrote :

Did you changed your opinion after for years of not fixing this bug?

Users still need this functionality (see https://community.ubuntu.com/t/keyboard-layout-switching-problems-and-poll/2876 and https://askubuntu.com/q/1009352/66509 as examples).

Revision history for this message
In , Norbert (nrbrtx) wrote :

All current Ubuntu versions are affected and RHEL too. And nobody cares.

14 years of doing nothing. My congratulations!

Revision history for this message
In , Daniel Stone (daniels) wrote :

(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.

Revision history for this message
In , Norbert (nrbrtx) wrote :

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://bugs.debian.org/cgi-bin/bugreport.cgi?bug=891915 ).

So versions 1.16.4, 1.19.2 and 1.19.5 should be patched to bring <Ctrl+Shift> keyboard shortcut functionality back.

Revision history for this message
In , Kovács Viktor (kovacs.viktor.developer) wrote :

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!

Revision history for this message
In , 7-andrew-0 (7-andrew-0) wrote :

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.

Revision history for this message
Norbert (nrbrtx) wrote :

Dear all!

I have created PPA ( https://launchpad.net/~nrbrtx/+archive/ubuntu/xorg-hotkeys ) with patched packages for Ubuntu 16.04 LTS (xenial, with HWE) and Ubuntu 18.04 LTS (bionic).

You can test them by the following commands:

sudo add-apt-repository ppa:nrbrtx/xorg-hotkeys
sudo apt-get update
sudo apt-get dist-upgrade

Hope this helps.

tags: added: artful bionic trusty xenial
Revision history for this message
In , Norbert (nrbrtx) wrote :

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://launchpad.net/~nrbrtx/+archive/ubuntu/xorg-hotkeys or " ppa:nrbrtx/xorg-hotkeys " ). It contains patched Xorg (with patch from kyak - https://aur.archlinux.org/packages/xorg-server-bug865/ ). Thank you very much again, kyak!

And it is unbelievable that we need to patch core graphical system component by ourselves to use traditional keyboard shortcuts ...

Revision history for this message
Telmenko Igor Valerievish (itelmenko) wrote :

Why does it not in default settings?

Revision history for this message
Dmitry Balashov (fixed77) wrote :

more than 12 years of this bug. a shame.

Revision history for this message
In , Gitlab-migration (gitlab-migration) wrote :

-- 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://gitlab.freedesktop.org/xorg/xserver/issues/258.

Changed in xorg-server:
status: Confirmed → Unknown
Revision history for this message
In , Idawallace89 (idawallace89) wrote :

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://www.assignmenthelpfolks.com/psychology/ at all university levels at Assignment Help Folks in Australia.

Revision history for this message
Mikhail Chabanov (bolderdush) wrote :

SOLUTION

For myself I created temporary hack allowing to switch Keyboard layouts (en<->ru) by Ctrl+Shift on key release. My method does not interfere with other shortcuts. I use it myself on daily basis.

It's simple Python script which listen to Ctrl+Shift key combination and switch layout on release.
It can be downloaded here https://drive.google.com/file/d/1IvgvLFwe2HoGtEzuQnrUoAnEZk1ZCcSw/view?usp=sharing
Please read instructions in ReadMe.txt

Norbert (nrbrtx)
tags: added: cosmic disco eoan
removed: artful
Revision history for this message
In , Alex-wilson (alex-wilson) wrote :

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://www.writecdr.com/ipenz-ka02-new-zealand-skilled-migration/
https://www.writecdr.com/acs-rpl-report-samples/

Revision history for this message
In , Olivia-jackson (olivia-jackson) wrote :

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://www.cdrreport.org/cpd-examples-samples/
https://www.cdrreport.org/how-to-create-a-perfect-resume-for-engineers-australia/

Revision history for this message
Alexey (alexwtan) wrote :

Someone should revert this 'Fix Released' status - it is here again in 18.04 (and according to forums in 17.x also).

Changed in xorg-server (Ubuntu):
status: Fix Released → Confirmed
Revision history for this message
In , Castro8583bennett (castro8583bennett) wrote :

Hi! I have the same issue i would like to know the solution

Castro B,
gratisdatingsite.nl

Changed in xorg-server:
status: Unknown → Fix Released
Changed in xorg-server:
importance: High → Unknown
status: Fix Released → Unknown
Changed in xorg-server:
status: Unknown → New
tags: removed: cosmic
Revision history for this message
Ioann (yanunim95) wrote :

So. This problem is still here.
I found following situation:
On older 16.04 layout is changing with releasing Control/Shift/Alt keys.
In 18.04 layout is changing while pressing key combination.
May be it is setting somewhere in X.org?
Is it possible to find and change it?
It can affect all people in the world, that have non latin-like language as basic language.

Revision history for this message
Norbert (nrbrtx) wrote :

Just a silent reminder: I have created a PPA ( https://launchpad.net/~nrbrtx/+archive/ubuntu/xorg-hotkeys ) with patched packages for Ubuntu 16.04 LTS (xenial, with HWE) and upwards (including upcoming 19.10), use its description for instructions.

Revision history for this message
Ioann (yanunim95) wrote :

@Norbert
I have Laptop with 16.04.
And Alt+Shift, Alt+Shift+Tab are working ok.
I have also tested it with new default installation in Virtualbox.
And it also works fine.
And it doesn't matter if i use Mate or Unity.

Revision history for this message
Dik (dikiy-evrej) wrote :

I would say, that change on Keypress _must_ be the default behavior, because in an opposite case, there are problems of typing, if toogle in on CapsLock, and you are typing rapidly.

Revision history for this message
Dik (dikiy-evrej) wrote :

I have an Ubuntu 16.04, how I can restore the normal behavior? I mean change on Keypress

Revision history for this message
In , Dik (dikiy-evrej) wrote :

It is not a bug, its a feature. To speed up the typing. Set the layout change on CapsLock. (In a case you need old caps behavior, you get it with shift + Caoslock)

Revision history for this message
ALEKSEI VOLKOV (softkot) wrote :

I am running ubuntu 19.10 with recent gnome 3 and did not find any working solution to switch layout on Ctrl +Shift shortcuts key release. Eventualy i wrote a dirty workoround to fit my needs.

https://gitlab.com/softkot/gnome3_xinput_layout_switch

Revision history for this message
Oded Arbel (oded-geek) wrote :

The GNOME issue is completely separate from the problem of "triggering layout change on press", as GNOME does not use the kxb behavior at all - it actually disables any options set using setxkbmap - instead it listens to keyboard events and implements its own custom behavior that mimics Mac OS.

Note that with the current release (19.10, but also tested 20.04) on Kubuntu with xserver 1.20 that *does not* include the patch from freedesktop.org bug 865, layout switching works on release - as requested. I'm not sure what introduced this behavior and whether it is in X.org or in KDE.

Revision history for this message
Yuriy Chernichenko (nikosid) wrote :

@nrbrtx, hi. Thank you so much for your patch. I'm using it for more than one year, but few days ago I reinstalled my OS and it doesn't work now. I tried to completely reinstall it but it didn't help.
I use xubuntu 18.04.5 LTS.
Can you help or give some advice?

Revision history for this message
Norbert (nrbrtx) wrote :

@Yuriy Chernichenko (nikosid)

I have just uploaded newest HWE package versions with latest security fixes to the PPA.
You have to try adding it again and upgrading Xorg packages from it.

Revision history for this message
Yuriy Chernichenko (nikosid) wrote :

@nrbrtx it works! Thank you very much!

Revision history for this message
Pavel Terentyev (sanovskiy) wrote :

@nrbrtx it works for me (Ubuntu 20.04 LTS)
Strl+Shift+t now works but still switches layout.

Norbert (nrbrtx)
tags: added: focal hirsute impish
removed: disco eoan trusty
tags: removed: hirsute
Revision history for this message
WildWeasel (wildweasel) wrote :

I don't understand why hirsute has been removed, I'm using it and just hit this bug. Please restore the hirsute tag.

Revision history for this message
Jeremy Bícha (jbicha) wrote :

Ubuntu 21.04 is no longer supported. If you are using 21.04, please upgrade to 21.10.
https://wiki.ubuntu.com/Releases

tags: removed: impish
Norbert (nrbrtx)
tags: added: jammy kinetic
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Looks like all recent reports are pointing to bug 1683383 instead so let's just use that.

Changed in xorg-server (Ubuntu):
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.