Fixed
Status Update
Comments
ma...@google.com <ma...@google.com> #2
al...@gmail.com <al...@gmail.com> #3
Thanks for the report!
nj...@google.com <nj...@google.com>
ap...@google.com <ap...@google.com> #4
The release notes documentation has been edited to clarify this change in behavior for line height.
To support non-standard text sizes, we encourage users to follow the Material design system and use a different style = LocalTextStyle.current.copy(lineHeight = TextUnit.Unspecified)
, or create a custom Typography
entirely.
ap...@google.com <ap...@google.com> #5
deleted
Description
When using Icon.kt, I personally would expect that passing
Color.Unspecified
as the tint would result in no tint being applied to the icon at all (in other words, anull
color filter would be passed to thePainter
and the colors that are rendered would just be whatever theVectorAsset
orImageAsset
internally draws).This is currently not the case, as passing
Color.Unspecified
results in a transparent color filter being applied (meaning the icon is not rendered at all).This behavior is particularly important to support multi-colored icons, where you need to avoid a tint being applied all together.
The code in question at the time this bug was filed is here:https://cs.android.com/androidx/platform/frameworks/support/+/androidx-master-dev:compose/material/material/src/commonMain/kotlin/androidx/compose/material/Icon.kt;l=96;drc=989981f1b2a64419ed599cb82e2c589b6b41e46d