From 64acc82b9634d948517ec5bb2ebe5a33cdf22df6 Mon Sep 17 00:00:00 2001
From: Petri Hienonen <petri.hienonen@gmail.com>
Date: Fri, 14 Nov 2025 21:39:29 +0200
Subject: Cleanup

---
 app/dom.js | 29 ++++++++++++++---------------
 1 file changed, 14 insertions(+), 15 deletions(-)

(limited to 'app/dom.js')

diff --git a/app/dom.js b/app/dom.js
index 9ea7bf9..9473bf1 100644
--- a/app/dom.js
+++ b/app/dom.js
@@ -80,7 +80,7 @@ export class Dom {
 		if (text) button.textContent = text;
 		for (const [k, v] of Object.entries(o.attributes)) button.setAttribute(k, v);
 		if (o.children) button.append(...o.children);
-		if (onClick) button.addEventListener("click", onClick);
+		if (onClick) button.addEventListener("pointerdown", onClick);
 		return button;
 	}
 
@@ -192,13 +192,15 @@ export class Dom {
 
 	/**
 	 * Create a `<select>`
+	 * @param {string|undefined} selected
 	 * @param {DomOptions} o
 	 * @param { (e: Event) => void } onChange
 	 * @returns {HTMLSelectElement}
 	 */
-	static select(onChange, o = new DomOptions()) {
+	static select(selected = undefined, onChange, o = new DomOptions()) {
 		const select = document.createElement("select");
 		Object.assign(select.style, o.styles);
+		if (selected !== undefined) select.value = selected;
 		if (o.id) select.id = o.id;
 		for (const cls of o.classes) select.classList.add(cls);
 		for (const [k, v] of Object.entries(o.attributes)) select.setAttribute(k, v);
@@ -373,8 +375,8 @@ export class Dom {
 
 		// Update active range position
 		const updateActiveRange = () => {
-			const minVal = parseInt(minSlider.value);
-			const maxVal = parseInt(maxSlider.value);
+			const minVal = parseInt(minSlider.value, 10);
+			const maxVal = parseInt(maxSlider.value, 10);
 
 			const minPercent = ((minVal - min) / (max - min)) * 100;
 			const maxPercent = ((maxVal - min) / (max - min)) * 100;
@@ -388,41 +390,38 @@ export class Dom {
 
 		// Event listeners
 		minSlider.addEventListener("input", () => {
-			const minVal = parseInt(minSlider.value);
-			const maxVal = parseInt(maxSlider.value);
+			const minVal = parseInt(minSlider.value, 10);
+			const maxVal = parseInt(maxSlider.value, 10);
 
 			if (minVal > maxVal) {
 				minSlider.value = maxVal.toString();
 			}
 
 			updateActiveRange();
-			onChange(parseInt(minSlider.value), parseInt(maxSlider.value));
+			onChange(parseInt(minSlider.value, 10), parseInt(maxSlider.value, 10));
 		});
 
 		maxSlider.addEventListener("input", () => {
-			const minVal = parseInt(minSlider.value);
-			const maxVal = parseInt(maxSlider.value);
+			const minVal = parseInt(minSlider.value, 10);
+			const maxVal = parseInt(maxSlider.value, 10);
 
 			if (maxVal < minVal) {
 				maxSlider.value = minVal.toString();
 			}
 
 			updateActiveRange();
-			onChange(parseInt(minSlider.value), parseInt(maxSlider.value));
+			onChange(parseInt(minSlider.value, 10), parseInt(maxSlider.value, 10));
 		});
 
 		// Initial update
 		updateActiveRange();
 
 		// Assemble track
-		track.appendChild(activeRange);
-		track.appendChild(minSlider);
-		track.appendChild(maxSlider);
+		track.append(activeRange, minSlider, maxSlider);
 		trackContainer.appendChild(track);
 
 		// Assemble container
-		container.appendChild(valueDisplay);
-		container.appendChild(trackContainer);
+		container.append(valueDisplay, trackContainer);
 
 		if (options.children) container.append(...options.children);
 
-- 
cgit v1.2.3-70-g09d2