ExposedDropdownMenuBox shows menu when scrolling [212091796]

Fixed

Bug

Status Update

No update yet.

Description

br...@gmail.com

created issue #1

Dec 27, 2021 03:23AM

Jetpack Compose release version: 1.1.0-rc01

Android Studio Build: 2020.3.1 Patch 4

Kotlin version: 203-1.6.10-release-923-AS7717.8

When you want to scroll the screen the menu is displayed if you tap the component. The attached GIF shows the problem.

To reproduce, just run the code below.

class MainActivity : ComponentActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContent {
            MyApplicationTheme {
                Surface(color = MaterialTheme.colors.background) {
                    Column(
                        modifier = Modifier
                            .padding(horizontal = 8.dp)
                            .verticalScroll(rememberScrollState()),
                        verticalArrangement = Arrangement.spacedBy(8.dp)
                    ) {
                        repeat(20){
                            ExposedDropdownMenuSample()
                        }
                    }
                }
            }
        }
    }
}

ExposedDropdownMenuSample was taken from the official samples here https://cs.android.com/androidx/platform/frameworks/support/+/androidx-main:compose/material/material/samples/src/main/java/androidx/compose/material/samples/ExposedDropdownMenuSamples.kt.

dropdown.gif

4.4 MB

View

Download

Comments

je...@google.com <je...@google.com> Jan 5, 2022 03:18PM

Assigned to ma...@google.com.

ma...@google.com <ma...@google.com> #2Jan 10, 2022 10:59AM

Reassigned to po...@google.com.

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

2g...@gmail.com <2g...@gmail.com> #3Feb 6, 2022 05:28PM

Thanks for the report!

po...@google.com <po...@google.com> Feb 21, 2022 10:25AM

Reassigned to je...@google.com.

ed...@gmail.com <ed...@gmail.com> #4Aug 29, 2022 09:35AM

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.

ed...@gmail.com <ed...@gmail.com> #5Aug 29, 2022 10:10AM

deleted

br...@gmail.com <br...@gmail.com> #6Sep 9, 2022 02:04AM

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).

ma...@google.com <ma...@google.com> Oct 21, 2022 05:15PM

Reassigned to ma...@google.com.

ma...@google.com <ma...@google.com> Oct 24, 2022 04:11PM

Accepted by ma...@google.com.

ap...@google.com <ap...@google.com> #7Nov 1, 2022 03:20PM

Marked as fixed.

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

commit 059c7f3c39ad8a0abe5bb10d04a65967cfd7ebc8
Author: Max Alfonso-Ying <maxying@google.com>
Date: Tue Oct 25 01:01:58 2022

Fix ExposedDropdownMenu opening while scrolling

Fixes:

b/212091796

Test: new tests in ExposedDropdownMenuTest
Relnote: "`awaitFirstDown` and `waitForUpOrCancellation` now
accept a PointerEventPass for greater flexibility."
Change-Id: I7579a2dbb44c748a3fd3e515d2e7ab086aaff443

M compose/foundation/foundation/api/current.ignore
M compose/foundation/foundation/api/current.txt
M compose/foundation/foundation/api/public_plus_experimental_current.txt
M compose/foundation/foundation/api/restricted_current.ignore
M compose/foundation/foundation/api/restricted_current.txt
M compose/foundation/foundation/src/commonMain/kotlin/androidx/compose/foundation/gestures/Draggable.kt
M compose/foundation/foundation/src/commonMain/kotlin/androidx/compose/foundation/gestures/TapGestureDetector.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/ExposedDropdownMenuTest.kt
M compose/material/material/src/androidMain/kotlin/androidx/compose/material/ExposedDropdownMenu.kt
M compose/material3/material3/src/androidAndroidTest/kotlin/androidx/compose/material3/ExposedDropdownMenuTest.kt
M compose/material3/material3/src/androidMain/kotlin/androidx/compose/material3/ExposedDropdownMenu.kt

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

na...@google.com <na...@google.com> #8Nov 9, 2022 06:28PM

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

androidx.compose.foundation:foundation:1.4.0-alpha02
androidx.compose.material:material:1.4.0-alpha02
androidx.compose.material3:material3:1.1.0-alpha02