Current incorrect behavior

The painterResource() currently only supports inflating android:tint if it references a simple COLOR_INT resource type, meaning that ColorStateLists or theme attributes pointing to ColorStateLists are simply ignored (link to source code).

Desired correct behavior

android:tint should inflate the color state list using TypedArrayUtils#getNamedColor() instead, which would default to the ColorStateList's default value (since Compose does not have any understanding of drawable states used with Views).

This behavior would mirror the way that ColorStateLists are currently handled in a <path>'s android:fillColor and android:strokeColor, both of which fallback to the ColorStateList's default color (they inflate using TypedArrayUtils#getNamedComplexColor() here, which returns a ComplexColor wrapper around a ColorStateList. Then when the vector is rendered to the display, a simple SolidColor using the color state list's default color is returned here).

This would closely mirror the way that VectorDrawableCompat is inflated, which uses TypedArrayUtils#getNamedColorStateList() (source code here).

Reasons for fixing the bug

android:fillColor and android:strokeColor currently support color state lists but android:tint does not. Ideally they should both be supported if possible.
Many teams adopting Compose don't have an Icon solution as elegant as Material's (which generates ImageVectors as part of a separate artifact automatically). For example, our team, plans to continue inflating icons from our existing icon library defined in XML resources while adopting Compose, and all of these resources currently declare a default tint color using android:tint.

Version used: Compose v1.0.1

Issue 195668138

Links (5)

Issue metadata