-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
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
[joy-ui][Accordion] Expanding breaks the AccordionDetails children tab index #40116
Comments
Why do you need inputs inside the AccordionDetails? I think I have never seen that. What is your use case? Please provide a CodeSandbox reproducing the issue. |
See e.g. this example of the Joy UI documentation. Furthermore, this would be the recommended behaviour according to the ARIA Authoring Practices Guide.
|
The issue seems to be, that certain elements ( When I explicitly set a |
The solution of @basti4242 only seems to work, if the accordion is initially expanded ( joy-ui-accordion-tab-index-bug.movIt would be beneficial to add two more test for these cases.
|
Took a look at this because I also ran into this problem. I'm pretty sure this line needs to have the The portion that adds the The other issue with this partial fix is that elements with an explicit tab-index in a default expanded details will get their tab index cleared. It would probably be better to use an explicit |
If no one is working can I take this up. |
This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue. Note We value your feedback @vanessag! How was your experience with our support team? |
Duplicates
Latest version
Steps to reproduce 🕹
Steps:
Current behavior 😯
When I click the Accordion to expand it and then click on the first Input within my AccordionDetails then press tab it does not focus the next Input.
Expected behavior 🤔
I expect that when I click on the Accordion to expand it and then click on an Input that is inside of AccordionDetails that pressing tab goes to the next Input.
Context 🔦
I am trying to accomplish an Accordion component that has form elements inside of it. These are advanced settings that the user can expand/collapse.
Your environment 🌎
npx @mui/envinfo
The text was updated successfully, but these errors were encountered: