Bug 480061: Lots of JS was toggling the "display" attribute instead of removing/adding the bz_default_hidden class.

Patch By Max Kanat-Alexander <mkanat@bugzilla.org> r=glob, a=mkanat

1 files changed, 3 insertions, 6 deletions

diff --git a/js/change-columns.js b/js/change-columns.js
index 5fd5c1085..e70cfd560 100644
--- a/js/change-columns.js
+++ b/js/change-columns.js
@@ -20,12 +20,9 @@ function initChangeColumns() {
     window.onunload = unload;
     var av_select = document.getElementById("available_columns");
     var sel_select = document.getElementById("selected_columns");
-    document.getElementById("avail_header").style.display = "inline";
-    document.getElementById("available_columns").style.display = "inline";
-    document.getElementById("select_button").style.display = "inline";
-    document.getElementById("deselect_button").style.display = "inline";
-    document.getElementById("up_button").style.display = "inline";
-    document.getElementById("down_button").style.display = "inline";
+    YAHOO.util.Dom.removeClass(
+        ['avail_header', av_select, 'select_button', 
+         'deselect_button', 'up_button', 'down_button'], 'bz_default_hidden');
     switch_options(sel_select, av_select, false);
     sel_select.selectedIndex = -1;
     updateView();