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

allow setting aria-current on menuitem with href #7587

Open
stefanprobst opened this issue Jan 9, 2025 · 0 comments
Open

allow setting aria-current on menuitem with href #7587

stefanprobst opened this issue Jan 9, 2025 · 0 comments

Comments

@stefanprobst
Copy link
Contributor

Provide a general summary of the issue here

menu items can have a href, but currently don't allow setting aria-current.

the apg does mention "aria-current" for "menuitem": https://www.w3.org/WAI/ARIA/apg/patterns/menubar/examples/menubar-navigation/

🤔 Expected Behavior?

aria-current should be forwarded to the <a role="menuitem" href="/" aria-current="page">

😯 Current Behavior

aria-current is not forwarded to the <a role="menuitem" href="/">

💁 Possible Solution

No response

🔦 Context

No response

🖥️ Steps to Reproduce

https://codesandbox.io/p/sandbox/awesome-brown-krr5tq?file=%2Fsrc%2FApp.js%3A11%2C32

Version

latest

What browsers are you seeing the problem on?

Firefox

If other, please specify.

No response

What operating system are you using?

linux

🧢 Your Company/Team

No response

🕷 Tracking Issue

No response

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

No branches or pull requests

1 participant