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
[utils] Allow state prefix to be configurable #32972
Conversation
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.
The logic looks good 👍
…/classname-joy
…/classname-joy
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.
Do we need a guide for Joy similar to https://mui.com/material-ui/customization/how-to-customize/#state-classes?
Yep, good point. I have added to the backlog and will work on it in a separate PR. |
…/classname-joy
Currently, Joy component className contains
Mui
as a global state prefix, eg.Mui-focusVisible
.This PR allows Joy to customize the global state prefix to
Joy-focusVisible
,Joy-checked
, etc.The change does not affect Material UI components (the prefix is default as
Mui
)I will update all Joy components to haveDone!Joy
prefix after the review to make it easier for the reviewer.