Expanded rows should be controllable

[icflorescu]

As requested here.

Gist:

The rowExpansion property object should also include:

• records → currently expanded records
• onChange → callback fired when currently expanded records change: onChange: (records: T[]) => void

[otobot1]

@icflorescu Have you had a chance to work on this issue yet? I can take a crack at it if you haven't.

[icflorescu]

@otobot1 Not yet, but I will over the weekend.

[icflorescu]

Done in v1.7.9. Can be used like this:

<DataTable
  // ...
  rowExpansion={{
    // ...
    expanded: {
      recordIds: expandedRecordIds,
      onRecordIdsChange: setExpandedRecordIds,
    },
  }}
/>

Full working example in Controlled mode section of this page.

[icflorescu]

@otobot1 if you're curious, this is handled mostly in the useRowExpansion hook here.

[otobot1]

@otobot1 if you're curious, this is handled mostly in the useRowExpansion hook here.

I'll check it out!