-
-
Notifications
You must be signed in to change notification settings - Fork 159
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
Add enableCellHoverReveal column option #429
Add enableCellHoverReveal column option #429
Conversation
@laratran is attempting to deploy a commit to the Kevin Vandy OSS Team on Vercel. A member of the Team first needs to authorize it. |
ac34389
to
c9855bc
Compare
LGTM, even works with virtualization! However, could you please 1) Add a storybook example for this? 2) Add the limitations somewhere in the docs? (Maybe in the props description) cc @posabsolute |
Also please rebase with the latest v2 as I have merged some stuff |
7622f57
to
705005e
Compare
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
705005e
to
198b71e
Compare
Great, thanks! |
Usage
basic table
highlightOnHover: false
striped: true
Limitations
passing props through mantineTableBodyCellProps
Passing some props through mantineTableBodyCellProps might not work as expected.
For example, passing
align:'right'
to mantineTableBodyCellProps is not applied on cells that have the hover reveal enabledBelow, all cells are right aligned except for the ones that have overflow.