Skip to content

feat(note): sync the ionic theme with the design kit - #31742

Merged
thetaPC merged 8 commits into
FW-6855-md-syncfrom
FW-6855-ionic-sync
Nov 13, 2026
Merged

thetaPC merged 8 commits into
FW-6855-md-syncfrom
FW-6855-ionic-sync

Conversation

@thetaPC

@thetaPC thetaPC commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

Issue number: internal


What is the current behavior?

The ionic note inherits its line height and letter spacing.

What is the new behavior?

The ionic note matches the info text of a list item in the ionic theme's design kit (body/md/regular):

  • 1.5rem (24px) line height
  • 0 letter spacing

The 14px font size and regular weight already match.

Does this introduce a breaking change?

  • Yes
  • No

The ionic theme hasn't been released.

Other information

Known gaps

  • The kit's info text color (#6c727a) isn't in the palette, so the note keeps its current color. Tracked as a follow-up with the palette generator
  • Centering the note in the ionic item divider is tracked as a follow-up to sync the item divider

How to test

Check Before After What to look for
ionic Before After Nothing moves. Inspect a note: line-height is 24px and letter-spacing is 0
ionic, large text Same links In devtools, add font-size: 310% to html. Rows with a note are slightly taller after, because the note's line height scales with the text

Only the ionic font scaling snapshot changes. At 100% the note fits within the item's minimum height, so nothing else moves.

@thetaPC
thetaPC added this pull request to stack #31730 November 12, 2026 00:21
@vercel

vercel Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
ionic-framework Ready Ready Preview Nov 12, 2026 11:47pm UTC

Request Review

@thetaPC
thetaPC marked this pull request as ready for review November 12, 2026 03:42
@thetaPC
thetaPC requested a review from a team as a code owner November 12, 2026 03:42
@thetaPC
thetaPC requested a review from ShaneK November 12, 2026 03:42

@ShaneK ShaneK left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! 🚀

@thetaPC
thetaPC merged commit f712750 into ionic-modular Nov 13, 2026
54 checks passed
@thetaPC
thetaPC deleted the FW-6855-ionic-sync branch November 13, 2026 00:03

This branch was successfully deployed

1 active deployment
Preview — e0850dcc Deployed Nov 12, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: core @ionic/core package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants