make IAM's popup window non-focusable

nan-li · nan-li · commit 3fd922393a91 · 2024-02-20T14:48:23.000-08:00
* Revert the focusable property of the PopupWindow that displays an IAM view to be false. This was added as part of fullbleed IAM implementation in #1481, but does not appear to be necessary. * This allows banner-style top and bottom IAMs to remain on the screen while the user interacts with the app. * Otherwise, currently, tapping on the background dismisses the banner-style IAM.
diff --git a/OneSignalSDK/onesignal/in-app-messages/src/main/java/com/onesignal/inAppMessages/internal/display/impl/InAppMessageView.kt b/OneSignalSDK/onesignal/in-app-messages/src/main/java/com/onesignal/inAppMessages/internal/display/impl/InAppMessageView.kt
@@ -274,7 +274,7 @@ internal class InAppMessageView(
                 parentRelativeLayout,
                 if (hasBackground) WindowManager.LayoutParams.MATCH_PARENT else pageWidth,
                 if (hasBackground) WindowManager.LayoutParams.MATCH_PARENT else WindowManager.LayoutParams.WRAP_CONTENT,
-                true,
+                false,
             )
         popupWindow!!.setBackgroundDrawable(ColorDrawable(Color.TRANSPARENT))
         popupWindow!!.isTouchable = true

Original file line number	Diff line number	Diff line change
`@@ -274,7 +274,7 @@ internal class InAppMessageView(`
`274`	`274`	`parentRelativeLayout,`
`275`	`275`	`if (hasBackground) WindowManager.LayoutParams.MATCH_PARENT else pageWidth,`
`276`	`276`	`if (hasBackground) WindowManager.LayoutParams.MATCH_PARENT else WindowManager.LayoutParams.WRAP_CONTENT,`
`277`		`- true,`
	`277`	`+ false,`
`278`	`278`	`)`
`279`	`279`	`popupWindow!!.setBackgroundDrawable(ColorDrawable(Color.TRANSPARENT))`
`280`	`280`	`popupWindow!!.isTouchable = true`