https://kotlinlang.org logo
#compose
Title
# compose
s

Siyamed

03/11/2022, 5:02 PM
Hi all, regarding includeFontPadding related changes b/171394808 in Version 1.2.0-alpha05, we’d love to hear your opinions and the impact on you. If you can give it a try better-er. Please let us know. aosp/1993911 has some screenshot updates showing the difference (it is mostly subtle). This prevents users to do things such as this b/171394808#comment32
m

Marcin Wisniowski

03/11/2022, 8:57 PM
The link you posted is not public.
👍 1
a

Albert Chang

03/12/2022, 11:29 AM
One problem I noticed is that the line heights of western scripts and CJK scripts (or CJK-western mixed scripts) become different, which seems somewhat wired.
OK it seems that if I set
lineHeight
to
TextUnit.Unspecified
in
TextStyle
the line heights become the same. Not sure why specifying a line height conversely makes them different. (This problem will be more obvious in M3 as in M3 all default styles have an explicit line height.)
s

Siyamed

03/12/2022, 3:32 PM
Thanks Albert, observed it. (there was a discrepancy with the first line’s line height between on/off which we fixed, but that seems to be causing the problem you mentioned)
c

Colton Idle

03/13/2022, 4:49 AM
For me everything seems pretty good and didn't even notice a difference honestly. lol
❤️ 1
s

Siyamed

03/14/2022, 4:52 AM
regarding line height, I checked it again after a hot fix. however it looks more about line height than includeFontPadding. the difference seems to be more visible with includeFontPadding but it is about line height not being applied to first/last lines on Android, and the line height / leading distribution for a line. b/181155707
That is something that we will check later (maybe soon)
👍 1