-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Fix: Fixed issue where OEM plus key for Alt+Shift++ (New pane) wasn't working #10405
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggested a few changes to make the code cleaner, thanks a lot for your contribution!
I cannot test the function on Files for the moment sadly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for the changes!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good but there is a coupe of spacing issues, can you make sure you have the file set to use tabs?
Yes, I didn't notice it was "mixed". It should be fixed now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
The New Pane keyboard shortcut (Alt+Shift++) is now also executed with the "oem" plus key, in addition to the numpad plus key.
Resolved / Related Issues
Items resolved / related issues by this PR.
Validation
How did you test these changes?