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
Limit max Dialog
and Popup
size by safe area on iOS
#732
Conversation
actual class PopupProperties @ExperimentalComposeUiApi constructor( | ||
actual val focusable: Boolean, | ||
actual val dismissOnBackPress: Boolean, | ||
actual val dismissOnClickOutside: Boolean | ||
actual val dismissOnClickOutside: Boolean, | ||
val usePlatformDefaultWidth: Boolean = false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addition of a new argument (even with default value) will break binary backward compatibility on stable Desktop.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or this class doesn't exists in Compose Multiplatform 1.4.3 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I see this class doesn't exists in 1.4.3
So, we need to cherry pick this PR also to preparing branch of Compose Multiplatform 1.5.0 Release ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looked only on the new API (usePlatformDefaultWidth
) - looks good to me.
Proposed Changes
Dialog
andPopup
size by safe area on iOSPopup
andDialog
toRootMeasurePolicy
usePlatformDefaultWidth
parameter toPopupProperties
Testing
Test: run
mpp
demo, open max sizeDialog