From fb2635db526dfae9647c9fd702b3e854ed606da9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Christian=20K=C3=B6nig?= Date: Thu, 13 Oct 2022 09:52:09 +0200 Subject: [PATCH] Add comment to explain 99.99 decision MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Christian König --- scripts/pi-hole/js/index.js | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/scripts/pi-hole/js/index.js b/scripts/pi-hole/js/index.js index f6671178..9c4fe11f 100644 --- a/scripts/pi-hole/js/index.js +++ b/scripts/pi-hole/js/index.js @@ -725,7 +725,13 @@ function updateSummaryData(runOnce) { function doughnutTooltip(tooltipLabel) { var percentageTotalShown = tooltipLabel.chart._metasets[0].total.toFixed(2); + // tooltipLabel.chart._metasets[0].total returns the total percentage of the shown slices + // to compensate rounding errors we round to two decimals + var label = " " + tooltipLabel.label; + + // even if no doughnut slice is hidden, sometimes percentageTotalShown is only 99.99 + // we therefore use this value to decide if slices are hidden if (percentageTotalShown >= 99.99) { // All items shown return label + ": " + tooltipLabel.parsed.toFixed(1) + "%";