Fixed
Status Update
Comments
ni...@google.com <ni...@google.com> #2
Resetting assignee to `None` in preparation of the triage bash on Thursday June 17th. Please reach out to nickanthony@ if you have any questions.
ma...@google.com <ma...@google.com> #3
Assigning to Anastasia for prioritisation or further triage
so...@google.com <so...@google.com> #4
This could be added as a slot API into TextFieldDecorationBox.
Not a huge priority given that it could be achieved with something like this for example:
BasicTextField() { innerTextField ->
TextFieldDecorationBox(
innerTextField = {
Row {
val alignModifier = Modifier.alignByBaseline()
Text("$", alignModifier.padding(end = 4.dp)) // prefix
Box(alignModifier.weight(1f)) { innerTextField() }
Text(".00", alignModifier) // suffix
}
}
)
}
ni...@google.com <ni...@google.com> #5
Could you please check if this is on the correct iteration?
ma...@google.com <ma...@google.com>
ap...@google.com <ap...@google.com> #6
Project: platform/frameworks/support-golden
Branch: androidx-main
commit c15bd36a088423392f6d607d0dd1a951a0d31702
Author: Max Alfonso-Ying <maxying@google.com>
Date: Wed Jan 25 00:38:05 2023
Add goldens for text field prefix/suffix.
Bug: b/179884561
Change-Id: I6522629d366b1249fdfcd887e9e8402ee32c7df6
A compose/material3/material3/outlinedTextField_prefixSuffix_withLabelAndInput_cuttlefish.png
A compose/material3/material3/outlinedTextField_prefixSuffix_withLabelAndInput_darkTheme_cuttlefish.png
A compose/material3/material3/outlinedTextField_prefixSuffix_withLabelAndInput_focused_cuttlefish.png
A compose/material3/material3/outlinedTextField_prefixSuffix_withLeadingTrailingIcons_cuttlefish.png
A compose/material3/material3/outlinedTextField_prefixSuffix_withPlaceholder_cuttlefish.png
A compose/material3/material3/textField_prefixSuffix_withLabelAndInput_cuttlefish.png
A compose/material3/material3/textField_prefixSuffix_withLabelAndInput_darkTheme_cuttlefish.png
A compose/material3/material3/textField_prefixSuffix_withLabelAndInput_focused_cuttlefish.png
A compose/material3/material3/textField_prefixSuffix_withLeadingTrailingIcons_cuttlefish.png
A compose/material3/material3/textField_prefixSuffix_withPlaceholder_cuttlefish.png
https://android-review.googlesource.com/2402252
Branch: androidx-main
commit c15bd36a088423392f6d607d0dd1a951a0d31702
Author: Max Alfonso-Ying <maxying@google.com>
Date: Wed Jan 25 00:38:05 2023
Add goldens for text field prefix/suffix.
Bug:
Change-Id: I6522629d366b1249fdfcd887e9e8402ee32c7df6
A compose/material3/material3/outlinedTextField_prefixSuffix_withLabelAndInput_cuttlefish.png
A compose/material3/material3/outlinedTextField_prefixSuffix_withLabelAndInput_darkTheme_cuttlefish.png
A compose/material3/material3/outlinedTextField_prefixSuffix_withLabelAndInput_focused_cuttlefish.png
A compose/material3/material3/outlinedTextField_prefixSuffix_withLeadingTrailingIcons_cuttlefish.png
A compose/material3/material3/outlinedTextField_prefixSuffix_withPlaceholder_cuttlefish.png
A compose/material3/material3/textField_prefixSuffix_withLabelAndInput_cuttlefish.png
A compose/material3/material3/textField_prefixSuffix_withLabelAndInput_darkTheme_cuttlefish.png
A compose/material3/material3/textField_prefixSuffix_withLabelAndInput_focused_cuttlefish.png
A compose/material3/material3/textField_prefixSuffix_withLeadingTrailingIcons_cuttlefish.png
A compose/material3/material3/textField_prefixSuffix_withPlaceholder_cuttlefish.png
ap...@google.com <ap...@google.com> #7
Project: platform/frameworks/support
Branch: androidx-main
commit d747bed10677621f865067e57a6099c7179502c3
Author: Max Alfonso-Ying <maxying@google.com>
Date: Tue Jan 03 17:50:57 2023
Implement prefix/suffix for M3 text field.
Fixes: b/179884561
Test: added
Relnote: "Slot APIs for prefix and suffix text have been
added to Material 3 text fields."
Change-Id: Ia85786c582981e8f16a7053a869e06c2c1a43fb8
M compose/material3/material3/api/public_plus_experimental_current.txt
M compose/material3/material3/integration-tests/material3-catalog/src/main/java/androidx/compose/material3/catalog/library/model/Examples.kt
M compose/material3/material3/samples/src/main/java/androidx/compose/material3/samples/TextFieldSamples.kt
M compose/material3/material3/src/androidAndroidTest/kotlin/androidx/compose/material3/OutlinedTextFieldScreenshotTest.kt
M compose/material3/material3/src/androidAndroidTest/kotlin/androidx/compose/material3/OutlinedTextFieldTest.kt
M compose/material3/material3/src/androidAndroidTest/kotlin/androidx/compose/material3/TextFieldScreenshotTest.kt
M compose/material3/material3/src/androidAndroidTest/kotlin/androidx/compose/material3/TextFieldTest.kt
M compose/material3/material3/src/commonMain/kotlin/androidx/compose/material3/OutlinedTextField.kt
M compose/material3/material3/src/commonMain/kotlin/androidx/compose/material3/TextField.kt
M compose/material3/material3/src/commonMain/kotlin/androidx/compose/material3/TextFieldDefaults.kt
M compose/material3/material3/src/commonMain/kotlin/androidx/compose/material3/TextFieldImpl.kt
https://android-review.googlesource.com/2400915
Branch: androidx-main
commit d747bed10677621f865067e57a6099c7179502c3
Author: Max Alfonso-Ying <maxying@google.com>
Date: Tue Jan 03 17:50:57 2023
Implement prefix/suffix for M3 text field.
Fixes:
Test: added
Relnote: "Slot APIs for prefix and suffix text have been
added to Material 3 text fields."
Change-Id: Ia85786c582981e8f16a7053a869e06c2c1a43fb8
M compose/material3/material3/api/public_plus_experimental_current.txt
M compose/material3/material3/integration-tests/material3-catalog/src/main/java/androidx/compose/material3/catalog/library/model/Examples.kt
M compose/material3/material3/samples/src/main/java/androidx/compose/material3/samples/TextFieldSamples.kt
M compose/material3/material3/src/androidAndroidTest/kotlin/androidx/compose/material3/OutlinedTextFieldScreenshotTest.kt
M compose/material3/material3/src/androidAndroidTest/kotlin/androidx/compose/material3/OutlinedTextFieldTest.kt
M compose/material3/material3/src/androidAndroidTest/kotlin/androidx/compose/material3/TextFieldScreenshotTest.kt
M compose/material3/material3/src/androidAndroidTest/kotlin/androidx/compose/material3/TextFieldTest.kt
M compose/material3/material3/src/commonMain/kotlin/androidx/compose/material3/OutlinedTextField.kt
M compose/material3/material3/src/commonMain/kotlin/androidx/compose/material3/TextField.kt
M compose/material3/material3/src/commonMain/kotlin/androidx/compose/material3/TextFieldDefaults.kt
M compose/material3/material3/src/commonMain/kotlin/androidx/compose/material3/TextFieldImpl.kt
pr...@google.com <pr...@google.com> #8
The following release(s) address this bug.It is possible this bug has only been partially addressed:
androidx.compose.material3:material3:1.1.0-alpha06
ya...@gmail.com <ya...@gmail.com> #9
your text fields in compose are terrible
Description
There is a leadingIcon and trailingIcon APIs in the text field that are intended to be used to put an icon or iconbutton. But it was not designed to be used with prefix/suffix text as it requires a baseline alignment. We need to explore how to nicely support this.
Possibly related ticket: 1