From 134431d88c7c9b063f362cc9f773a1e3a20f085a Mon Sep 17 00:00:00 2001
From: Arthur Lu <learthurgo@gmail.com>
Date: Tue, 14 Feb 2023 20:05:56 +0000
Subject: [PATCH] remove handler for inactive buttons, change delete icon

Signed-off-by: Arthur Lu <learthurgo@gmail.com>
---
 images/actions/delete-active.svg              |  2 +-
 images/actions/delete-inactive.svg            |  2 +-
 .../detach_attach-inactive.svg}               |  0
 scripts/config.js                             | 22 +++++++++----------
 4 files changed, 12 insertions(+), 14 deletions(-)
 rename images/{blank.svg => actions/detach_attach-inactive.svg} (100%)

diff --git a/images/actions/delete-active.svg b/images/actions/delete-active.svg
index f336ecd..bcbc96a 100644
--- a/images/actions/delete-active.svg
+++ b/images/actions/delete-active.svg
@@ -1 +1 @@
-<svg fill="#ff0000" width="800" height="800" viewBox="0 0 1024 1024" xmlns="http://www.w3.org/2000/svg"><path d="M512.481 421.906L850.682 84.621c25.023-24.964 65.545-24.917 90.51.105s24.917 65.545-.105 90.51L603.03 512.377 940.94 850c25.003 24.984 25.017 65.507.033 90.51s-65.507 25.017-90.51.033L512.397 602.764 174.215 940.03c-25.023 24.964-65.545 24.917-90.51-.105s-24.917-65.545.105-90.51l338.038-337.122L84.14 174.872c-25.003-24.984-25.017-65.507-.033-90.51s65.507-25.017 90.51-.033L512.48 421.906z"/></svg>
\ No newline at end of file
+<svg width="16" height="16" viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg"><path d="M10 11v6M14 11v6M4 7h16M6 7h12v11a3 3 0 01-3 3H9a3 3 0 01-3-3V7zM9 5a2 2 0 012-2h2a2 2 0 012 2v2H9V5z" stroke="red" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/></svg>
\ No newline at end of file
diff --git a/images/actions/delete-inactive.svg b/images/actions/delete-inactive.svg
index 831a08c..49d1dbe 100644
--- a/images/actions/delete-inactive.svg
+++ b/images/actions/delete-inactive.svg
@@ -1 +1 @@
-<svg fill="#400000" width="800" height="800" viewBox="0 0 1024 1024" xmlns="http://www.w3.org/2000/svg"><path d="M512.481 421.906L850.682 84.621c25.023-24.964 65.545-24.917 90.51.105s24.917 65.545-.105 90.51L603.03 512.377 940.94 850c25.003 24.984 25.017 65.507.033 90.51s-65.507 25.017-90.51.033L512.397 602.764 174.215 940.03c-25.023 24.964-65.545 24.917-90.51-.105s-24.917-65.545.105-90.51l338.038-337.122L84.14 174.872c-25.003-24.984-25.017-65.507-.033-90.51s65.507-25.017 90.51-.033L512.48 421.906z"/></svg>
\ No newline at end of file
+<svg width="16" height="16" viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg"><path d="M10 11v6M14 11v6M4 7h16M6 7h12v11a3 3 0 01-3 3H9a3 3 0 01-3-3V7zM9 5a2 2 0 012-2h2a2 2 0 012 2v2H9V5z" stroke="#400000" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/></svg>
\ No newline at end of file
diff --git a/images/blank.svg b/images/actions/detach_attach-inactive.svg
similarity index 100%
rename from images/blank.svg
rename to images/actions/detach_attach-inactive.svg
diff --git a/scripts/config.js b/scripts/config.js
index 5a90007..adfa2ba 100644
--- a/scripts/config.js
+++ b/scripts/config.js
@@ -137,18 +137,16 @@ function addDiskLine (fieldset, busPrefix, busName, device, diskDetails) {
 			let active = diskMetaData[type][busPrefix].actions.includes(element) ? "active" : "inactive"; // resize
 			action.src = `images/actions/${element}-${active}.svg`;
 			action.title = `${element.charAt(0).toUpperCase()}${element.slice(1)} Disk`;
-			if (element === "move") {
-				action.addEventListener("click", handleDiskMove);
-			}
-			else if (element === "resize") {
-				action.addEventListener("click", handleDiskResize);
-			}
-			else if (element === "delete") {
-				action.addEventListener("click", handleDiskDelete);
-			}
-			else { // entry does not support anything in this category, override the src and title for a blank tile
-				action.src = "images/blank.svg";
-				action.title = "";
+			if (active === "active") {
+				if (element === "move") {
+					action.addEventListener("click", handleDiskMove);
+				}
+				else if (element === "resize") {
+					action.addEventListener("click", handleDiskResize);
+				}
+				else if (element === "delete") {
+					action.addEventListener("click", handleDiskDelete);
+				}
 			}
 		}
 		action.dataset.disk = diskID;