Skip to content

fix: remove specifying aria-label#583

Merged
chrisrolfe198 merged 1 commit intomainfrom
fix/remove-duplicate-aria-label
Apr 8, 2026
Merged

fix: remove specifying aria-label#583
chrisrolfe198 merged 1 commit intomainfrom
fix/remove-duplicate-aria-label

Conversation

@chrisrolfe198
Copy link
Copy Markdown
Contributor

What

Remove the aria-label from the combobox

Media

Before:
Screenshot 2026-04-07 at 16 08 02

After:
Screenshot 2026-04-07 at 16 07 50

Why

aria-label and aria-labelledby stack for good reasons but in our case this lead to react-aria marking the label that it creates as aria-labelledby so we end up with a double message, see the screenshots above.

How

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Apr 7, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@freenow/wave@583

commit: c0c2e38

@chrisrolfe198 chrisrolfe198 merged commit 0648f79 into main Apr 8, 2026
8 checks passed
@chrisrolfe198 chrisrolfe198 deleted the fix/remove-duplicate-aria-label branch April 8, 2026 09:05
github-actions bot pushed a commit that referenced this pull request Apr 8, 2026
## <small>2.56.1 (2026-04-08)</small>

* fix: remove specifying aria-label (#583) ([0648f79](0648f79)), closes [#583](#583)
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

🎉 This PR is included in version 2.56.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Development

Successfully merging this pull request may close these issues.

2 participants