-
Notifications
You must be signed in to change notification settings - Fork 28.8k
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
Create AccessibleView
, use it for terminal and chat accessibility help menus
#184435
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.
I wonder if it makes sense to leverage MultiCommand
vscode/src/vs/editor/browser/editorExtensions.ts
Lines 193 to 194 in d4f29ca
export class MultiCommand extends Command { | |
- Single command contribution, each contributor only needs to register a provider other than both the provider and the action contribution
- If users want to change the keybinding, they don't have to change the keybindings for every
*.showAccessibilityHelpView
command. - This command can then also be added to the Menu
Current approach doesn't seem to scale.
Yes @rebornix i thought the same and was unaware such a thing existed. Thanks! |
@rebornix i'll need to defer that to the PR where I convert the editor accessibility help action to use this - currently, what's happening is that is overriding the terminal's action since there's no Edit: got it to work |
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
src/vs/editor/standalone/browser/accessibilityHelp/accessibilityHelp.css
Outdated
Show resolved
Hide resolved
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
src/vs/workbench/contrib/accessibility/browser/accessibleView.ts
Outdated
Show resolved
Hide resolved
Co-authored-by: Daniel Imms <2193314+Tyriar@users.noreply.github.com>
…View Create `AccessibleView`, use it for terminal and chat accessibility help menus
Fixes #183363
fixes #179689
Remaining help menu adoption (more complex so in separate PRs):
To do later:
AccessibleView
#184612demo.mov