Skip to content

Commit 8697cb7

Browse files
committed
Don't set focusable for banners
We want focusable on for non banner IAMs to support keyboard input, but we don't want it on for banners since it restricts interacting with the app underneath the banner
1 parent 9166053 commit 8697cb7

File tree

1 file changed

+3
-1
lines changed
  • OneSignalSDK/onesignal/in-app-messages/src/main/java/com/onesignal/inAppMessages/internal/display/impl

1 file changed

+3
-1
lines changed

OneSignalSDK/onesignal/in-app-messages/src/main/java/com/onesignal/inAppMessages/internal/display/impl/InAppMessageView.kt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,9 @@ internal class InAppMessageView(
279279
)
280280
popupWindow?.setBackgroundDrawable(ColorDrawable(Color.TRANSPARENT))
281281
popupWindow?.isTouchable = true
282-
popupWindow?.isFocusable = true
282+
// Focusable allows keyboard input for HTML IAMs, but also prevents interacting under banners
283+
popupWindow?.isFocusable = !displayPosition.isBanner
284+
283285
// NOTE: This is required for getting fullscreen under notches working in portrait mode
284286
popupWindow?.isClippingEnabled = false
285287
var gravity = 0

0 commit comments

Comments
 (0)