Skip to content

feat(list-header): support dynamic type - #27938

Merged
brandyscarney merged 4 commits into
FW-4146from
FW-4434
Aug 8, 2023
Merged

feat(list-header): support dynamic type#27938
brandyscarney merged 4 commits into
FW-4146from
FW-4434

Conversation

@brandyscarney

@brandyscarney brandyscarney commented Aug 4, 2023

Copy link
Copy Markdown
Member

Issue number: N/A


What is the current behavior?

List headers do not support dynamic type.

What is the new behavior?

  • Adds support for dynamic type by converting font-size to use rem units
  • Adds max font size for ios
  • Adds test for scaling the font which checks a list header at 310%

Does this introduce a breaking change?

  • Yes
  • No

Other information

Native

I used the App Store for comparison

80% 100% 135% 310%
IMG_3171 IMG_3172 IMG_3173 IMG_3174

Ionic Framework

80% 100% 135% 310%
IMG_3305 IMG_3306 IMG_3307 IMG_3308
IMG_3309 IMG_3310 IMG_3311 IMG_3312

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@brandyscarney
brandyscarney changed the base branch from main to FW-4146 August 4, 2023 21:13
@github-actions github-actions Bot added the package: core @ionic/core package label Aug 4, 2023
@brandyscarney
brandyscarney marked this pull request as ready for review August 4, 2023 21:56
@brandyscarney
brandyscarney requested review from a team and liamdebeasi and removed request for a team August 4, 2023 21:57

@liamdebeasi liamdebeasi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does this branch have the latest button/buttons work for dynamic type? Large text sizes on MD get cut off
image

@brandyscarney

Copy link
Copy Markdown
Member Author

It is branched off FW-4146 so yes it has the button/buttons dynamic type work. If you're talking about the min-height changes for button that was done for v7.2 and it looks like FW-4146 hasn't been updated from main since v7.0.14 so those changes wouldn't be in here.

@liamdebeasi liamdebeasi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great work!

@brandyscarney
brandyscarney merged commit b37121d into FW-4146 Aug 8, 2023
@brandyscarney
brandyscarney deleted the FW-4434 branch August 8, 2023 15:22
sean-perkins pushed a commit that referenced this pull request Aug 18, 2023
Adds support for dynamic type by converting `font-size` to use `rem` units and setting a max `font-size` which matches native iOS.
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