Bug 421250 - Split-view keybinding label incorrect in menu
Summary: Split-view keybinding label incorrect in menu
Status: CONFIRMED
Alias: None
Product: konsole
Classification: Applications
Component: split-view (show other bugs)
Version: 19.12.3
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Konsole Developer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-05-09 21:58 UTC by Woods Wannamaker
Modified: 2020-11-01 13:11 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Woods Wannamaker 2020-05-09 21:58:59 UTC
SUMMARY
Konsole split-view key-bindings shown under View > Split View are:
Split View Left/Right crtl+(
Split View Top/Bottom crtl+)

but do not work.

STEPS TO REPRODUCE in Konsole 19.12.3
1. Test key-binding crtl+(
2. Test key-binding crtl+)

OBSERVED RESULT
Konsole Split L/R and T/B does not occur

EXPECTED RESULT
Konsole Screen should split L/R or T/B

SOFTWARE/OS VERSIONS
Operating System: Kubuntu 20.04
KDE Plasma Version: 5.18.4
KDE Frameworks Version: 5.68.0
Qt Version: 5.12.8
Kernel Version: 5.4.0-29-generic
OS Type: 64-bit
Processors: 8 × Intel® Core™ i7-9700K CPU @ 3.60GHz
Memory: 31.3 GiB of RAM

ADDITIONAL INFORMATION
The key-bindings that DO work are:
    Split View Left/Right: crtl + shift + (
    Split View Top/Bottom: crtl + shift + )

Key-bindings to Expand and Shrink the view are correctly labeled:
    Expand view: crtl + shift + ]
    Shrink view: crtl + shift + [

Please investigate and either correct the key-bindings or the key-binding labels in the View menu.
Comment 1 Justin Zobel 2020-10-22 23:54:36 UTC
So they key bindings do work (version 20.08.1) but they are poorly worded. It is expected that the modifiers should be shown for all shortcuts so users are not confused. 

To get a ( you have to press shift, so it is assumed, but not all users might know/understand this. 

I would propose the label be changed to Ctrl Shift 9 / 0.
Comment 2 Justin Zobel 2020-11-01 05:07:54 UTC
Confirmed.
Comment 3 Carlos Alves 2020-11-01 13:11:00 UTC
(In reply to Justin Zobel from comment #1)
> So they key bindings do work (version 20.08.1) but they are poorly worded.
> It is expected that the modifiers should be shown for all shortcuts so users
> are not confused. 
> 
> To get a ( you have to press shift, so it is assumed, but not all users
> might know/understand this. 
> 
> I would propose the label be changed to Ctrl Shift 9 / 0.


You'r right it will not accept the Shift+(, for example if you go to Settings->Configure Keyboard and changes "Expand View" and press custom Ctrl+Shift+[ it will now show Ctrl+{, and work the same way as before.

So if you want to use Ctrl+Shift+( you have to set it Ctrl+Shift+9 in code. There is no way to set in code Ctrl+Shift+(, because Shift+( does not exist, when you press Shift+9 it will see as Shift+9 or just a '('.