Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make DropdownMenu focusable by default #375

Closed
igordmn opened this issue Feb 11, 2021 · 2 comments
Closed

Make DropdownMenu focusable by default #375

igordmn opened this issue Feb 11, 2021 · 2 comments
Assignees
Labels
component ui glitch Visual glitch in UI, usually not blocking normal usage

Comments

@igordmn
Copy link
Collaborator

igordmn commented Feb 11, 2021

Android version of DropdownMenu is focusable by default:
https://cs.android.com/androidx/platform/frameworks/support/+/androidx-main:compose/material/material/src/androidMain/kotlin/androidx/compose/material/AndroidMenu.android.kt;l=74

And so it supports dismissal, when we click outside menu.

Desktop version should support dismissal too, so it should be focusable by default:
https://cs.android.com/androidx/platform/frameworks/support/+/androidx-main:compose/material/material/src/desktopMain/kotlin/androidx/compose/material/DesktopMenu.desktop.kt;l=88

@olonho olonho added component ui glitch Visual glitch in UI, usually not blocking normal usage labels Feb 15, 2021
@ggeetha
Copy link

ggeetha commented Mar 18, 2021

Any updates on this issue?
I am also facing issue while using DropdownMenu. As per my understanding, its internally using Popup where there is no option to set the expanded attribute true or false. By default it takes false. So its kind of blocking UI component. It doesnt allow any click outside this. Please consider this issue as priority. We are using multiple DropdownMenu in our class which is creating usability issue.

@Rsedaikin
Copy link
Contributor

Fixed.

MatkovIvan pushed a commit to MatkovIvan/compose-multiplatform that referenced this issue May 10, 2023
Implement ScrollbarAdapter for lazy grids.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component ui glitch Visual glitch in UI, usually not blocking normal usage
Projects
None yet
Development

No branches or pull requests

4 participants