Skip to content

DataGrid Component issues #2140

Discussion options

You must be logged in to vote

Hey @vignesh-sivaprakasam -

Currently onClick is not supported on the cell because it is used for cell focus, for keyboard support. The current suggested approach is for the element within the cell to have the onClick, but if you could share more about what you're trying to achieve, we can discuss other options if needed.

Snapshot testing works best with deterministic renders, and the DataGrid is non-deterministic due to the ID generation. Could you share more about why you're hoping to snapshot test the DataGrid itself? The component is fully tested in Paste, so it may not be necessary for you to test it depending on your use case.

Replies: 2 comments 2 replies

Comment options

You must be logged in to vote
0 replies
Comment options

You must be logged in to vote
2 replies
@andioneto
Comment options

@vignesh-sivaprakasam
Comment options

Answer selected by vignesh-sivaprakasam
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Category
Q&A
Labels
None yet
3 participants