-
-
Notifications
You must be signed in to change notification settings - Fork 31.7k
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
[docs-infra] Fix Base UI API tabs layout regresions #41122
Comments
About the last item on the list: I'm assuming we can only fix this if we change the approach from having the Tab render an |
@danilo-leal I disagree, using links with tabs looks common to me. The closer the tabs is to the top of the page, the more likely its state should be stored in the URL. For example, we need to change from button to link in https://github.com/mui/mui-private/issues/392 for the tabs of the store product pages. Or on this very page, the top is a list of tabs with links: https://primer.style/components/tab-nav/react/alpha. Or on https://hub.docker.com/repositories/, looks like they used the tabs as link |
That's true, okay. But I'm more curious whether that's part of the problem with the wrong aria-role and keyboard navigation. For example, The Material UI tabs component — which is what we're using for the page's tabs — seems to be fine on these two aspects if the Tab is rendered as a button. |
However, maybe the GitHub example is not the best as they don't seem to use |
@danilo-leal I suspect an issue with Base UI Tabs component. |
Yup, but those things are happening to the Material UI Tabs; it's what the page uses, so maybe there's a bug there, too. |
Problem
#35938 introduced a number of UX problems that we didn't have before:
Context
As of today, I don't think that the tabs experience yields a stronger UX. We know it varies from person to person, so only talking for myself, it's not there.
I think that fixing these UX regressions would make it to the point where maybe it would be a superior UX for most people. I could understand why.
Your environment
npx @mui/envinfo
Search keywords: Base UI API tabs layout regresions
The text was updated successfully, but these errors were encountered: