Skip to content

fix(remix): data table review follow-ups - #111

Open
leoafarias wants to merge 1 commit into
mainfrom
fix/data-table-followups
Open

fix(remix): data table review follow-ups#111
leoafarias wants to merge 1 commit into
mainfrom
fix/data-table-followups

Conversation

@leoafarias

Copy link
Copy Markdown
Member

Follow-up to #109.

Fixes

  • Preserve StyleProvider-inherited DataTable stylers during per-row widget-state resolution.
  • Clear the row-index hover state when row content changes so unmounted regions cannot leak hover styling.
  • Keep the pagination footer outside the horizontal table scroller, following Flutter's PaginatedDataTable sibling-footer precedent.
  • Assert that a paginated rows list contains no more than the visible page size.
  • Document that raw lastBodyRow specs replace rather than merge with bodyRow.
  • Restore alphabetical ordering for the Remix README component list.

Intentionally out of scope

  • Existing footer-row overflow at extremely narrow viewports, where the Spacer collapses first.
  • The empty surface remains inside the horizontally scrolling table content next to the header, per its documented contract.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant