It seems that FF32 does not support hidden property in XUL. Fixes iss…

…ue 7824
SeleniumHQ · Sep 11, 2014 · 70ce404 · 70ce404
1 parent 5163bce
commit 70ce404
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/ide/main/src/content/editor.js b/ide/main/src/content/editor.js
@@ -1416,13 +1416,13 @@ Editor.InfoPanel.prototype.switchView = function (view) {
 Editor.InfoView = function () { };
 
 Editor.InfoView.prototype.show = function () {
-  document.getElementById(this.name + "View").hidden = false;
+  document.getElementById(this.name + "View").style.display = "block";
   document.getElementById(this.name + "Tab").setAttribute("selected", "true");
 };
 
 Editor.InfoView.prototype.hide = function () {
   document.getElementById(this.name + "Tab").removeAttribute("selected");
-  document.getElementById(this.name + "View").hidden = true;
+  document.getElementById(this.name + "View").style.display = "none";
 };
 
 /*
@@ -1446,12 +1446,12 @@ Editor.LogView.prototype = new Editor.InfoView;
 
 Editor.LogView.prototype.show = function () {
   Editor.InfoView.prototype.show.call(this);
-  document.getElementById("logButtons").hidden = false;
+  document.getElementById("logButtons").style.display = "flex";
 };
 
 Editor.LogView.prototype.hide = function () {
   Editor.InfoView.prototype.hide.call(this);
-  document.getElementById("logButtons").hidden = true;
+  document.getElementById("logButtons").style.display = "none";
 } ;
 
 Editor.LogView.prototype.setLog = function (log) {

diff --git a/ide/main/src/content/selenium-ide-common.xul b/ide/main/src/content/selenium-ide-common.xul
@@ -311,11 +311,11 @@ limitations under the License.
     </hbox>
     <iframe flex="1" id="logView" src="LogFrame.html" type="content">
     </iframe>
-    <iframe flex="1" id="helpView" src="HelpView.html" hidden="true" type="content">
+    <iframe flex="1" id="helpView" src="HelpView.html" style="display:none" type="content">
     </iframe>
-    <iframe flex="1" id="uiView" src="ui-view.html" hidden="true" type="content">
+    <iframe flex="1" id="uiView" src="ui-view.html" style="display:none" type="content">
     </iframe>
-    <iframe flex="1" id="rollupView" src="rollup-view.html" hidden="true" type="content">
+    <iframe flex="1" id="rollupView" src="rollup-view.html" style="display:none" type="content">
     </iframe>
   </vbox>
 </overlay>