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] Replace row
prop with orientation
prop in all Joy UI components
#35721
Conversation
row
prop to orientation
in all Joy UI componentsrow
prop with orientation
prop in all Joy UI components
row
prop with orientation
prop in all Joy UI componentsrow
prop with orientation
prop in all Joy UI components
@mui/joy: parsed: +0.17% , gzip: +0.05% |
row
prop with orientation
prop in all Joy UI componentsrow
prop with orientation
prop in all Joy UI components
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.
@hbjORbj Can you check Argos change again? I look wrong to me.
059e0a3
to
ee92b12
Compare
2347621
to
dcc8efd
Compare
dcc8efd
to
dac5ee3
Compare
a1b9ca8
to
dac5ee3
Compare
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.
👍 Fixed the visual regression.
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.
@hbjORbj Please recheck the docs and replace row
with orientation, e.g. in https://deploy-preview-35721--material-ui.netlify.app/joy-ui/react-list/#horizontal-list.
Can't understand why I couldn't spot them. Thanks Thanks :)
Done! |
This PR includes:
row
prop (of boolean type) withorientation
prop (of type 'horizontal' or 'vertical') inCard
,List
,RadioGroup
and some internal components relevant toRadioGroup
row
prop withorientation
prop in instances of components in Joy demosorientation
prop to have eithervertical
orhorizontal
classname