Print Preview: Improving accessibility of margins UI.

chrome/browser/resources/print_preview/margins_ui.js

Index: chrome/browser/resources/print_preview/margins_ui.js
diff --git a/chrome/browser/resources/print_preview/margins_ui.js b/chrome/browser/resources/print_preview/margins_ui.js
index 8f142c59b04e672021a59852e58d77ddaf4809d7..9106bc8d0dbd3988e548f893e981fee2c13c1b98 100644
--- a/chrome/browser/resources/print_preview/margins_ui.js
+++ b/chrome/browser/resources/print_preview/margins_ui.js
@@ -107,13 +107,16 @@ cr.define('print_preview', function() {
      */
     show: function() {
       this.hidden = false;
+      this.classList.remove('invisible');
     },
 
     /**
-     * Hides the margins UI.
+     * Hides the margins UI and removes from the DOM if requested.
+     * @param {boolean} removeFromDOM True if |this| should also be removed from

[Lei Zhang, 2011/10/26 00:51:48]

Technically it's still in the DOM, just hidden.

[dpapad, 2011/10/26 01:24:24]

Yes, display: none; prevents it from being rendered, I made this more clear.

+     *     the DOM (in order to not interfere with the tab order).
      */
-    hide: function() {
-      this.hidden = true;
+    hide: function(removeFromDOM) {
+      removeFromDOM ? this.hidden = true : this.classList.add('invisible');
     },
 
     /**