-
Notifications
You must be signed in to change notification settings - Fork 974
Change session tab shortcuts to fix AltGr problem #13015
Change session tab shortcuts to fix AltGr problem #13015
Conversation
Update so that the shortcut doesn't interfere with AltGr shortcuts. The new shortcut is Control + Shift + [session number] for Windows and Command + Option + [session number] on Mac OS.
Update so that the shortcut doesn't interfere with AltGr shortcuts. The new shortcut is Control + Shift + [session number] for Windows and Command + Shift + [session number] on Mac OS.
Compiled it myself and tested it out. Seems to be working as it should. tested on windows only. SUGGESTION: Make Windows use Control + Shift and MacOS use Command + Opition, what do you think? |
Just curious, is the proposed change following de facto standard? |
If you are talking about shortcut standard I'd say it does, at least on Windows and Linux, somebody please help me out with MacOS ;-) My reasoning for this is that |
Thanks for your pull request @vinicentus! Unfortunately we cannot accept it at this time since cmd-shift-[number] will interfere with macOS global shortcuts for screenshots. Until we find something suitable, I'm going to revert #10364 and remove the new shortcut entirely in order to fix user's input issues as soon as possible. |
How about this: Then it would be
|
Codecov Report
@@ Coverage Diff @@
## master #13015 +/- ##
==========================================
+ Coverage 56.13% 56.16% +0.02%
==========================================
Files 279 279
Lines 27332 27352 +20
Branches 4446 4452 +6
==========================================
+ Hits 15343 15362 +19
- Misses 11989 11990 +1
|
Opening a new issue for this... |
Update so that the shortcut doesn't interfere with AltGr shortcuts as seen here: #12956. This would also change the shortcuts to
Cmd + Alt + partitionNumber
for MacOS andCtrl + Shift + partitionNumber
for Windows and Linux.I personally think this is a better shortcut since it's easier to do with only one hand (at least on Windows and Linux).
The original problem comes from here #6915
NOTE: The shortcuts also need to be changed like seen here #13000, but I couldn't find it in the code.Fixes #13000
Fixes #12956
Submitter Checklist:
git rebase -i
to squash commits (if needed).Test Plan:
Reviewer Checklist:
Tests