Skip to content

Add MRT loading and row intent helpers - #350

Open
zoharma wants to merge 1 commit into
mainfrom
zma/mrt-loading-row-intent
Open

zoharma wants to merge 1 commit into
mainfrom
zma/mrt-loading-row-intent

Conversation

@zoharma

@zoharma zoharma commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Adds shared MRT helpers to DiamondDSIntegrations, with docs.

  • Loading state: mrtLoadingState(loading, data) shows skeleton rows on first load and a progress bar on refetch, never both. An empty array counts as no data.
  • Skeleton-safe styling: skipWhileSkeleton(callback) skips row, cell and checkbox style callbacks while skeletons show, so placeholder rows aren't styled.
  • Row emphasis: mrtRowIntent(intent) tints a row with the intent's container colour and keeps hover and selected states visible. Use it sparingly and always with a status chip or icon.

Also adds a Loading section and a Row emphasis section to the MRT guidance, a Row emphasis section to the Table guidance, a WarningRow story, and a warning row in the dev app.

Skeleton (as seen on i15-1)
image

Warning row (as seen on Dev App)
image

- Add `mrtLoadingState` and `skipWhileSkeleton` to `DiamondDSIntegrations`
- Add `mrtRowIntent` to tint rows with an intent's container colour, keeping hover and selected states visible
- Document loading and row emphasis for MRT and plain tables
- Show a warning row paired with a status chip in the Table story and dev app
@zoharma
zoharma requested a review from a team October 9, 2026 16:10
@zoharma zoharma added this to the 0.9.0 milestone Oct 9, 2026

This branch has not been deployed

No deployments
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