use vis hidden and pos abs for iframe to workaround FF bug DuendeArchive#311

brockallen · brockallen · commit 2e16ef7de629 · 2017-09-24T11:11:57.000-04:00
diff --git a/src/CheckSessionIFrame.js b/src/CheckSessionIFrame.js
@@ -16,7 +16,9 @@ export default class CheckSessionIFrame {
         this._frame_origin = url.substr(0, idx);
 
         this._frame = window.document.createElement("iframe");
-        this._frame.style.display = "none";
+        //this._frame.style.display = "none";
+        this._frame.style.visibility = "hidden";
+        this._frame.style.position = "absolute";
         this._frame.src = url;
 
     }
diff --git a/src/IFrameWindow.js b/src/IFrameWindow.js
@@ -3,7 +3,7 @@
 
 import Log from './Log';
 
-const DefaultTimeout = 30000;
+const DefaultTimeout = 10000;
 
 export default class IFrameWindow {
 
@@ -19,7 +19,9 @@ export default class IFrameWindow {
         window.addEventListener("message", this._boundMessageEvent, false);
         
         this._frame = window.document.createElement("iframe");
-        this._frame.style.display = "none";
+        //this._frame.style.display = "none";
+        this._frame.style.visibility = "hidden";
+        this._frame.style.position = "absolute";
         window.document.body.appendChild(this._frame);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,9 @@ export default class CheckSessionIFrame {`
`16`	`16`	`this._frame_origin = url.substr(0, idx);`
`17`	`17`
`18`	`18`	`this._frame = window.document.createElement("iframe");`
`19`		`- this._frame.style.display = "none";`
	`19`	`+ //this._frame.style.display = "none";`
	`20`	`+ this._frame.style.visibility = "hidden";`
	`21`	`+ this._frame.style.position = "absolute";`
`20`	`22`	`this._frame.src = url;`
`21`	`23`
`22`	`24`	`}`