try to ensure iframe is hidden DuendeArchive#430

brockallen · brockallen · commit af360a31d3d6 · 2017-10-04T09:15:12.000-04:00
diff --git a/src/CheckSessionIFrame.js b/src/CheckSessionIFrame.js
@@ -16,11 +16,15 @@ export default class CheckSessionIFrame {
         this._frame_origin = url.substr(0, idx);
 
         this._frame = window.document.createElement("iframe");
-        //this._frame.style.display = "none";
+
+        // shotgun approach
         this._frame.style.visibility = "hidden";
         this._frame.style.position = "absolute";
-        this._frame.src = url;
+        this._frame.style.display = "none";
+        this._frame.style.width = 0;
+        this._frame.style.height = 0;
 
+        this._frame.src = url;
     }
     load() {
         return new Promise((resolve) => {
diff --git a/src/IFrameWindow.js b/src/IFrameWindow.js
@@ -19,9 +19,14 @@ export default class IFrameWindow {
         window.addEventListener("message", this._boundMessageEvent, false);
         
         this._frame = window.document.createElement("iframe");
-        //this._frame.style.display = "none";
+
+        // shotgun approach
         this._frame.style.visibility = "hidden";
         this._frame.style.position = "absolute";
+        this._frame.style.display = "none";
+        this._frame.style.width = 0;
+        this._frame.style.height = 0;
+        
         window.document.body.appendChild(this._frame);
     }
 
