Summary: | New key-held options need to default to repeat for backward compatibility | ||
---|---|---|---|
Product: | [Plasma] plasma-integration | Reporter: | Duncan <1i5t5.duncan> |
Component: | character alternatives palette | Assignee: | Janet Blackquill <uhhadd> |
Status: | RESOLVED DUPLICATE | ||
Severity: | normal | CC: | nate, plasma-bugs, wengxt |
Priority: | NOR | ||
Version: | master | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Other | ||
Latest Commit: | Version Fixed In: |
Description
Duncan
2020-12-06 08:44:08 UTC
I'm afraid I'm a bit confused here. Can you clarify what the exact bug is, and not report multiple issues or propose a solution? (In reply to Nate Graham from comment #1) > Can you clarify Thanks for asking. After my update and a plasma restart, key repeat was, from what I could see, simply broken -- keys no longer repeated. There was no indication what had changed; the key-repeat that had worked fine before now simply didn't. Eventually I found out that was because the new keyboard kcm settings had defaulted to do-nothing for key-hold-down, and I was able to switch it back to the repeat I had before. But changing the setting didn't seem to do anything, repeat was still broken. I only got repeat back after I set it *and* then restarted plasma. Expectation: If a user had key-repeat set before, as I did, that setting should be retained thru the upgrade. And if a user changes the repeat setting in the kcm the change should take effect immediately, not require a plasma restart to take effect. Clearer? Do I need to split the doesn't-take-effect-immediately to a separate bug? So when you went to the keyboard KCM, the "Do nothing" radio button was checked? (In reply to Nate Graham from comment #3) > So when you went to the keyboard KCM, the "Do nothing" radio button was > checked? Yes. That's weird. Does pressing the "Defaults" button in the Keyboard KCM reset it to "Do nothing? It doesn't do that for me. (In reply to Nate Graham from comment #5) > That's weird. Does pressing the "Defaults" button in the Keyboard KCM reset > it to "Do nothing? It doesn't do that for me. Defaults button changes it to accented/similar. BTW, I don't believe I specified on this bug yet, I'm on wayland by default now. If you want me to test on xorg I can, but I now default to wayland and all tests so far are on wayland. Meanwhile, after the defaults button switched me to accented/similar, I decided to try it. But holding some letters like a and e that I know have variants... didn't show any variants, as I expected. Instead, the keys repeated. But I had it on repeat previously and remembered I had to restart kde/plasma for a change, so I did that... and I /still/ got repeat, not variants. So I decided to try do-nothing again, since I knew what it did, and sure enough, while I did have to restart plasma again for it to take effect, it did switch to do-nothing. Switching a few more times (and restarting plasma each time) to double-check the tests, do-nothing does what it says on the label -- nothing. Repeat does what it says on the label -- repeat. But show-accented/similar just does repeat as well, only the repeat delay and rate sliders aren't there to adjust the repeat as they are in actual repeat mode. (Again, this is on wayland.) Finally, while I'm upgraded now on my working copy, I do have a backup from a few days before the switch to the new-style keyboard-kcm. If it's helpful, I can logout of my normal user and move my working homedir to a new backup, then copy the old pre-keyboard-kcm-upgrade homedir from that backup, and log back in as my normal user and start plasma to repeat the keyboard-kcum upgrade and see if it again switches to do-nothing, or if that was a one-time thing perhaps due to the order in which the components upgraded or something. And/or I can test the upgrade on xorg and see if the results are different than on wayland. So your behavior, if on X11, is probably due to this bug (I don't know how that config is applied on wayland, probably kwin is involved.): https://bugs.kde.org/show_bug.cgi?id=418175 The bug is when you apply rate config from kcm, it doesn't read the new value, which I just fixed. Also, the accent option, would rely on QT_IM_MODULE set to plasmaim, which probably doesn't set on your desktop. Due to the fact that it may break normal input method usage, it won't gonna be set by default or being override by your desktop setings. And right now when it is not set to plasmaim, accent and repeat will basically have the same behavior. I'll probably send patch to hide the key-held if QT_IM_MODULE is not plasmaim. (In reply to Weng Xuetian from comment #7) > So your behavior, if on X11, is probably due to this bug (I don't know how > that config is applied on wayland, probably kwin is involved.): > https://bugs.kde.org/show_bug.cgi?id=418175 > The bug is when you apply rate config from kcm, it doesn't read the new > value, which I just fixed. I was on wayland, but that makes total sense. Thank you. (I'm temporarily stuck on X ATM due to bug #430501.) > Also, the accent option, would rely on QT_IM_MODULE set to plasmaim, which > probably doesn't set on your desktop. Due to the fact that it may break > normal input method usage, it won't gonna be set by default or being > override by your desktop setings. And right now when it is not set to > plasmaim, accent and repeat will basically have the same behavior. That makes sense as well, and in fact, being en_US and on gentoo where compile-time-options usually translate to user-settable USE flags, if it's compile-time-optional there's a very good chance I don't have that module at all. > I'll probably send patch to hide the key-held if QT_IM_MODULE is not > plasmaim. Should reduce confusion. Thanks again. All this likely makes this bug a dupe of bug #418175. After updating today I'll test a bit more. Trouble is if #430051 isn't fixed I won't be able to test on wayland again until it's fixed too and I get back on wayland... But then I can probably close this as a dupe of that one. Others may be able to do so sooner. Yep seems like a dupe of Bug 418175. *** This bug has been marked as a duplicate of bug 418175 *** Git commit 6b7040f3ea2e02c7ca805278fe7fe06ff31b0ec7 by Xuetian Weng, on behalf of Weng Xuetian. Committed on 19/12/2020 at 18:11. Pushed by xuetianweng into branch 'master'. Hide accent option if it is not supported M +14 -1 kcms/keyboard/kcmmisc.cpp M +38 -25 kcms/keyboard/kcmmiscwidget.ui https://invent.kde.org/plasma/plasma-desktop/commit/6b7040f3ea2e02c7ca805278fe7fe06ff31b0ec7 |