From e30a1922ed1b231214ded330387b4ab28f9966ff Mon Sep 17 00:00:00 2001 From: "Luna D." Date: Sun, 23 Jun 2024 19:59:06 +0200 Subject: [PATCH 1/2] convert misc scripts to ts --- assets/js/{misc.js => misc.ts} | 45 ++++++++++++++++++++-------------- 1 file changed, 26 insertions(+), 19 deletions(-) rename assets/js/{misc.js => misc.ts} (60%) diff --git a/assets/js/misc.js b/assets/js/misc.ts similarity index 60% rename from assets/js/misc.js rename to assets/js/misc.ts index 5733e954..c8365d22 100644 --- a/assets/js/misc.js +++ b/assets/js/misc.ts @@ -4,35 +4,40 @@ import store from './utils/store'; import { $, $$ } from './utils/dom'; +import { assertNotNull } from './utils/assert'; +import '../types/ujs'; let touchMoved = false; -function formResult({target, detail}) { - - const elements = { +function formResult({target, detail}: FetchcompleteEvent) { + const elements: {[key: string]: string} = { '#description-form': '.image-description', '#uploader-form': '.image_uploader' }; - function showResult(resultEl, formEl, response) { + function showResult(resultEl: HTMLElement, formEl: HTMLFormElement, response: string) { resultEl.innerHTML = response; resultEl.classList.remove('hidden'); formEl.classList.add('hidden'); - formEl.querySelector('input[type="submit"],button').disabled = false; + const inputEl = $('input[type="submit"]', formEl); + const buttonEl = $('button', formEl); + + if (inputEl) inputEl.disabled = false; + if (buttonEl) buttonEl.disabled = false; } for (const element in elements) { - if (target.matches(element)) detail.text().then(text => showResult($(elements[element]), target, text)); + if (target.matches(element)) { + detail.text().then(text => showResult(assertNotNull($(elements[element])), target as HTMLFormElement, text)); + } } - } -function revealSpoiler(event) { - - const { target } = event; +function revealSpoiler(event: MouseEvent | TouchEvent) { + const target = assertNotNull(event.target) as HTMLElement; const spoiler = target.closest('.spoiler'); - let imgspoiler = target.closest('.spoiler .imgspoiler, .spoiler-revealed .imgspoiler'); const showContainer = target.closest('.image-show-container'); + let imgspoiler = target.closest('.spoiler .imgspoiler, .spoiler-revealed .imgspoiler'); // Prevent reveal if touchend came after touchmove event if (touchMoved) { @@ -42,7 +47,8 @@ function revealSpoiler(event) { if (spoiler) { if (showContainer) { - const imageShow = showContainer.querySelector('.image-show'); + const imageShow = assertNotNull(showContainer.querySelector('.image-show')); + if (!imageShow.classList.contains('hidden') && imageShow.classList.contains('spoiler-pending')) { imageShow.classList.remove('spoiler-pending'); return; @@ -62,19 +68,22 @@ function revealSpoiler(event) { if (imgspoiler) { imgspoiler.classList.remove('imgspoiler'); imgspoiler.classList.add('imgspoiler-revealed'); + if (event.type === 'touchend' && !event.defaultPrevented) { event.preventDefault(); } } - } -function setupEvents() { - const extrameta = $('#extrameta'); +export function setupEvents() { + const extrameta = $('#extrameta'); + + if (extrameta && store.get('hide_uploader')) { + extrameta.classList.add('hidden'); + } - if (store.get('hide_uploader') && extrameta) extrameta.classList.add('hidden'); if (store.get('hide_score')) { - $$('.upvotes,.score,.downvotes').forEach(s => s.classList.add('hidden')); + $$('.upvotes,.score,.downvotes').forEach(s => s.classList.add('hidden')); } document.addEventListener('fetchcomplete', formResult); @@ -82,5 +91,3 @@ function setupEvents() { document.addEventListener('touchend', revealSpoiler); document.addEventListener('touchmove', () => touchMoved = true); } - -export { setupEvents }; From e5b476f07f4ec4ca66355ce6f8daebc99ab5ab1c Mon Sep 17 00:00:00 2001 From: Liam Date: Sun, 23 Jun 2024 14:56:31 -0400 Subject: [PATCH 2/2] Further simplify formResult, setupEvents --- assets/js/misc.ts | 32 +++++++++++++++++--------------- 1 file changed, 17 insertions(+), 15 deletions(-) diff --git a/assets/js/misc.ts b/assets/js/misc.ts index c8365d22..3de3ef86 100644 --- a/assets/js/misc.ts +++ b/assets/js/misc.ts @@ -3,32 +3,34 @@ */ import store from './utils/store'; -import { $, $$ } from './utils/dom'; -import { assertNotNull } from './utils/assert'; +import { $, $$, hideEl, showEl } from './utils/dom'; +import { assertNotNull, assertType } from './utils/assert'; import '../types/ujs'; let touchMoved = false; function formResult({target, detail}: FetchcompleteEvent) { - const elements: {[key: string]: string} = { + const elements: Record = { '#description-form': '.image-description', '#uploader-form': '.image_uploader' }; - function showResult(resultEl: HTMLElement, formEl: HTMLFormElement, response: string) { + function showResult(formEl: HTMLFormElement, resultEl: HTMLElement, response: string) { resultEl.innerHTML = response; - resultEl.classList.remove('hidden'); - formEl.classList.add('hidden'); - const inputEl = $('input[type="submit"]', formEl); - const buttonEl = $('button', formEl); + hideEl(formEl); + showEl(resultEl); - if (inputEl) inputEl.disabled = false; - if (buttonEl) buttonEl.disabled = false; + $$('input[type="submit"],button', formEl).forEach(button => { + button.disabled = false; + }); } - for (const element in elements) { - if (target.matches(element)) { - detail.text().then(text => showResult(assertNotNull($(elements[element])), target as HTMLFormElement, text)); + for (const [ formSelector, resultSelector ] of Object.entries(elements)) { + if (target.matches(formSelector)) { + const form = assertType(target, HTMLFormElement); + const result = assertNotNull($(resultSelector)); + + detail.text().then(text => showResult(form, result, text)); } } } @@ -79,11 +81,11 @@ export function setupEvents() { const extrameta = $('#extrameta'); if (extrameta && store.get('hide_uploader')) { - extrameta.classList.add('hidden'); + hideEl(extrameta); } if (store.get('hide_score')) { - $$('.upvotes,.score,.downvotes').forEach(s => s.classList.add('hidden')); + $$('.upvotes,.score,.downvotes').forEach(s => hideEl(s)); } document.addEventListener('fetchcomplete', formResult);