painterResource() ignores ColorStateLists completely when inflating android:tint in VectorDrawables [195668138]

Fixed

Bug

Status Update

No update yet.

Description

al...@gmail.com

created issue #1

Aug 6, 2021 02:17AM

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

Comments

nj...@google.com <nj...@google.com> Aug 6, 2021 02:56AM

Assigned to nj...@google.com.

ap...@google.com <ap...@google.com> #2Aug 11, 2021 11:09PM

Marked as fixed.

Project: platform/frameworks/support
Branch: androidx-main

commit 0154910724cdc44253af1d2f8cede76264783226
Author: Aurimas Liutikas <aurimas@google.com>
Date: Thu Jun 27 15:25:02 2024

Expand native target support for annotation and collection libraries

- Enable watchos and tvos download in importMaven
- Add support for watchos and tvos targets in AndroidXMultiplatformExtension
- Enable watchos and tvos in :annotation:annotation
- Enable linuxArm64, watchos, and tvos in :collection:collection

This work is required as we work towards setting up native stubs for
compose projects.

Test: ./gradlew collection:collection:publish
Bug: 349894318
Change-Id: Idfd1faa3a826bb91ee14722f7437bdcf99cf0018

M annotation/annotation/build.gradle
M buildSrc/private/src/main/kotlin/androidx/build/AndroidXMultiplatformExtension.kt
M buildSrc/public/src/main/kotlin/androidx/build/KmpPlatforms.kt
M collection/collection/build.gradle
M development/build_log_simplifier/messages.ignore
M development/importMaven/src/main/kotlin/androidx/build/importMaven/KmpConfig.kt

https://android-review.googlesource.com/3151736

Issue 195668138