Fixed
Status Update
Comments
co...@google.com <co...@google.com>
sg...@google.com <sg...@google.com>
ap...@google.com <ap...@google.com> #2
Project: platform/frameworks/support
Branch: androidx-main
commit a69ecbe83d71d06d94b7f98f24287654ec0b9e74
Author: Shalom Gibly <sgibly@google.com>
Date: Fri Feb 04 14:46:50 2022
Fixes hidden buttons when a Dialog has a long text
Fixes an issue where a LazyColumn that is set as a text Composable for a
dialog hides the dialog's buttons.
Fixes: 216663029
Test: AlertDialogTest
Relnote: "Bug fix for hidden AlertDialog buttons when a long text is added with a LazyColumn."
Change-Id: Ib2cc936b917bdb64c13fe92820c47482e0b1e200
M compose/material3/material3/src/androidAndroidTest/kotlin/androidx/compose/material3/AlertDialogTest.kt
M compose/material3/material3/src/commonMain/kotlin/androidx/compose/material3/AlertDialog.kt
https://android-review.googlesource.com/1973483
Branch: androidx-main
commit a69ecbe83d71d06d94b7f98f24287654ec0b9e74
Author: Shalom Gibly <sgibly@google.com>
Date: Fri Feb 04 14:46:50 2022
Fixes hidden buttons when a Dialog has a long text
Fixes an issue where a LazyColumn that is set as a text Composable for a
dialog hides the dialog's buttons.
Fixes: 216663029
Test: AlertDialogTest
Relnote: "Bug fix for hidden AlertDialog buttons when a long text is added with a LazyColumn."
Change-Id: Ib2cc936b917bdb64c13fe92820c47482e0b1e200
M compose/material3/material3/src/androidAndroidTest/kotlin/androidx/compose/material3/AlertDialogTest.kt
M compose/material3/material3/src/commonMain/kotlin/androidx/compose/material3/AlertDialog.kt
Description
When using a large LazyColumn as content in a material3 AlertDialog the buttons are not visible.
They are visible on material version of AlertDialog. See attachements.
As a side note Material3 version properly handle scrolling unlike material version seehttps://kotlinlang.slack.com/archives/CJLTWPH7S/p1643385774067189 that will probably open an issue so I don't to avoid duplicates.