-
Couldn't load subscription status.
- Fork 35.8k
Include model info in chat input aria label #273160
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
Co-authored-by: meganrogge <29464607+meganrogge@users.noreply.github.com>
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.
Pull Request Overview
This PR enhances accessibility by including the current language model name in the chat input's aria-label, helping screen reader users understand which AI model they're interacting with.
Key Changes:
- Updated the
_getAriaLabel()method to extract and include the current language model name in all aria-label variants - Added automatic aria-label refresh when the language model changes via the
_onDidChangeCurrentLanguageModelevent listener
Co-authored-by: Daniel Imms <2193314+Tyriar@users.noreply.github.com>
Improves accessibility by including the current language model name in the chat input's aria-label, helping screen reader users understand which AI model they're interacting with.
Changes
Modified
src/vs/workbench/contrib/chat/browser/chatInputPart.ts:Updated
_getAriaLabel()method to extract and include the current language model namethis._currentLanguageModel?.metadata.nameAdded aria-label refresh on model change
_onDidChangeCurrentLanguageModelevent firesExample
Before:
After:
When no model is selected:
(Falls back gracefully to original behavior)
Accessibility Impact
Screen reader users now receive:
cc @jooyoungseo
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.