Skip to content

Combobox role now works with button element #38535

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

Merged
merged 7 commits into from
Jul 16, 2025

Conversation

dbarabashh
Copy link
Contributor

@dbarabashh dbarabashh commented Mar 8, 2025

Update combobox role docs to clarify input/button usage and improve keyboard interaction details. Fixes #36436

@dbarabashh dbarabashh requested a review from a team as a code owner March 8, 2025 23:04
@dbarabashh dbarabashh requested review from scottaohara and removed request for a team March 8, 2025 23:04
@github-actions github-actions bot added Content:Accessibility Accessibility docs size/m [PR only] 51-500 LoC changed labels Mar 8, 2025
Copy link
Contributor

github-actions bot commented Mar 8, 2025

@bsmth
Copy link
Member

bsmth commented Jun 23, 2025

hi @scottaohara - some changes here if you want to have a look 👀

Let us know if you need someone else to review instead. Thanks!

@github-actions github-actions bot added the merge conflicts 🚧 [PR only] label Jun 23, 2025
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@bsmth bsmth self-requested a review July 10, 2025 10:50
@github-actions github-actions bot removed the merge conflicts 🚧 [PR only] label Jul 10, 2025
bsmth and others added 2 commits July 10, 2025 16:03
…le/index.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…le/index.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

github-actions bot commented Jul 10, 2025

Preview URLs

(comment last updated: 2025-07-16 10:37:58)

@github-actions github-actions bot added size/s [PR only] 6-50 LoC changed and removed size/m [PR only] 51-500 LoC changed labels Jul 10, 2025
Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

I've some comments for you to have a look at, but this looks really good from my side. @scottaohara do you want to have a look through to check that everything's okay?

Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Thanks a lot! Let's avoid this PR going stale and merge the improvements! Well done 👏🏻

@bsmth bsmth merged commit d86426e into mdn:main Jul 16, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Accessibility Accessibility docs size/s [PR only] 6-50 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Combobox role now works with button element
2 participants