Toggle table display by clicking header (#2)

BrainMaestro · Feb 23, 2017 · d87d073 · d87d073
1 parent 2675924
commit d87d073
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/src/display.js b/src/display.js
@@ -9,6 +9,8 @@
     var body = document.createElement('tbody');
 
     header.textContent = 'Dependencies (' + config.name + ')';
+    header.title = 'Toggle ' + config.name + ' table display';
+    header.onclick = toggleTable;
     header.style.display = 'none';
     table.style.display = 'none';
     tableHeaders.forEach(addTableHeader);
@@ -26,6 +28,10 @@
       tableHeader.textContent = header;
       table.appendChild(tableHeader);
     }
+
+    function toggleTable() {
+      table.style.display = table.style.display == 'block' ? 'none' : 'block';
+    }
   }
 
   function addDependencies(body, deps, registry, dev) {
@@ -82,9 +88,11 @@
     data[2].children[0].textContent = latestVersion;
     data[3].textContent = description;
     var table = this.parentNode.parentNode;
+    var header = table.previousSibling;
 
     table.style.display = 'block';
-    table.previousSibling.style.display = 'block';
+    header.style.display = 'block';
+    header.style.cursor = 'pointer';
   }
 
   window.display = display;

diff --git a/src/manifest.json b/src/manifest.json
@@ -1,6 +1,6 @@
 {
   "name": "Package Hub",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "manifest_version": 2,
   "description": "Display dependencies for many package managers on GitHub",
   "author": "Ezinwa Okpoechi",