Status Update
Comments
ti...@google.com <ti...@google.com>
uo...@google.com <uo...@google.com> #2
Project: platform/frameworks/support
Branch: androidx-main
Author: Louis Pullen-Freilich <
Link:
Adds OverscrollEffect#withoutDrawing and OverscrollEffect#withoutEventHandling
Expand for full commit details
Adds OverscrollEffect#withoutDrawing and OverscrollEffect#withoutEventHandling
These APIs allow overscroll to have events dispatched to it by one component, and rendered in a separate component.
Fixes: b/266550551
Fixes: b/204650733
Fixes: b/255554340
Fixes: b/229537244
Test: OverscrollTest
Relnote: "Adds OverscrollEffect#withoutDrawing and OverscrollEffect#withoutEventHandling APIs - these APIs create a wrapped instance of the provided overscroll effect that doesn't draw / handle events respectively, which allows for rendering overscroll in a separate component from the component that is dispatching events. For example, disabling drawing the overscroll inside a lazy list, and then drawing the overscroll separately on top / elsewhere."
Change-Id: Idbb3d91546b49c1987a041f959bce4b2b09a9f61
Files:
- M
compose/foundation/foundation/api/current.txt
- M
compose/foundation/foundation/api/restricted_current.txt
- M
compose/foundation/foundation/integration-tests/foundation-demos/src/main/java/androidx/compose/foundation/demos/OverscrollDemo.kt
- M
compose/foundation/foundation/samples/src/main/java/androidx/compose/foundation/samples/OverscrollSample.kt
- M
compose/foundation/foundation/src/androidInstrumentedTest/kotlin/androidx/compose/foundation/OverscrollTest.kt
- M
compose/foundation/foundation/src/commonMain/kotlin/androidx/compose/foundation/Overscroll.kt
Hash: f64e25b7a473c757d080521e7dd97b3f6670f60d
Date: Fri Nov 01 18:43:56 2024
uo...@google.com <uo...@google.com>
ra...@gmail.com <ra...@gmail.com> #3
The following release(s) address this bug.It is possible this bug has only been partially addressed:
androidx.compose.foundation:foundation:1.8.0-alpha06
androidx.compose.foundation:foundation-android:1.8.0-alpha06
androidx.compose.foundation:foundation-jvmstubs:1.8.0-alpha06
androidx.compose.foundation:foundation-linuxx64stubs:1.8.0-alpha06
kh...@gmail.com <kh...@gmail.com> #4
I agree with the previous message, and wanted to add that in LazyVerticalGrid/LazyHorizontalGrid it is possible to have horizontalArrangement and verticalArrangement, so it feels weird to no have both of them here too. Again, not consistent.
uo...@google.com <uo...@google.com>
ap...@google.com <ap...@google.com> #5
Branch: androidx-main
commit b347a7f5ed2ab16832d56c5c715e0c78cc3f7ef2
Author: Faithful Uchenna Okoye <uokoye@google.com>
Date: Wed Mar 08 20:09:36 2023
Add support for Cross Axis Arrangement and Spacing to Flow Row and FlowColumn
Adds support for verticalArrangement in FlowRow and horizontalArrangement in
FlowColumn, which provides spacing and arrangement options for the cross axis.
Test: Tested manually and automated
Bug: 268365538
Relnote: "Adds support for cross axis spacing/arrangement using the
verticalArrangement in FlowRow and horizontalArrangement in FlowColumn.
We also remove the top-level verticalAlignment and horizontalAlignment in FlowRow/FlowColumn.
Developers can use Modifier.align instead. This reduces confusion between the naming conventions of
verticalAlignment and verticalArrangement."
Change-Id: I87b60b6bb09586d9126b9494e998be0710df3b62
M compose/foundation/foundation-layout/api/public_plus_experimental_current.txt
M compose/foundation/foundation-layout/samples/src/main/java/androidx/compose/foundation/layout/samples/FlowColumnSample.kt
M compose/foundation/foundation-layout/samples/src/main/java/androidx/compose/foundation/layout/samples/FlowRowSample.kt
M compose/foundation/foundation-layout/src/androidAndroidTest/kotlin/androidx/compose/foundation/layout/FlowRowColumnTest.kt
M compose/foundation/foundation-layout/src/commonMain/kotlin/androidx/compose/foundation/layout/FlowLayout.kt
M compose/material/material/samples/src/main/java/androidx/compose/material/samples/ChipSamples.kt
uo...@google.com <uo...@google.com> #6
Thanks for the feedback. As the API is experimental, we are still ironing out the final API details. Your feedback is very welcome.
We have added horizontalArrangement and verticalArrangement to FlowRow/FlowColumn, and should become available in the next alpha release.
na...@google.com <na...@google.com> #7
The following release(s) address this bug.It is possible this bug has only been partially addressed:
androidx.compose.foundation:foundation-layout:1.5.0-alpha02
androidx.compose.material:material:1.5.0-alpha02
Description
Currently, there isn't an option to specify spacing between the multiple resulting rows of a
FlowRow
or multiple resulting columns of aFlowColumn
.One partial workaround would be to add that padding to each element, but that padding will apply to all rows/columns, even the last one (which may not be desirable).