[BottomSheetState] Regression: Since the removal of the targetValue is not possible to know if a bottom sheet is expanding or collapsing [271169225]

Fixed

Bug

Status Update

No update yet.

Description

ro...@veeva.com

created issue #1

Mar 1, 2023 04:35PM

Jetpack Compose version: 1.4.0-beta02 Jetpack Compose component used: androidx.compose.material.BottomSheetState Android Studio Build: Android Studio Electric Eel | 2022.1.1 Patch 2 Kotlin version: 1.8.10

As per title, after the removal of the targetValue isn't possible to know if a bottom sheet is expanding or collapsing.

The use case on why this is needed is to change the alpha of an element depending on the current expanded progress of the bottom sheet itself.

The bottomSheetState.progress goes always from 0.0 to 1.0, no matter if is expanding or collapsing.

And checking the current value doesn't help, because it switches from Expanded to Collapsed remaining with the value of 1.0:

Expanded - 0.91185045
Expanded - 0.93893886
Expanded - 0.97347516
Expanded - 0.9933874
Expanded - 1.0
Collapsed - 1.0
Collapsed - 0.342198
Collapsed - 0.39998567
Collapsed - 0.48986524
Collapsed - 0.5680212
Collapsed - 0.6550029
Collapsed - 0.7393994
Collapsed - 0.83361727
Collapsed - 0.9114269
Collapsed - 1.0
Expanded - 1.0

Before it was possible to control the alpha of the element with this logic:

        val fraction = bottomSheetState.progress.fraction
        val targetValue = bottomSheetState.targetValue
        val currentValue = bottomSheetState.currentValue
        return when {
            currentValue == BottomSheetValue.Collapsed && targetValue == BottomSheetValue.Collapsed -> 0f
            currentValue == BottomSheetValue.Expanded && targetValue == BottomSheetValue.Expanded -> 1f
            currentValue == BottomSheetValue.Collapsed && targetValue == BottomSheetValue.Expanded -> fraction
            else -> 1 - fraction
        }

But now, without the target value, it is impossible because, once you fully expand or fully collapse the bottom sheet, the currentValue will change from Expanded to Collapse, but the progress will stay to 1.0.

Steps to Reproduce or Code Sample to Reproduce: 1.Add a bottom BottomSheetScaffold with a bottom sheet 2.Try to change the alpha of a component depending on the current expanded/collapsed state, so that the alpha should decrease when the bottom sheet shrink and increase when it expand.

Comments

ma...@google.com <ma...@google.com> Mar 1, 2023 05:40PM

Reassigned to jo...@google.com.

[Deleted User] <[Deleted User]> #2Mar 6, 2023 10:14AM

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

commit 01e6286fba92911346fa8cb2efd9bd024f18e11d
Author: Louis Pullen-Freilich <lpf@google.com>
Date: Fri Oct 30 18:20:15 2020

Adds androidx.compose.material.AmbientContentColor to replace androidx.compose.foundation.AmbientContentColor

Content color is a design system specific concept, and so the foundation
AmbientContentColor will be deprecated and replaced with the new material one.
Bug:

b/172067770

Test: updateApi
Relnote: "Adds androidx.compose.material.AmbientContentColor to replace androidx.compose.foundation.AmbientContentColor"

Change-Id: I84f7be83a15ed02b301c3d9d0e44481ebb01ddb0

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
A compose/material/material/src/commonMain/kotlin/androidx/compose/material/ContentColor.kt

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

jo...@google.com <jo...@google.com> #3Mar 13, 2023 02:50PM

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

commit dea1c5dc2c9b8ed70e5c0c617cb9f96fa92335c1
Author: Louis Pullen-Freilich <lpf@google.com>
Date: Fri Oct 30 20:56:57 2020

Deprecates foundation content color and text style ambients, replacing them with Material theming ambients

Bug:

b/172067770

Test: updateApi
Relnote: "The foundation AmbientTextStyle, ProvideTextStyle, and AmbientContentColor have been deprecated. Instead use the new versions available in the Material library. For non-Material applications, you should instead create your own design system specific theming ambients that can be consumed in your own components."
Change-Id: I74accf7166eaca28e9e2d14402ed08d80f8625ab

M compose/animation/animation/integration-tests/animation-demos/src/main/java/androidx/compose/animation/demos/AnimateContentSizeDemo.kt
M compose/animation/animation/samples/src/main/java/androidx/compose/animation/samples/AnimationModifierSample.kt
M compose/desktop/desktop/src/jvmTest/kotlin/androidx/compose/desktop/ParagraphTest.kt
M compose/foundation/foundation/api/current.txt
M compose/foundation/foundation/api/public_plus_experimental_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/ListDemos.kt
M compose/foundation/foundation/samples/src/main/java/androidx/compose/foundation/samples/InteractionStateSample.kt
M compose/foundation/foundation/src/commonMain/kotlin/androidx/compose/foundation/BaseTextField.kt
M compose/foundation/foundation/src/commonMain/kotlin/androidx/compose/foundation/ContentColor.kt
M compose/foundation/foundation/src/commonMain/kotlin/androidx/compose/foundation/Icon.kt
M compose/foundation/foundation/src/commonMain/kotlin/androidx/compose/foundation/Text.kt
M compose/integration-tests/demos/src/main/java/androidx/compose/integration/demos/DemoFilter.kt
M compose/material/material/integration-tests/material-demos/src/main/java/androidx/compose/material/demos/ColorPickerDemo.kt
M compose/material/material/samples/src/main/java/androidx/compose/material/samples/ListSamples.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/AlertDialogTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/AppBarTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/ButtonTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/EmphasisTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/MenuTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/SurfaceContentColorTest.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/TextFieldTest.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/AlertDialog.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/AppBar.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/BottomNavigation.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Button.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Colors.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Emphasis.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/FloatingActionButton.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Icon.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/ListItem.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/MaterialTheme.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Menu.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/OutlinedTextField.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Snackbar.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Surface.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Tab.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Text.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/TextField.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/TextFieldImpl.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/ripple/RippleTheme.kt

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

ro...@veeva.com <ro...@veeva.com> #4Apr 12, 2023 11:02AM

Hi, any news about this issue? In which release it will be fixed?

jo...@google.com <jo...@google.com> #5Apr 17, 2023 07:55AM

Hello, no news to share yet. We are currently focusing on a Foundation API for anchored draggable components and will come back to this after.

[Deleted User] <[Deleted User]> #6May 15, 2023 04:39AM

Hello, is there a release version with this issue fixed?

jo...@google.com <jo...@google.com> #7May 16, 2023 02:33PM

Hi #6, we will update this issue once the change has been merged and when it has been released.

ap...@google.com <ap...@google.com> #8May 24, 2023 10:57AM

Marked as fixed.

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

commit 762bbc3cea2a9bcfb6fae34e71c6bb8da917f43c
Author: Jossi Wolf <jossiwolf@google.com>
Date: Mon May 15 11:50:25 2023

[M2] Expose AnchoredDraggable components' progress

Fixes: 271169225
Fixes: 276375124
Fixes: 276776071
Fixes: 270066861
Relnote: BottomSheetState, ModalBottomSheetState and BottomDrawerState now expose a progress property indicating the progress between the current (settled) anchor and the closest anchor in the swipe direction.
Test: AnchoredDraggable tests

Change-Id: I1b31727b4f56df890e336bea9f4de526733aaacc

M compose/material/material/api/current.txt
M compose/material/material/api/restricted_current.txt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/AnchoredDraggable.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/BottomSheetScaffold.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Drawer.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/ModalBottomSheet.kt

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

pr...@google.com <pr...@google.com> #9Jun 21, 2023 08:57PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.compose.material:material:1.6.0-alpha01
androidx.compose.material:material-android:1.6.0-alpha01

ju...@google.com <ju...@google.com> #10Jun 28, 2023 11:52PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.compose.material:material:1.5.0-beta03
androidx.compose.material:material-android:1.5.0-beta03

na...@google.com <na...@google.com> #11Jul 26, 2023 05:21PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.wear.compose:compose-material-core:1.2.0-rc01
androidx.wear.compose:compose-ui-tooling:1.2.0-rc01

Issue 271169225

Description

Issue summary

Comments

ma...@google.com <ma...@google.com> Mar 1, 2023 05:40PM

[Deleted User] <[Deleted User]> #2Mar 6, 2023 10:14AM

jo...@google.com <jo...@google.com> #3Mar 13, 2023 02:50PM

ro...@veeva.com <ro...@veeva.com> #4Apr 12, 2023 11:02AM

jo...@google.com <jo...@google.com> #5Apr 17, 2023 07:55AM

[Deleted User] <[Deleted User]> #6May 15, 2023 04:39AM

jo...@google.com <jo...@google.com> #7May 16, 2023 02:33PM

ap...@google.com <ap...@google.com> #8May 24, 2023 10:57AM

pr...@google.com <pr...@google.com> #9Jun 21, 2023 08:57PM

ju...@google.com <ju...@google.com> #10Jun 28, 2023 11:52PM

na...@google.com <na...@google.com> #11Jul 26, 2023 05:21PM

Add comment

Issue metadata