Skip to content

fix: add accessible labels to sidebar controls#12327

Open
danyalahmed1995 wants to merge 2 commits intocontinuedev:mainfrom
danyalahmed1995:fix/issue-12325-sidebar-a11y-labels
Open

fix: add accessible labels to sidebar controls#12327
danyalahmed1995 wants to merge 2 commits intocontinuedev:mainfrom
danyalahmed1995:fix/issue-12325-sidebar-a11y-labels

Conversation

@danyalahmed1995
Copy link
Copy Markdown

@danyalahmed1995 danyalahmed1995 commented May 7, 2026

Related to #12325

Summary

Adds accessible labels and titles to a few obvious icon only controls in the sidebar/chat UI.This is intended as a small first pass improvement for screen reader support. It does not try to solve the full sidebar accessibility issue in one PR.

Testing

  • Ran npm.cmd run lint --prefix gui
  • Ran git diff --check

Notes

I kept the scope focused on controls where the action was clear from the existing UI.


Summary by cubic

Adds accessible labels and titles to sidebar/chat controls and labels the main chat input for screen readers. Addresses #12325 by clearly labeling close, send, configure, and the chat input.

  • Bug Fixes
    • FindWidget: add "Close find widget" aria-label and title to the close button.
    • InputToolbar: add "Send message" aria-label and title to the submit button.
    • ModelSelect: add "Configure " to per-option cog and "Configure models" to the toolbar cog.
    • HeaderButtonWithToolTip: set aria-label and title from the text prop for icon-only header buttons.
    • TipTap editor (main input): add aria-label "Chat message input".

Written for commit 0b3ac42. Summary will update on new commits.

@danyalahmed1995 danyalahmed1995 requested a review from a team as a code owner May 7, 2026 18:35
@danyalahmed1995 danyalahmed1995 requested review from sestinj and removed request for a team May 7, 2026 18:35
@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label May 7, 2026
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 4 files

@danyalahmed1995
Copy link
Copy Markdown
Author

It looks like PR Checks / jetbrains-tests failed, but this PR only changes GUI accessibility attributes in React components.I’m checking the logs now. If this is unrelated, would it be possible to rerun that check?

@danyalahmed1995
Copy link
Copy Markdown
Author

danyalahmed1995 commented May 7, 2026

Hmph the failing check is PR Checks / jetbrains-tests, specifically Autocomplete > testAutocomplete() in the JetBrains integration suite.This PR only adds aria-label / title attributes to GUI React components and does not touch the JetBrains extension or autocomplete code, so this looks unrelated to the patch.

@paul-geoghegan
Copy link
Copy Markdown

Hey, great work. Can I make one request for this pr and thats if you could label the input box. I'll make a new issue now for the message history. Thanks!

@danyalahmed1995
Copy link
Copy Markdown
Author

@paul-geoghegan Thank you , I will be able to add the input box label to this PR in an hour or so.

@paul-geoghegan
Copy link
Copy Markdown

@danyalahmed1995 perfect, thanks!

@danyalahmed1995
Copy link
Copy Markdown
Author

@paul-geoghegan Added the input box label in the latest commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants