Fix sortable types and separate JS init into functions

agatemosu · Dec 29, 2024 · 099d3db · 099d3db
1 parent ab7e860
commit 099d3db
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/src/main.js b/src/main.js
@@ -105,22 +105,26 @@ function dynamicStyle(e) {
 
 // #region Setup
 
-function main() {
+function setUpSortable() {
+	/** @type {HTMLElement} */
+	const imagesBar = document.querySelector("#images-bar");
+	Sortable.create(imagesBar, { group: TierRow.sortableGroup });
+}
+
+function setUpEvents() {
 	const newTierButton = document.querySelector("#new-tier");
 	const selectImagesButton = document.querySelector("#select-images");
-	const imagesBar = document.querySelector("#images-bar");
 	const checkboxes = document.querySelectorAll(".dynamic-style");
 
 	newTierButton.addEventListener("click", addNewTier);
 	selectImagesButton.addEventListener("click", selectImages);
 
-	Sortable.create(imagesBar, { group: TierRow.sortableGroup });
-
 	for (const checkbox of checkboxes) {
 		checkbox.addEventListener("change", dynamicStyle);
 	}
 }
 
-main();
+setUpSortable();
+setUpEvents();
 
 // #endregion
diff --git a/src/tier-row.js b/src/tier-row.js
@@ -61,7 +61,7 @@ export default class TierRow extends HTMLElement {
 		this.createColorPicker();
 
 		// Sortable
-		const tierSort = this.querySelector(".sort");
+		const tierSort = /** @type {HTMLElement} */ (this.querySelector(".sort"));
 		this.sort = new Sortable(tierSort, { group: TierRow.sortableGroup });
 
 		// Options