-
-
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] Fix double API page redirection #36743
[docs] Fix double API page redirection #36743
Conversation
/base/api/use-option/ /base/react-select/hooks-api/#use-option 301 | ||
/base/api/use-select/ /base/react-select/hooks-api/#use-select 301 | ||
/base/api/use-slider/ /base/react-slider/hooks-api/#use-slider 301 | ||
/base/api/use-snackbar/ /base/react-snackbar/hooks-api/#use-snackbar 301 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A side note: the previous URLs feel clearer, when sharing them with someone else.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree, but the new urls are at least understandable and the overall DX feels better.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mnajdova I wonder if Nevermind, we probably want to keep API in the pathname, and props would be too narrow./props/
and /hooks/
could work better for the URL, e.g. https://ark-ui.com/docs/react/components/popover/props. It's shorter and maybe clearer.
Netlify deploy previewBundle size report |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nica catch, thanks for creating the PR.
Signed-off-by: Marija Najdova <mnajdova@gmail.com>
"themeDefaultProps": true, | ||
"muiName": "JoyModalOverflow", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#36262 did not update with the latest master.
There is a double redirection from the existing API pages. https://deploy-preview-35938--material-ui.netlify.app/base/api/use-tabs-list/
It was introduced in #35938.