Skip to content

fix(app): add aria-labels to titlebar and sidebar buttons#9843

Merged
adamdotdevin merged 1 commit intoanomalyco:devfrom
ndarilek:label-title-screen-buttons
Jan 21, 2026
Merged

fix(app): add aria-labels to titlebar and sidebar buttons#9843
adamdotdevin merged 1 commit intoanomalyco:devfrom
ndarilek:label-title-screen-buttons

Conversation

@ndarilek
Copy link
Copy Markdown
Contributor

@ndarilek ndarilek commented Jan 21, 2026

Add accessibility labels for screen readers:

  • Sidebar toggle button with aria-expanded state
  • Settings and Help buttons in sidebar
  • More options dropdown trigger
  • New session button

Related to #8702 (adds button labels; other accessibility issues in that issue remain)

Add accessibility labels for screen readers:
- Sidebar toggle button with aria-expanded state
- Settings and Help buttons in sidebar  
- More options dropdown trigger
- New session button

Partially addresses anomalyco#8702
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions
Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

No duplicate PRs found

@adamdotdevin adamdotdevin merged commit 0059fdc into anomalyco:dev Jan 21, 2026
3 checks passed
@ndarilek
Copy link
Copy Markdown
Contributor Author

Quick clarifying question: how strict is the "existing issue" guideline which the bot flagged?

I'm a blind screen reader user. My intent was to just try using OpenCode and have Claude Code fix accessibility issues as I find them. I can open issues first if needed, but it will definitely slow things down if the first thing I have to do prior to every change is opening an issue that, say, a role or label is missing, then fix that issue in a follow-up PR. I don't have any intentions of changing the visual design, just adding roles and labels changing non-visual aspects for screen reader users.

Thanks!

@ndarilek ndarilek deleted the label-title-screen-buttons branch January 21, 2026 19:02
sergiu-svinarciuc pushed a commit to sergiu-svinarciuc/opencode that referenced this pull request Jan 22, 2026
* dev: (345 commits)
  fix after merge
  tweak(session title): change prompt to have the response with user language (anomalyco#9847)
  fix(app): new workspace button with all languages (anomalyco#9848)
  chore: generate
  fix(app): add aria-labels to titlebar and sidebar buttons (anomalyco#9843)
  test(app): fix e2e
  wip: black
  fix: type error
  fix(app): breaking out of auto-scroll
  chore: generate
  feat(app): model tooltip metadata in chooser (per Figma request) (anomalyco#9707)
  chore: generate
  fix(app): workspaces padding wonkiness (anomalyco#9772)
  feat(desktop): properly integrate window controls on windows (anomalyco#9835)
  test(app): fix e2e
  fix(app): terminal improvements - focus, rename, error state, CSP (anomalyco#9700)
  fix(app): japanese language support
  fix(desktop): add workaround for nushell
  test(app): fix e2e
  fix(core): snapshot regression
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants