Skip to content

Conversation

liamdebeasi
Copy link
Contributor

@liamdebeasi liamdebeasi commented Sep 25, 2023

Issue number: N/A


What is the current behavior?

When testing Dynamic Font Scaling with a custom font I noticed that the large title does not respect --ion-font-family on transition.

What is the new behavior?

  • The cloned large title now respect --ion-font-family

Note: This happens in main too which is why I am merging into there instead of the Dynamic Font Scaling branch.

main branch
IMG_0182 IMG_0183

Does this introduce a breaking change?

  • Yes
  • No

Other information

@github-actions github-actions bot added the package: core @ionic/core package label Sep 25, 2023
@liamdebeasi liamdebeasi changed the base branch from FW-4146 to main September 25, 2023 19:17
@liamdebeasi liamdebeasi force-pushed the title-clone-custom-font branch from a8bd740 to 1fca7fc Compare September 25, 2023 19:19
@liamdebeasi liamdebeasi marked this pull request as ready for review September 25, 2023 19:35
@liamdebeasi liamdebeasi requested review from a team and thetaPC and removed request for a team September 25, 2023 19:35
Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

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

LGTM

@liamdebeasi liamdebeasi added this pull request to the merge queue Sep 26, 2023
Merged via the queue into main with commit 71a7af0 Sep 26, 2023
@liamdebeasi liamdebeasi deleted the title-clone-custom-font branch September 26, 2023 16:31
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.

2 participants