Fixed
Status Update
Comments
lp...@google.com <lp...@google.com> #2
Project: platform/frameworks/support
Branch: androidx-main
commit 9082f62682f853ad5251a1c79dde9eccba7abdd9
Author: Max Alfonso-Ying <maxying@google.com>
Date: Thu Apr 18 00:34:40 2024
[M2 text field] Apply background to decoration box
...instead of to the BasicTextField, so changing the
backgroundColor will properly change the decoration
box's background color.
Fixes: b/307694651
Test: added unit tests
Relnote: "Fix backgroundColor not applying to
TextFieldDecorationBox and OutlinedTextFieldDecorationBox.
Decoration boxes now accept a `shape` parameter."
Change-Id: I371c26718597cb36ac537e9412ce476532afb40d
M compose/material/material/api/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/TextFieldDecorationBoxDemos.kt
M compose/material/material/src/androidInstrumentedTest/kotlin/androidx/compose/material/textfield/TextFieldDecorationBoxTest.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
M 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/3046833
Branch: androidx-main
commit 9082f62682f853ad5251a1c79dde9eccba7abdd9
Author: Max Alfonso-Ying <maxying@google.com>
Date: Thu Apr 18 00:34:40 2024
[M2 text field] Apply background to decoration box
...instead of to the BasicTextField, so changing the
backgroundColor will properly change the decoration
box's background color.
Fixes:
Test: added unit tests
Relnote: "Fix backgroundColor not applying to
TextFieldDecorationBox and OutlinedTextFieldDecorationBox.
Decoration boxes now accept a `shape` parameter."
Change-Id: I371c26718597cb36ac537e9412ce476532afb40d
M compose/material/material/api/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/TextFieldDecorationBoxDemos.kt
M compose/material/material/src/androidInstrumentedTest/kotlin/androidx/compose/material/textfield/TextFieldDecorationBoxTest.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
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/TextFieldDefaults.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/TextFieldImpl.kt
an...@google.com <an...@google.com> #3
I actually had an idea to try to start using subcomposition similar to the one we have in WithConstraints for TabRow and Scaffold to fix this one frame lag
[Deleted User] <[Deleted User]> #4
<purged>
[Deleted User] <[Deleted User]> #5
<purged>
an...@google.com <an...@google.com>
an...@google.com <an...@google.com> #6
It will be fixed when we migrate Scaffold to SubcomposeLayout
lp...@google.com <lp...@google.com>
lp...@google.com <lp...@google.com>
ap...@google.com <ap...@google.com> #7
Project: platform/frameworks/support
Branch: androidx-master-dev
commit e243833b9705e35184bb8aec84cb9ef49f66eb76
Author: Louis Pullen-Freilich <lpf@google.com>
Date: Mon Sep 14 16:10:51 2020
Refactors Scaffold to use SubcomposeLayout, avoiding frame delays in calculating the content padding / FAB cutout location
Also removes nullable type from Composable lambda parameters, these now just default to emptyContent()
Bug: b/157633857
Bug: b/158551084
Test: ScaffoldTest
Test: ScaffoldScreenshotTest
Relnote: "Removes nullable type from Scaffold lambda parameters, you can use emptyContent() to represent no content for a given parameter."
Change-Id: I2b3181de9314b8b0dd48c30e2f663cddc0d62448
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/MaterialDemos.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/ScaffoldScreenshotTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/ScaffoldTest.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/AppBar.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Scaffold.kt
https://android-review.googlesource.com/1426753
Branch: androidx-master-dev
commit e243833b9705e35184bb8aec84cb9ef49f66eb76
Author: Louis Pullen-Freilich <lpf@google.com>
Date: Mon Sep 14 16:10:51 2020
Refactors Scaffold to use SubcomposeLayout, avoiding frame delays in calculating the content padding / FAB cutout location
Also removes nullable type from Composable lambda parameters, these now just default to emptyContent()
Bug:
Bug:
Test: ScaffoldTest
Test: ScaffoldScreenshotTest
Relnote: "Removes nullable type from Scaffold lambda parameters, you can use emptyContent() to represent no content for a given parameter."
Change-Id: I2b3181de9314b8b0dd48c30e2f663cddc0d62448
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/MaterialDemos.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/ScaffoldScreenshotTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/ScaffoldTest.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/AppBar.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Scaffold.kt
Description
Component used: BottomAppBar
Version used: AS 4.1 Canary 8 + Compose snapshot at 6557927
I created a BottomAppBar with a FAB and cutout, and running the emulator it appears correct but in the preview the cutout is not rendered correctly.
Code: