Renaming isFullscreen to isFullbleed to avoid confusion

Author: emawby

OneSignalSDK/onesignal/src/main/java/com/onesignal/InAppMessageView.java

@@ -18,7 +18,6 @@
 import android.view.Gravity;
 import android.view.View;
 import android.view.ViewGroup;
-import android.view.WindowInsets;
 import android.view.WindowManager;
 import android.view.animation.Animation;
 import android.webkit.WebView;
@@ -287,16 +286,11 @@ private void createPopupWindow(@NonNull RelativeLayout parentRelativeLayout) {
                 true
         );
 
-        OSViewUtils.getWindowHeight(currentActivity);
         popupWindow.setBackgroundDrawable(new ColorDrawable(Color.TRANSPARENT));
         popupWindow.setTouchable(true);
-        // NOTE: This seems like the key to getting fullscreen under notches working?!
+        // NOTE: This is required for getting fullscreen under notches working in portrait mode
         popupWindow.setClippingEnabled(false);
 
-        View container = (View)popupWindow.getContentView();
-        System.out.println("container: " + container);
-        System.out.println("container.getClass: " + container.getClass());
-
         int gravity = 0;
         if (!hasBackground) {
             switch (displayLocation) {
@@ -315,7 +309,7 @@ private void createPopupWindow(@NonNull RelativeLayout parentRelativeLayout) {
 
         // Using panel for fullbleed IAMs and dialog for non-fullbleed. The attached dialog type
         // does not allow content to bleed under notches but panel does.
-        int displayType = this.messageContent.isFullScreen() ?
+        int displayType = this.messageContent.isFullBleed() ?
                 WindowManager.LayoutParams.TYPE_APPLICATION_PANEL :
                 WindowManager.LayoutParams.TYPE_APPLICATION_ATTACHED_DIALOG;
         PopupWindowCompat.setWindowLayoutType(

OneSignalSDK/onesignal/src/main/java/com/onesignal/OSInAppMessageContent.kt

@@ -12,7 +12,7 @@ internal open class OSInAppMessageContent constructor(jsonObject: JSONObject) {
     var contentHtml: String? = null
     var useHeightMargin: Boolean = true
     var useWidthMargin: Boolean = true
-    var isFullScreen: Boolean = false
+    var isFullBleed: Boolean = false
     // The following properties are populated from Javascript events
     var displayLocation: WebViewManager.Position? = null
     var displayDuration: Double? = null
@@ -24,6 +24,6 @@ internal open class OSInAppMessageContent constructor(jsonObject: JSONObject) {
         var styles: JSONObject? = jsonObject.optJSONObject(STYLES)
         useHeightMargin = !(styles?.optBoolean(REMOVE_HEIGHT_MARGIN, false) ?: false)
         useWidthMargin = !(styles?.optBoolean(REMOVE_WIDTH_MARGIN, false) ?: false)
-        isFullScreen = !useHeightMargin
+        isFullBleed = !useHeightMargin
     }
 }

OneSignalSDK/onesignal/src/main/java/com/onesignal/WebViewManager.java

@@ -1,6 +1,5 @@
 package com.onesignal;
 
-
 import android.annotation.SuppressLint;
 import android.annotation.TargetApi;
 import android.app.Activity;
@@ -145,7 +144,7 @@ private static void setContentSafeAreaInsets(OSInAppMessageContent content, @Non
     }
 
     private static void initInAppMessage(@NonNull final Activity currentActivity, @NonNull OSInAppMessageInternal message, @NonNull final OSInAppMessageContent content) {
-        if (content.isFullScreen()) {
+        if (content.isFullBleed()) {
             setContentSafeAreaInsets(content, currentActivity);
         }
         try {
@@ -163,7 +162,7 @@ private static void initInAppMessage(@NonNull final Activity currentActivity, @N
                 public void run() {
                     // Handles exception "MissingWebViewPackageException: Failed to load WebView provider: No WebView installed"
                     try {
-                        webViewManager.setupWebView(currentActivity, base64Str, content.isFullScreen());
+                        webViewManager.setupWebView(currentActivity, base64Str, content.isFullBleed());
                     } catch (Exception e) {
                         // Need to check error message to only catch MissingWebViewPackageException as it isn't public
                         if (e.getMessage() != null && e.getMessage().contains("No WebView installed")) {
@@ -239,7 +238,7 @@ public void postMessage(String message) {
         }
 
         private void handleResize() {
-            if (messageContent.isFullScreen()) {
+            if (messageContent.isFullBleed()) {
                 updateSafeAreaInsets();
             }
         }
@@ -334,8 +333,8 @@ private void calculateHeightAndShowWebViewAfterNewActivity() {
         if (messageView == null)
             return;
 
-        // Don't need a CSS / HTML height update for fullscreen
-        if (messageView.getDisplayPosition() == Position.FULL_SCREEN && !messageContent.isFullScreen()) {
+        // Don't need a CSS / HTML height update for fullscreen unless its fullbleed
+        if (messageView.getDisplayPosition() == Position.FULL_SCREEN && !messageContent.isFullBleed()) {
             showMessageView(null);
             return;
         }
@@ -349,7 +348,7 @@ public void run() {
                 // At time point the webView isn't attached to a view
                 // Set the WebView to the max screen size then run JS to evaluate the height.
                 setWebViewToMaxSize(activity);
-                if (messageContent.isFullScreen()) {
+                if (messageContent.isFullBleed()) {
                     updateSafeAreaInsets();
                 }
 
@@ -512,7 +511,7 @@ private static void enableWebViewRemoteDebugging() {
     }
 
     private int getWebViewMaxSizeX(Activity activity) {
-        if (messageContent.isFullScreen()) {
+        if (messageContent.isFullBleed()) {
             return getFullbleedWindowWidth(activity);
         } else {
 
@@ -522,7 +521,7 @@ private int getWebViewMaxSizeX(Activity activity) {
     }
 
     private int getWebViewMaxSizeY(Activity activity) {
-        int margin = messageContent.isFullScreen() ? 0 : (MARGIN_PX_SIZE * 2);
+        int margin = messageContent.isFullBleed() ? 0 : (MARGIN_PX_SIZE * 2);
        return OSViewUtils.getWindowHeight(activity) - margin;
     }