Created attachment 52146 [details] [review] "acts as onetime lock" patch These options (why are they named differently, btw?) allow to delay level switch action when two same level choosers are pressed simultaniously. However, their descriptions are inconsistent with each other, and the term "latching" is unclear. The two patches attached reword those options to be consistent. First patch makes them sound like this: KEYNAME chooses 3rd level, acts as onetime lock when pressed together with another 3rd-level-chooser the second one rewords them like this: KEYNAME chooses 3rd level, delays when pressed together with another 3rd-level-chooser Choose one. :)
Created attachment 52147 [details] [review] "delays" patch
NB: It looks like lv5:*_switch_lock_cancel options are actually nonfunctional, cause their descriptions look very much the same as lv5:*_switch_lock (without _cancel). Is this a bug?
s/descriptions/definitions/ in last comment
*_switch_lock_cancel are using different xkb_type. Which is commented out because of the bug in XKB code. But at some point they will be enabled. I hope.
Regarding the bug in XKB code, affecting *_switch_lock_cancel, see bug #27988 Otherwise - committed, thanks!
Since the lv5:*_switch_lock_cancel don't work yet (nothing to break thus), wouldn't it make sense to rename them to lv5:*_switch_latch for the sake of consistency?
I am not sure... perhaps hiding them altogether from base.xml.in would be more sane option
(In reply to comment #7) > I am not sure... perhaps hiding them altogether from base.xml.in would be more > sane option That makes sense for sure, please do so. I was simply suggesting to rename those options BEFORE bug #27988 is fixed.
ok, commented them out for now
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.