Add a "draggable: Boolean" argument to material3 ModalBottomSheet [288211587]

Fixed

Feature Request

Status Update

No update yet.

Description

lo...@gmail.com

created issue #1

Jun 21, 2023 11:05AM

Jetpack Compose version: 1.2.0-alpha02

Jetpack Compose component used: androidx.compose.material3:material3

Similar to https://issuetracker.google.com/issues/186669832 (Material 2 ModalBottomSheet) I need to have an option to disable the drag gesture in certain cases for the Material 3 version of ModalBottomSheet.

Currently, there is only the confirmValueChange parameter in rememberModalBottomSheetState but that is only called for the 3 SheetValues, no for any draggable state.

I noticed that the enabled state for draggable in Modifier.modalBottomSheetSwipeable is inly dependent on the visibility of the sheet. Maybe SheetState could get another parameter dragEnabled which could then also be checked.

Comments

se...@google.com <se...@google.com> Aug 29, 2023 02:20PM

Reassigned to ma...@google.com.

ca...@thetrainline.com <ca...@thetrainline.com> #2Oct 19, 2023 12:19PM

There is a report about it in kotlinlang Slack: https://kotlinlang.slack.com/archives/C01D6HTPATV/p1706005056105739

ko...@gmail.com <ko...@gmail.com> #3Jan 5, 2024 03:48PM

Thanks for the report!

kl...@naver.com <kl...@naver.com> #4Mar 4, 2024 12:44AM

The release notes documentation has been edited to clarify this change in behavior for line height.

To support non-standard text sizes, we encourage users to follow the Material design system and use a different type scale rather than changing the font size directly. Alternatively, users can overwrite the line height like so: style = LocalTextStyle.current.copy(lineHeight = TextUnit.Unspecified), or create a custom Typography entirely.

ni...@gmail.com <ni...@gmail.com> #5Mar 7, 2024 06:30PM

deleted

mu...@gmail.com <mu...@gmail.com> #6Mar 18, 2024 06:39AM

In my case, I have multiple font sizes in the same Text (using SpanStyle in AnnotatedString). There are legitimate reasons for this. For example, when combining Chinese and English (phonetic) together (for language-learning purposes).

j6...@gmail.com <j6...@gmail.com> #7Apr 29, 2024 08:36PM

An update would be appreciated. As of now, one workaround I'm using is

confirmValueChange = { it != SheetValue.Hidden }

Message last modified on Apr 30, 2024 02:15AM

se...@google.com <se...@google.com> Apr 30, 2024 08:37PM

Reassigned to se...@google.com.

at...@gmail.com <at...@gmail.com> #8Aug 4, 2024 11:31AM

Any update on the issue?

va...@gmail.com <va...@gmail.com> #9Aug 21, 2024 01:01PM

Same question

r....@kaizengaming.com <r....@kaizengaming.com> #10Aug 26, 2024 08:38AM

Any update on this one?

sv...@gmail.com <sv...@gmail.com> #11Sep 3, 2024 12:24PM

Any update on this please?

aj...@gmail.com <aj...@gmail.com> #12Sep 10, 2024 07:34PM

It's really impressive how a small feature like this is still not in progress.

r....@kaizengaming.com <r....@kaizengaming.com> #13Sep 27, 2024 08:14AM

This issue has remained unresolved for an extended period. A status update regarding the progress and any potential roadblocks encountered would be appreciated.

se...@google.com <se...@google.com> Sep 30, 2024 05:47PM

Accepted by se...@google.com.

ap...@google.com <ap...@google.com> #14Oct 2, 2024 08:59PM

Project: platform/frameworks/support
Branch: androidx-main
Author: Jose Figueroa <serniebanders@google.com>
Link: https://android-review.googlesource.com/3287876

[Material3][ModalBottomSheet] Add sheetGestureEnabled parameter to allow manual disabling of gestures

Expand for full commit details

[Material3][ModalBottomSheet] Add sheetGestureEnabled parameter to allow manual disabling of gestures

RelNote: "ModalBottomSheet now has sheetGestureEnabled parameter"
Bug: 288211587
Test: Added unit test for nested scrolling behavior
Change-Id: I856cbb6b8907ce773d47589f2c83a828a5740eb8

Files:

M compose/material3/material3/api/current.txt
M compose/material3/material3/api/restricted_current.txt
M compose/material3/material3/src/androidInstrumentedTest/kotlin/androidx/compose/material3/ModalBottomSheetTest.kt
M compose/material3/material3/src/commonMain/kotlin/androidx/compose/material3/ModalBottomSheet.kt

Hash: 18bebb5575db25ab0ff3b5e439401ea40e7bad0e
Date: Mon Sep 30 14:31:26 2024

se...@google.com <se...@google.com> Oct 2, 2024 09:04PM

Marked as fixed.

aj...@gmail.com <aj...@gmail.com> #15Oct 3, 2024 12:17AM

Can we have the fix version please?

se...@gmail.com <se...@gmail.com> #16Oct 15, 2024 10:39PM

Was this fix also added to Material ModalBottomSheet?

na...@google.com <na...@google.com> #17Oct 16, 2024 05:48PM

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

androidx.compose.material3:material3:1.4.0-alpha02
androidx.compose.material3:material3-android:1.4.0-alpha02
androidx.compose.material3:material3-jvmstubs:1.4.0-alpha02
androidx.compose.material3:material3-linuxx64stubs:1.4.0-alpha02

ch...@gmail.com <ch...@gmail.com> #18Dec 9, 2024 05:55AM

Comment has been deleted.

Message last modified on Dec 9, 2024 05:56AM

se...@google.com <se...@google.com> #19Dec 12, 2024 05:40PM

We do not have a timeline for a Material implementation. Please create a separate issue for that component if it does not exist.

ma...@extrabat.com <ma...@extrabat.com> #20Jan 13, 2025 01:39PM

Hi, any delay to have it on a stable release ?