-
Notifications
You must be signed in to change notification settings - Fork 40
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
Use u-tabs under the hood of Tabs? #2449
Comments
Lets investigate and make a POC PR we can look at. Aside from finding in #2355 it would be good to document other benefits or lack of support in the current implementation. |
Quick example (not implemented controlled state): #2494 Pros:
Cons:
|
awaiting #2964 for further work on this |
Our own implementation of tabs is now feature equivalent to Our implementation of a tabs component has the same DOM structure as |
2025.01.25 @Barsnes @mimarz @eirikbacker Moving this to the backlog, as we might move to PR #2944 had a few issues with rendering in chromatic visual tests. I will close that PR. |
This would ensure good accessibility, provide a go-to-option for non-React-users, as well as remove the need for manually setting
value
on all tabs subcomponentsThe text was updated successfully, but these errors were encountered: