Bug 470203 - Custom keyboard shortcuts handled incorrectly
Summary: Custom keyboard shortcuts handled incorrectly
Status: CONFIRMED
Alias: None
Product: frameworks-kglobalaccel
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 5.106.0
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: kdelibs bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-05-24 10:31 UTC by Marian Ciobanu
Modified: 2023-05-26 15:42 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marian Ciobanu 2023-05-24 10:31:00 UTC
SUMMARY
I'm trying to customize the key shortcuts but I ran into an issue where a conflict is detected and supposedly solved when assigning the shortcut. When pressing the key combination, I get told to solve the ambiguity.

STEPS TO REPRODUCE
1. Go to Settings / Configure Keyboard Shortcuts.
2. Find "Move to Top of View" and assign "Ctrl+PgUp"
3. Get prompted with "The "Ctrl+PgUp" shortcut is ambiguous with the following shortcut. Do you want to assign an empty shortcut to this action? Shortcut 'Alt+Left' for action 'Previous Tab'"
4. Choose "Reassign".
5. Close the settings dialog and back in the edit window press "Ctrl+PgUp".

OBSERVED RESULT
A message pops up, stating: "The key sequence 'Ctrl+PgUp' is ambiguous. Use 'Configure Keyboard Shortcuts' from the 'Settings' menu to solve the ambiguity. No action will be triggered."

EXPECTED RESULT
The cursor should move to the top of the page.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: Probably latest, as I'm on opeSUSE Tumbleweed. Not sure how to find it, but I don't see it in "About System"
KDE Frameworks Version: 5.106.0
Qt Version: 5.15.9

ADDITIONAL INFORMATION
Workaround:

Open "Configure Keyboard Shortcuts", find "Previous Tab" and unassign the alternate or make some other change (once I saw a Ctrl+PgUp there, but mostly it's "Back").
Open /home/$USER/.local/share/kxmlgui5/kate/kateui.rc go to the end, find view_prev_tab, and remove Ctrl+PgUp from its entry.

Before edit:
    <Action name="view_prev_tab" shortcut="Back; Ctrl+PgUp; Ctrl+["/>
After edit it's
    <Action name="view_prev_tab" shortcut="Back"/>
Comment 1 Paul Worrall 2023-05-24 16:32:44 UTC
This does not seem to be an issue with Kate but with how plasma handles shortcuts in general (see bug #434851).
Comment 2 Marian Ciobanu 2023-05-26 08:28:35 UTC
(In reply to Paul Worrall from comment #1)
> This does not seem to be an issue with Kate but with how plasma handles
> shortcuts in general (see bug #434851).

Thanks, but I'm not sure what the next step should be. Should I reassign this to another product? Do you have any suggestion regarding which one, though? There are so many ... (Looking at https://bugs.kde.org/show_bug.cgi?id=434851, it seems that not much is likely to change if left as is.)
Comment 3 Paul Worrall 2023-05-26 15:42:59 UTC
I'm not sure either, I'll assign it to kglobalaccel for now until someone who knows better reassigns it.