-
-
Notifications
You must be signed in to change notification settings - Fork 124
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
BottomSheet closes when recomposing #330
Comments
@jordond I am also facing the similar issue with textfield, were you able to solve it? |
for me its only happening on ios not on android |
@DevSrSouza can you please have a look at this? for me it happens only in ios and after updating to compose 1.6 |
Same here, only in iOS. It closes when TextField is focused and keyboard opens up. Compose 1.6 |
It seems the culprit is Removing confirmValueChange and using LaunchedEffect to observe sheetState.currentValue solves the issue. Let me raise a PR for this. |
The PR from @hm-tamim was released on |
@DevSrSouza I had the same issue after updating compose, and can confirm that the issue is fixed in |
@DevSrSouza it's fixed with @hm-tamim's contribution but there is a case, open any bottom sheet with a text field, keep the text field in focus and navigate to another screen pop the screen and in some cases backstack is cleared |
I have a BottomSheet that when launched, allows the user to choose a color to base the app theme on. When a color is selected a new theme is generated and the colors in the app change.
When this happens the BottomSheet gets dismissed.
Here is some relevant code:
And a video of it happening:
Screen.Recording.2024-02-09.at.4.29.17.PM.mov
The text was updated successfully, but these errors were encountered: