Skip to content

[codemod] Improve styled and sx prop transformation#42598

Merged
siriwatknp merged 5 commits intomui:nextfrom
siriwatknp:codemod/improve-sx
Jun 11, 2024
Merged

[codemod] Improve styled and sx prop transformation#42598
siriwatknp merged 5 commits intomui:nextfrom
siriwatknp:codemod/improve-sx

Conversation

@siriwatknp
Copy link
Member

Please review the test cases


@siriwatknp siriwatknp added package: codemod Specific to codemod. v6.x type: bug It doesn't behave as expected. labels Jun 10, 2024
@siriwatknp siriwatknp requested a review from mnajdova June 10, 2024 06:31
@mui-bot
Copy link

mui-bot commented Jun 10, 2024

Netlify deploy preview

https://deploy-preview-42598--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 97eb2ac

Copy link
Member

@mnajdova mnajdova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Found just one problem, the rest looks good 👌

...theme.variants.plain.neutral,
variants: [{
props: {
pressed: 'false'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be 'aria-pressed'?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@siriwatknp siriwatknp requested a review from mnajdova June 11, 2024 03:44
Copy link
Member

@mnajdova mnajdova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for the fixes 👌

@siriwatknp siriwatknp enabled auto-merge (squash) June 11, 2024 10:45
@siriwatknp siriwatknp merged commit 2cc5193 into mui:next Jun 11, 2024
mnajdova pushed a commit to mnajdova/material-ui that referenced this pull request Jun 12, 2024
joserodolfofreitas pushed a commit to joserodolfofreitas/material-ui that referenced this pull request Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: codemod Specific to codemod. type: bug It doesn't behave as expected. v6.x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants