Bug 1385270 - Fix about:telemetry subsection display r?gfritzsche draft
authorflyingrub <flyinggrub@gmail.com>
Wed, 09 Aug 2017 01:16:02 +0200
changeset 646491 4011fda85c94d46d9cb0864e19285cb1503a0e0d
parent 645963 6213fec03a47275d8fcbf79b951547e3993d677f
child 726261 909bb67696e6c306787755e5d9e9963322ef289c
push id74140
push userbmo:flyinggrub@gmail.com
push dateTue, 15 Aug 2017 09:32:34 +0000
reviewersgfritzsche
bugs1385270
milestone57.0a1
Bug 1385270 - Fix about:telemetry subsection display r?gfritzsche Show all subsection after comming back from another section. MozReview-Commit-ID: I02cR2IaBrm
toolkit/content/aboutTelemetry.js
--- a/toolkit/content/aboutTelemetry.js
+++ b/toolkit/content/aboutTelemetry.js
@@ -1902,22 +1902,33 @@ function show(selected) {
   let selectedValue = selected.getAttribute("value");
   if (selectedValue === "raw-json-viewer") {
     openJsonInFirefoxJsonViewer(JSON.stringify(gPingData, null, 2));
     return;
   }
 
   let current_button = document.querySelector(".category.selected");
   current_button.classList.remove("selected");
+  if (current_button.classList.contains("has-subsection")) {
+    for (let subsection of current_button.children) {
+      subsection.classList.remove("selected");
+    }
+  }
   selected.classList.add("selected");
   // Hack because subsection text appear selected. See Bug 1375114.
   document.getSelection().empty();
 
   let current_section = document.querySelector("section.active");
   let selected_section = document.getElementById(selectedValue);
+  let subsections = current_section.querySelectorAll(".sub-section");
+  if (subsections) {
+    for (let subsection of subsections) {
+      subsection.hidden = false;
+    }
+  }
   if (current_section == selected_section)
     return;
   current_section.classList.remove("active");
   selected_section.classList.add("active");
 
   let title = selected.querySelector(".category-name").textContent.trim();
   document.getElementById("sectionTitle").textContent = title;