-
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
enable easy configuration of unassigned keybindings for extension contributed accessibility help dialogs #212997
Conversation
return; | ||
} | ||
const quickPick: IQuickPick<IQuickPickItem> = this._quickInputService.createQuickPick(); | ||
this._register(quickPick); |
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.
You should dispose the quickpick when you're done with it, and the listeners below should also be disposed
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 I do on lines 381 and 383. The listeners are registered to the quick pick I believe?
…bility help dialogs (microsoft#212997)
…bility help dialogs (microsoft#212997)
fixes #210665
The parsing/keybinding resolution will move into the
AccessibleViewService
or a helper class when I next work on #210695. This is just a step in that direction.demo.mov