Skip to content

Conversation

@siriwatknp
Copy link
Member

@siriwatknp siriwatknp commented Aug 5, 2024

Found this issue from mui/material-ui#43065 (review)

It's not possible to do this at the moment:

The style attribute is not supported in PigmentGrid

image

@siriwatknp siriwatknp requested a review from brijeshb42 August 5, 2024 12:59
@siriwatknp siriwatknp added typescript type: bug It doesn't behave as expected. labels Aug 5, 2024
className?: string;
};

declare const Stack: PolymorphicComponent<StackBaseProps>;
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add a typecheck test for the change if possible ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

@siriwatknp siriwatknp changed the title [TypeScript] Fix missing style prop from layout components [TypeScript] Fix sx types and missing style prop from layout components Aug 5, 2024
@siriwatknp siriwatknp requested a review from brijeshb42 August 5, 2024 13:25
@siriwatknp siriwatknp merged commit 4576df1 into master Aug 6, 2024
@siriwatknp siriwatknp deleted the fix/layout-component-types branch August 6, 2024 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug It doesn't behave as expected. typescript

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants