You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I can't see the reason for this, but could well be missing something.
I noticed this because we have some cards that sit in a Grid and on hover have an overlay that can expand beyond the bounds of the grid. After upgrading to v7, this was clipped because of the overflow rule. As can be seen in the linked codesandbox, drop shadows on children are clipped as well.
Overriding the style works, so it's not a huge problem, but if there's no reason for it to be there I'm happy to submit a PR to remove.
What version of @mantine/* packages do you have in package.json? (Note that all @mantine/* packages must have the same version in order to work correctly)
7.2.2
If possible, please include a link to a codesandbox with a minimal reproduction
What package has an issue?
@mantine/core
Describe the bug
The
Grid
component now setsoverflow: hidden
on its root element, which it did not in v6.I can't see the reason for this, but could well be missing something.
I noticed this because we have some cards that sit in a
Grid
and on hover have an overlay that can expand beyond the bounds of the grid. After upgrading to v7, this was clipped because of the overflow rule. As can be seen in the linked codesandbox, drop shadows on children are clipped as well.Overriding the style works, so it's not a huge problem, but if there's no reason for it to be there I'm happy to submit a PR to remove.
What version of @mantine/* packages do you have in package.json? (Note that all @mantine/* packages must have the same version in order to work correctly)
7.2.2
If possible, please include a link to a codesandbox with a minimal reproduction
https://codesandbox.io/s/sharp-sunset-83cdkc
Do you know how to fix the issue
Yes
Do you want to send a pull request with a fix?
Yes
Possible fix
Remove the
overflow: hidden
rule.The text was updated successfully, but these errors were encountered: