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
MBS-10365 (V): Convert entity lists to react-table #1414
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works nicely. Just commented about tracking changes in second commit. See other PR for first commit.
c636b17
to
2915370
Compare
2915370
to
6f926e9
Compare
You might want to cherry-pick yvanzo@2fd4a18 that patches |
This allows passing custom props to headers, so we can actually pass aria-label (and style) to the header for merge removals.
601898c
to
07ba609
Compare
* `accessor` is mandatory in options for column with value, e.g., it can be used to sort non-paginated tables later on. * `Cell` defaults to returning `String(accessor(row))`. See https://github.com/tannerlinsley/react-table/blob/v7.0.0/docs/api/useTable.md#column-options
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
On top of #1413
MBS-10365
Artist (the only one left) is a bit more annoying since the component is currently being shared between search results and normal lists. We might want to split that one for now, until we can figure out how to easily do search results with react-table.