<html> <head> <base href="https://bugzilla.gnome.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - inconsistent interpretation of preserved modifiers with xkbcommon" href="https://bugzilla.gnome.org/show_bug.cgi?id=754110#c29">Comment # 29</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - inconsistent interpretation of preserved modifiers with xkbcommon" href="https://bugzilla.gnome.org/show_bug.cgi?id=754110">bug 754110</a> from <a href="page.cgi?id=describeuser.html&login=daniel%40fooishbar.org" title="Daniel Stone <daniel@fooishbar.org>"> Daniel Stone</a> <pre>Yeah, looking at things just as they stand, I agree that Carlos's patch is probably fine, in that it fixes things for just the CTRL+ALT type. However, it's also pretty nasty, and I can nearly guarantee it will have unforseen issues later on down. We now have a bunch of pretty weird small keyboards which rely heavily on multiple modifiers for chording - the full 80s Emacs experience - so I worry that encoding this will just cause trouble later on down the line. I really like Owen's conclusion here: > - consumed_modifiers contains modifiers that effect the translation > of the key *that are found in event->state* although I would perhaps reword as: > - consumed_modifiers contains modifiers that directly affected the translation of the last keypress and I think that would be something good to encode generically through xkbcommon. Whether that's something we do by default (i.e. break current interface and also legacy XKB's definition of 'consumed'), or something we require people to opt in through an xkb_context flag for is up for discussion. My suggestion though, would be doing it by default, with an xkb_context flag to _disable_ (i.e. force old behaviour), which happens to work fine as we don't currently check for unsupported flags in xkb_context_new. :\ If you're fine with that, I can get the patch together and test with GTK+, since this hits me every day.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are on the CC list for the bug.</li> </ul> </body> </html>