Skip to content

refactor: align List tokens, spacing and typography with MD3 - #5041

Open
adam-sajko wants to merge 3 commits into
callstack:mainfrom
adam-sajko:feat/list-md3-tokens-spacing
Open

refactor: align List tokens, spacing and typography with MD3#5041
adam-sajko wants to merge 3 commits into
callstack:mainfrom
adam-sajko:feat/list-md3-tokens-spacing

Conversation

@adam-sajko

@adam-sajkoadam-sajko commented Aug 7, 2026

Copy link
Copy Markdown

Motivation

List.Item and List.Accordion now follow the MD3 list spec for padding, type, and colour instead of hardcoded sizes and the old expanded primary tint.

Visual changes

  • One-line rows: 16dp vertical padding (was 14). Height 56dp.
  • Two-line rows: 14dp vertical padding. Height 72dp.
  • Three-line rows: 12dp vertical padding when the description wraps. Height 88dp.
  • Title is bodyLarge, supporting text is bodyMedium (line height 24 / 20).
  • Trailing inset 16dp (was 24). Checkboxes and chevrons sit 8dp closer to the edge.
  • Expanded accordion title and leading icon stay onSurface / onSurfaceVariant (no primary tint).

Related issue

Related to #4976

Test plan

  • Unit tests for the theme override reaching title/description typography
  • Unit tests for 12dp padding when the description wraps
  • Snapshots updated on purpose (padding, type, trailing inset)

@adam-sajko
adam-sajko marked this pull request as ready for review August 18, 2026 08:26

@MikitasKMikitasK left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good job 👏 overall, this refactoring looks solid 👍
just a few thing to consider before merge:

Comment threadsrc/components/List/ListItem.tsx
Comment threadsrc/components/List/ListItem.tsx
Comment threadsrc/components/List/ListAccordion.tsx
Comment threadsrc/components/List/ListItem.tsx Outdated
Comment threadsrc/components/List/ListAccordion.tsx Outdated

@MikitasKMikitasK left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ✅

@adam-sajko
adam-sajkoforce-pushed the feat/list-md3-tokens-spacing branch from 8e2f57e to b81344fCompareAugust 24, 2026 09:48
Sign up for freeto 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.

2 participants

@adam-sajko@MikitasK