Skip to content
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

[ProDev Copilot] Enhance accessibility by adding ARIA attributes and keyboard navigation #1085

Merged
merged 3 commits into from
Dec 23, 2024

Conversation

amitjoshi438
Copy link
Contributor

This pull request focuses on improving accessibility in the src/common/copilot/assets/scripts/copilot.js file. The changes include adding aria-label attributes to various elements and implementing keyboard accessibility for interactive components.

Accessibility improvements:

  • Added aria-label attributes to copyButton, insertButton, suggested-prompt, walkthrough-content, and loginButton to improve screen reader support. [1] [2] [3] [4] [5]
  • Added aria-live attributes to thinking and apiResponseElement to ensure dynamic content updates are announced by screen readers. [1] [2]
  • Added keyboard event listeners to suggested-prompt and walkthroughContent to support keyboard navigation and interaction.…on support

@amitjoshi438 amitjoshi438 requested review from a team as code owners December 20, 2024 06:37
@amitjoshi438 amitjoshi438 changed the title Enhance accessibility by adding ARIA attributes and keyboard navigation [ProDev Copilot] Enhance accessibility by adding ARIA attributes and keyboard navigation Dec 20, 2024
@amitjoshi438 amitjoshi438 enabled auto-merge (squash) December 23, 2024 09:47
@amitjoshi438 amitjoshi438 merged commit b9db334 into main Dec 23, 2024
6 checks passed
@amitjoshi438 amitjoshi438 deleted the users/amitjoshi/ariaLabelForCopilotPanel branch December 23, 2024 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants