🐛 Fix broken rendering of migration waves table rows, properly pass isExpandable prop in useTableControlProps

[mturley]

In #1345, we changed the structure of the expandable rows in the Migration Waves table and caused the isExpanded prop to be present on all rows (which should only be on the expanded content rows), which caused all rows to be hidden because it was false. This PR restores the intended structure: Each row is wrapped in a <Tbody> with the isExpanded prop, which contains two <Tr>s: one for the row itself and one for the expanded content.

Also removes the cursor: "pointer" inline style on the Tbody which was causing the whole row to appear clickable even though only the expandable cells are clickable.

[codecov]

Codecov Report

All modified lines are covered by tests ✅

❗ No coverage uploaded for pull request base (main@4ab5428). Click here to learn what that means.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1398   +/-   ##
=======================================
  Coverage        ?   41.34%           
=======================================
  Files           ?      138           
  Lines           ?     4334           
  Branches        ?     1039           
=======================================
  Hits            ?     1792           
  Misses          ?     2454           
  Partials        ?       88           

Flag	Coverage Δ
client	41.34% <ø> (?)
server	∅ <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
...c/app/hooks/table-controls/useTableControlProps.ts	13.69% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.