Skip to content
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

[Grid] size proptype to accept grow and auto values #379

Merged
merged 2 commits into from
Feb 19, 2025

Conversation

ablagoev
Copy link
Contributor

@ablagoev ablagoev commented Feb 11, 2025

Closes #323

@ablagoev ablagoev changed the title fix: type Grid size proptype to accep grow and auto values (#323) fix: type Grid size proptype to accept grow and auto values (#323) Feb 11, 2025
@zannager zannager added the component: Grid The React component. label Feb 12, 2025
@zannager zannager requested a review from DiegoAndai February 12, 2025 12:36
Copy link
Member

@DiegoAndai DiegoAndai left a comment

Choose a reason for hiding this comment

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

Thanks for working on this @ablagoev!

May I ask you to run this command: pnpm test:update and check if there are any updates? If there are and they are related to this change, please commit them. If they're not related, let me know so I can take a look.

@ablagoev
Copy link
Contributor Author

Hey,

Yes, I am sorry, I've missed this step in the process. I've updated the tests and squashed the commit.

@DiegoAndai DiegoAndai changed the title fix: type Grid size proptype to accept grow and auto values (#323) [material-ui][Grid2] size proptype to accept grow and auto values Feb 19, 2025
@DiegoAndai DiegoAndai changed the title [material-ui][Grid2] size proptype to accept grow and auto values [Grid] size proptype to accept grow and auto values Feb 19, 2025
@DiegoAndai DiegoAndai merged commit 2992662 into mui:master Feb 19, 2025
12 checks passed
@DiegoAndai
Copy link
Member

Thanks @ablagoev!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: Grid The React component.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Grid] Difference between prop type of the size prop and the typescript definition for string values
3 participants