Skip to content

Commit

Permalink
fix Modal content being cut off when Android Activity is edge-to-edge (
Browse files Browse the repository at this point in the history
…#47339)

Summary:
Pull Request resolved: #47339

Fixing issue raised in #47307

This is a follow up from D62286026.
It appears there was a line that went missing while trying to refactor the code.

`fitsSystemWindows = true` is needeod for < API 30 to avoid content rendering under the system bars when Modal is shown with Activity that is edge-to-edge.

Changelog:
[Android][Fixed] Fix Regression - Modal content rendering below system bar on < API 30 when activity is edge-to-edge

Reviewed By: cortinico

Differential Revision: D65280014

fbshipit-source-id: 616ff739be55635f1295ef3bf8b997a27ef769ae
  • Loading branch information
alanleedev authored and blakef committed Nov 12, 2024
1 parent fbe38bb commit e8c4faa
Showing 1 changed file with 8 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -296,7 +296,14 @@ public class ReactModalHostView(context: ThemedReactContext) :
* changed. This has the pleasant side-effect of us not having to preface all Modals with "top:
* statusBarHeight", since that margin will be included in the FrameLayout.
*/
get() = FrameLayout(context).apply { addView(dialogRootViewGroup) }
get() =
FrameLayout(context).apply {
addView(dialogRootViewGroup)
if (!statusBarTranslucent) {
// this is needed to prevent content hiding behind systems bars < API 30
this.fitsSystemWindows = true
}
}

/**
* updateProperties will update the properties that do not require us to recreate the dialog
Expand Down

0 comments on commit e8c4faa

Please sign in to comment.