Merge pull request #469 from arv/hidden-property

Add support for reflecting the hidden attribute
googlearchive · Jul 16, 2014 · 67607e7 · 67607e7
2 parents db04e21 + 4e3bffa
commit 67607e7
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/src/wrappers/HTMLElement.js b/src/wrappers/HTMLElement.js
@@ -230,6 +230,17 @@
 
       var df = frag(contextElement, text);
       contextElement.insertBefore(df, refNode);
+    },
+
+    get hidden() {
+      return this.hasAttribute('hidden');
+    },
+    set hidden(v) {
+      if (v) {
+        this.setAttribute('hidden', '');
+      } else {
+        this.removeAttribute('hidden');
+      }
     }
   });
 

diff --git a/test/js/HTMLElement.js b/test/js/HTMLElement.js
@@ -101,4 +101,13 @@ suite('HTMLElement', function() {
     assert.equal(div.innerHTML, '<!--&\u00A0<>"-->');
   });
 
+  test('hidden property', function() {
+    var div = document.createElement('div');
+    assert.isFalse(div.hidden);
+    div.hidden = true;
+    assert.isTrue(div.hasAttribute('hidden'));
+    assert.equal(div.getAttribute('hidden'), '');
+    div.hidden = false;
+    assert.isFalse(div.hasAttribute('hidden'));
+  });
 });