Explore representing colors used by a TextField in a TextFieldColors interface [171305338]

Fixed

Bug

Status Update

No update yet.

Description

lp...@google.com

created issue #1

Oct 20, 2020 03:53PM

Following recent changes to other components (such as SwitchColors and ButtonElevation) we use type safe interfaces to represent parameters that change in different states.

Currently the color parameters on TextField are misleading, as they are modified a lot and used in different places, making it hard to customize parts of the text field, such as having a different error color for the cursor and label. Additionally, emphasis is applied internally meaning that there is no way to opt out / change the alpha applied to certain parts of the text field.

We could allow for easier customization and make the API surface more clear with something like:

interface TextFieldColors {
    fun cursorColor(isError: Boolean): Color

    fun labelColor(isError: Boolean): Color

    ...
}

Happy to help discuss this, but I think the first step is figuring out what all the combination of colors that are used in different states / places in the TextField are.

Comments

tc...@google.com <tc...@google.com> #2Feb 5, 2021 10:02PM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 23a7d960caf43390a554700d3c56ada189a9d10e
Author: Louis Pullen-Freilich <lpf@google.com>
Date: Mon Aug 10 15:11:36 2020

IconButton / IconToggleButton API scrub

Test: ./gradlew updateApi
Bug:

b/161809385

Bug:

b/161807956

Relnote: "Adds enabled parameter to IconButton, and reorders parameters in IconToggleButton"
Change-Id: I0a9419b1a631cadad451395302ad87b7f9214f96

M ui/ui-material/api/current.txt
M ui/ui-material/api/public_plus_experimental_current.txt
M ui/ui-material/api/restricted_current.txt
M ui/ui-material/src/commonMain/kotlin/androidx/compose/material/IconButton.kt

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

ap...@google.com <ap...@google.com> #3Feb 10, 2021 05:20PM

Marked as fixed.

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

commit 80b5d30138ac6afd462bdf7cb5b4e617d5258395
Author: Anastasia Soboleva <soboleva@google.com>
Date: Sun Feb 07 01:02:06 2021

Add TextFieldColors interface and TextFieldDefaults object.

Rename isErrorValue parameter of text fields into isError.

Fixes: 171305338
Fixes: 179256385
Fixes: 168004067
Test: material tests passed
Relnote: """
- Added TextFieldColors interface to represent different colors used in TextField and OutlinedTextField in different states. For defaut implementation see TextFieldDefaults.textFieldColors and TextFieldDefaults.outlinedTextFieldColors.
- Renamed isErrorValue parameter inside TextField and OutlinedTextField
into isError.
"""

Change-Id: I831f90c3be79452bcaa6dff41d1777298a23ca09

M compose/material/material/api/current.txt
M compose/material/material/api/public_plus_experimental_current.txt
M compose/material/material/api/restricted_current.txt
M compose/material/material/integration-tests/material-demos/src/main/java/androidx/compose/material/demos/MaterialTextField.kt
M compose/material/material/samples/src/main/java/androidx/compose/material/samples/TextFieldSamples.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/textfield/OutlinedTextFieldScreenshotTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/textfield/OutlinedTextFieldTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/textfield/TextFieldScreenshotTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/textfield/TextFieldTest.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/OutlinedTextField.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/TextField.kt
A compose/material/material/src/commonMain/kotlin/androidx/compose/material/TextFieldDefaults.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/TextFieldImpl.kt

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

Issue 171305338

Description

Issue summary

Comments

tc...@google.com <tc...@google.com> #2Feb 5, 2021 10:02PM

ap...@google.com <ap...@google.com> #3Feb 10, 2021 05:20PM

Add comment

Issue metadata