Skip to content

Conversation

alexzhirkevich
Copy link

Adapt Android non-linear font scaling and map Dynamic Type HIG to font scaling tables on IOS

Fixes https://youtrack.jetbrains.com/issue/CMP-738/Support-non-linear-text-scale-for-other-platforms

Testing

FontScalingTest.uikit.kt

This should be tested by QA

Release Notes

Features - iOS

  • Default Density now performs non-linear font scaling and conforms to HIG

@igordmn igordmn requested a review from mazunin-v-jb February 21, 2025 21:12
Copy link

@ASalavei ASalavei left a comment

Choose a reason for hiding this comment

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

LDTM with 2 minor comments.
Thank you for the contribution!

@ASalavei ASalavei requested a review from igordmn March 7, 2025 20:42
@ASalavei
Copy link

ASalavei commented Mar 7, 2025

@igordmn , since there are changes in Desktop API, we need your approval.

}

public abstract interface class androidx/compose/ui/unit/Density : androidx/compose/ui/unit/FontScalingLinear {
public abstract interface class androidx/compose/ui/unit/Density : androidx/compose/ui/unit/FontScaling {
Copy link
Member

Choose a reason for hiding this comment

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

Isn't it a breaking change (for binary compatibility)?

cc @igordmn

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants