-
Notifications
You must be signed in to change notification settings - Fork 27.9k
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
suppport alt+f2
, ctrl/cmd+Up/DownArrow
to go to terminal accessible buffer, synchronize windows/mac keybinding for command nav in accessible buffer
#191108
Conversation
alt+f2
, ctrl/cmd+Up/DownArrow
to go to terminal accessible bufferalt+f2
, ctrl/cmd+Up/DownArrow
to go to terminal accessible buffer, synchronize windows/mac keybinding for command nav in accessible buffer
@@ -22,6 +22,10 @@ export const terminalStrings = { | |||
value: localize('workbench.action.terminal.focus', "Focus Terminal"), | |||
original: 'Focus Terminal' | |||
}, | |||
focusAndHideAccessibleBuffer: { | |||
value: localize('workbench.action.terminal.focusAndHideAccessibleBuffer', "Focus Terminal and Hide Accessible Buffer"), | |||
original: 'Focus Terminal and Hide Accessible Buffer' |
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.
Can this be the default behavior of the focus terminal command? As opposed to FocusAccessibleBuffer
which focuses the accessible buffer, not the textarea?
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.
I tried that originally. that does not work because hide
has to be called on the accessible view component and we cannot access that here. can refactor this when we move over to using the accessible view for this in the future
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.
I think you could handle that with ITerminalInstance.onDidFocus
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.
that's what I tried initially. It was confusing because the event is named onDidRequestFocus
fixes #188927
fixes #189360
Re this one, I think it makes sense to unify across platforms for screen reader users. It doesn't present an inconsistency as currently, in the terminal, ctrl+up/down does not jump to commands.
fixes #189358