From 5b668d32ebd5a7697b9b766b4b21eb9c64bfae58 Mon Sep 17 00:00:00 2001
From: byxor <brandnewbyxor@gmail.com>
Date: Sun, 20 Oct 2024 23:48:46 +0100
Subject: [PATCH] Small tidyups, fix division by 0

---
 models/coverage.js                   |  4 ++-
 models/power.js                      | 39 ----------------------------
 notes/browser/components/progress.js |  4 +--
 3 files changed, 5 insertions(+), 42 deletions(-)

diff --git a/models/coverage.js b/models/coverage.js
index c12c545..a40d977 100644
--- a/models/coverage.js
+++ b/models/coverage.js
@@ -87,7 +87,9 @@ const newPinCoverage = (hole, pin) => {
 class HoleCoverage {
 
     constructor(hole) {
-        this.pinCoverages = hole.pins.map(pin => {
+        const eligiblePins = hole.pins.filter(pin => pin.setups.length > 0);
+
+        this.pinCoverages = eligiblePins.map(pin => {
             return newPinCoverage(hole, pin);
         });
 
diff --git a/models/power.js b/models/power.js
index 187c938..0ec009e 100644
--- a/models/power.js
+++ b/models/power.js
@@ -33,45 +33,6 @@ function newPower(value, direction) {
 	return value;
 }
 
-// ...
-// const POWER_48_UP = newPower("48%", "up");
-// const POWER_48_DOWN = newPower("48%", "down");
-// const POWER_49_UP = newPower("49%", "up");
-// const POWER_49_DOWN = newPower("49%", "down");
-// const POWER_50_UP = newPower("50%", "up");
-// const POWER_50_DOWN = newPower("50%", "down");
-// const POWER_51_UP = newPower("51%", "up");
-// const POWER_51_DOWN = newPower("51%", "down");
-// const POWER_52_UP = newPower("52%", "up");
-// const POWER_53_UP = newPower("53%", "up");
-// const POWER_53_DOWN = newPower("53%", "down");
-// const POWER_54_DOWN = newPower("54%", "down");
-// const POWER_55_UP = newPower("55%", "up");
-// ...
-// const POWER_97_UP = newPower("97%", "up");
-// const POWER_97_DOWN = newPower("97%", "down");
-// const POWER_98_UP = newPower("98%", "up");
-// const POWER_98_DOWN = newPower("98%", "down");
-// const POWER_99_UP = newPower("99%", "up");
-// const POWER_100_DOWN = newPower("100%", "down");
-// const POWER_101_UP = newPower("101%", "up");
-// const POWER_101_DOWN = newPower("101%", "down");
-// const POWER_102_UP = newPower("102%", "up");
-// const POWER_102_DOWN = newPower("102%", "down");
-// const POWER_103_UP = newPower("103%", "up");
-// const POWER_103_DOWN = newPower("103%", "down");
-// const POWER_104_UP = newPower("104%", "up");
-// const POWER_104_DOWN = newPower("104%", "down");
-// const POWER_105_UP = newPower("105%", "up");
-// const POWER_MAX_4_DOWN = newPower("max-4%", "down");
-// const POWER_MAX_3_UP = newPower("max-3%", "up");
-// const POWER_MAX_3_DOWN = newPower("max-3%", "down");
-// const POWER_MAX_2_UP = newPower("max-2%", "up");
-// const POWER_MAX_2_DOWN = newPower("max-2%", "down");
-// const POWER_MAX_1_UP = newPower("max-1%", "up");
-// const POWER_MAX_1_DOWN = newPower("max-1%", "down");
-// const POWER_MAX = newPower("max%", "");
-
 // Should there be 2 entries for max? 1 for each frame?
 // how about MAX_UP is the early one (feels like it "jumps" up to max a little).
 // and MAX_DOWN is the later one.
diff --git a/notes/browser/components/progress.js b/notes/browser/components/progress.js
index 6b19a7f..b8c3ab9 100644
--- a/notes/browser/components/progress.js
+++ b/notes/browser/components/progress.js
@@ -27,7 +27,7 @@ class ProgressComponent extends HTMLElement {
             const holeCoverage = newHoleCoverage(this.#hole);
             return `Hole Coverage: &nbsp;${holeCoverage.toString()}`;
         }
-        return "...";
+        return "";
     }
 
     #calculatePinProgressText() {
@@ -35,7 +35,7 @@ class ProgressComponent extends HTMLElement {
             const pinCoverage = newPinCoverage(this.#hole, this.#pin);
             return `Pin Coverage: &nbsp;&nbsp;&nbsp;${pinCoverage.toString()}`;
         }
-        return "...";
+        return "";
     }
 
     #render() {