Remove Rails.delegate from admin.tsx

This commit is contained in:
Renaud Chaput 2025-03-28 22:39:16 +01:00
parent 74ee96505a
commit e30e5a974c
No known key found for this signature in database
GPG Key ID: BCFC859D49B46990
3 changed files with 40 additions and 34 deletions

View File

@ -1,7 +1,7 @@
import './public-path'; import './public-path';
import { createRoot } from 'react-dom/client'; import { createRoot } from 'react-dom/client';
import Rails from '@rails/ujs'; import { on } from 'delegated-events';
import ready from '../mastodon/ready'; import ready from '../mastodon/ready';
@ -24,10 +24,9 @@ const setAnnouncementEndsAttributes = (target: HTMLInputElement) => {
} }
}; };
Rails.delegate( on(
document,
'input[type="datetime-local"]#announcement_starts_at',
'change', 'change',
'input[type="datetime-local"]#announcement_starts_at',
({ target }) => { ({ target }) => {
if (target instanceof HTMLInputElement) if (target instanceof HTMLInputElement)
setAnnouncementEndsAttributes(target); setAnnouncementEndsAttributes(target);
@ -63,7 +62,7 @@ const hideSelectAll = () => {
if (hiddenField) hiddenField.value = '0'; if (hiddenField) hiddenField.value = '0';
}; };
Rails.delegate(document, '#batch_checkbox_all', 'change', ({ target }) => { on('change', '#batch_checkbox_all', ({ target }) => {
if (!(target instanceof HTMLInputElement)) return; if (!(target instanceof HTMLInputElement)) return;
const selectAllMatchingElement = document.querySelector( const selectAllMatchingElement = document.querySelector(
@ -85,7 +84,7 @@ Rails.delegate(document, '#batch_checkbox_all', 'change', ({ target }) => {
} }
}); });
Rails.delegate(document, '.batch-table__select-all button', 'click', () => { on('click', '.batch-table__select-all button', () => {
const hiddenField = document.querySelector<HTMLInputElement>( const hiddenField = document.querySelector<HTMLInputElement>(
'#select_all_matching', '#select_all_matching',
); );
@ -113,7 +112,7 @@ Rails.delegate(document, '.batch-table__select-all button', 'click', () => {
} }
}); });
Rails.delegate(document, batchCheckboxClassName, 'change', () => { on('change', batchCheckboxClassName, () => {
const checkAllElement = document.querySelector<HTMLInputElement>( const checkAllElement = document.querySelector<HTMLInputElement>(
'input#batch_checkbox_all', 'input#batch_checkbox_all',
); );
@ -140,14 +139,9 @@ Rails.delegate(document, batchCheckboxClassName, 'change', () => {
} }
}); });
Rails.delegate( on('change', '.filter-subset--with-select select', ({ target }) => {
document, if (target instanceof HTMLSelectElement) target.form?.submit();
'.filter-subset--with-select select', });
'change',
({ target }) => {
if (target instanceof HTMLSelectElement) target.form?.submit();
},
);
const onDomainBlockSeverityChange = (target: HTMLSelectElement) => { const onDomainBlockSeverityChange = (target: HTMLSelectElement) => {
const rejectMediaDiv = document.querySelector( const rejectMediaDiv = document.querySelector(
@ -168,11 +162,11 @@ const onDomainBlockSeverityChange = (target: HTMLSelectElement) => {
} }
}; };
Rails.delegate(document, '#domain_block_severity', 'change', ({ target }) => { on('change', '#domain_block_severity', ({ target }) => {
if (target instanceof HTMLSelectElement) onDomainBlockSeverityChange(target); if (target instanceof HTMLSelectElement) onDomainBlockSeverityChange(target);
}); });
const onEnableBootstrapTimelineAccountsChange = (target: HTMLInputElement) => { function onEnableBootstrapTimelineAccountsChange(target: HTMLInputElement) {
const bootstrapTimelineAccountsField = const bootstrapTimelineAccountsField =
document.querySelector<HTMLInputElement>( document.querySelector<HTMLInputElement>(
'#form_admin_settings_bootstrap_timeline_accounts', '#form_admin_settings_bootstrap_timeline_accounts',
@ -194,12 +188,11 @@ const onEnableBootstrapTimelineAccountsChange = (target: HTMLInputElement) => {
); );
} }
} }
}; }
Rails.delegate( on(
document,
'#form_admin_settings_enable_bootstrap_timeline_accounts',
'change', 'change',
'#form_admin_settings_enable_bootstrap_timeline_accounts',
({ target }) => { ({ target }) => {
if (target instanceof HTMLInputElement) if (target instanceof HTMLInputElement)
onEnableBootstrapTimelineAccountsChange(target); onEnableBootstrapTimelineAccountsChange(target);
@ -239,11 +232,11 @@ const onChangeRegistrationMode = (target: HTMLSelectElement) => {
}); });
}; };
const convertUTCDateTimeToLocal = (value: string) => { function convertUTCDateTimeToLocal(value: string) {
const date = new Date(value + 'Z'); const date = new Date(value + 'Z');
const twoChars = (x: number) => x.toString().padStart(2, '0'); const twoChars = (x: number) => x.toString().padStart(2, '0');
return `${date.getFullYear()}-${twoChars(date.getMonth() + 1)}-${twoChars(date.getDate())}T${twoChars(date.getHours())}:${twoChars(date.getMinutes())}`; return `${date.getFullYear()}-${twoChars(date.getMonth() + 1)}-${twoChars(date.getDate())}T${twoChars(date.getHours())}:${twoChars(date.getMinutes())}`;
}; }
function convertLocalDatetimeToUTC(value: string) { function convertLocalDatetimeToUTC(value: string) {
const date = new Date(value); const date = new Date(value);
@ -251,14 +244,9 @@ function convertLocalDatetimeToUTC(value: string) {
return fullISO8601.slice(0, fullISO8601.indexOf('T') + 6); return fullISO8601.slice(0, fullISO8601.indexOf('T') + 6);
} }
Rails.delegate( on('change', '#form_admin_settings_registrations_mode', ({ target }) => {
document, if (target instanceof HTMLSelectElement) onChangeRegistrationMode(target);
'#form_admin_settings_registrations_mode', });
'change',
({ target }) => {
if (target instanceof HTMLSelectElement) onChangeRegistrationMode(target);
},
);
async function mountReactComponent(element: Element) { async function mountReactComponent(element: Element) {
const componentName = element.getAttribute('data-admin-component'); const componentName = element.getAttribute('data-admin-component');
@ -305,7 +293,7 @@ ready(() => {
if (registrationMode) onChangeRegistrationMode(registrationMode); if (registrationMode) onChangeRegistrationMode(registrationMode);
const checkAllElement = document.querySelector<HTMLInputElement>( const checkAllElement = document.querySelector<HTMLInputElement>(
'input#batch_checkbox_all', '#batch_checkbox_all',
); );
if (checkAllElement) { if (checkAllElement) {
const allCheckboxes = Array.from( const allCheckboxes = Array.from(
@ -318,7 +306,7 @@ ready(() => {
} }
document document
.querySelector('a#add-instance-button') .querySelector<HTMLAnchorElement>('a#add-instance-button')
?.addEventListener('click', (e) => { ?.addEventListener('click', (e) => {
const domain = document.querySelector<HTMLInputElement>( const domain = document.querySelector<HTMLInputElement>(
'input[type="text"]#by_domain', 'input[type="text"]#by_domain',
@ -342,7 +330,7 @@ ready(() => {
} }
}); });
Rails.delegate(document, 'form', 'submit', ({ target }) => { on('submit', 'form', ({ target }) => {
if (target instanceof HTMLFormElement) if (target instanceof HTMLFormElement)
target target
.querySelectorAll<HTMLInputElement>('input[type="datetime-local"]') .querySelectorAll<HTMLInputElement>('input[type="datetime-local"]')

View File

@ -73,6 +73,7 @@
"cross-env": "^7.0.3", "cross-env": "^7.0.3",
"css-loader": "^5.2.7", "css-loader": "^5.2.7",
"cssnano": "^7.0.0", "cssnano": "^7.0.0",
"delegated-events": "^1.1.2",
"detect-passive-events": "^2.0.3", "detect-passive-events": "^2.0.3",
"emoji-mart": "npm:emoji-mart-lazyload@latest", "emoji-mart": "npm:emoji-mart-lazyload@latest",
"escape-html": "^1.0.3", "escape-html": "^1.0.3",

View File

@ -2874,6 +2874,7 @@ __metadata:
cross-env: "npm:^7.0.3" cross-env: "npm:^7.0.3"
css-loader: "npm:^5.2.7" css-loader: "npm:^5.2.7"
cssnano: "npm:^7.0.0" cssnano: "npm:^7.0.0"
delegated-events: "npm:^1.1.2"
detect-passive-events: "npm:^2.0.3" detect-passive-events: "npm:^2.0.3"
emoji-mart: "npm:emoji-mart-lazyload@latest" emoji-mart: "npm:emoji-mart-lazyload@latest"
escape-html: "npm:^1.0.3" escape-html: "npm:^1.0.3"
@ -7350,6 +7351,15 @@ __metadata:
languageName: node languageName: node
linkType: hard linkType: hard
"delegated-events@npm:^1.1.2":
version: 1.1.2
resolution: "delegated-events@npm:1.1.2"
dependencies:
selector-set: "npm:^1.1.5"
checksum: 10c0/b295a6d6c6cef4b9312bfd4132ac3a1255f3c2fadf3692a04cf7ddf8f0d472bfce9de06323faa75e922d20e5674d45022e1a5378b8500cd7d573ffa0cf7ca602
languageName: node
linkType: hard
"denque@npm:^2.1.0": "denque@npm:^2.1.0":
version: 2.1.0 version: 2.1.0
resolution: "denque@npm:2.1.0" resolution: "denque@npm:2.1.0"
@ -15827,6 +15837,13 @@ __metadata:
languageName: node languageName: node
linkType: hard linkType: hard
"selector-set@npm:^1.1.5":
version: 1.1.5
resolution: "selector-set@npm:1.1.5"
checksum: 10c0/4835907846eb8496c2cc4e5ce48355cce1ef3b55e82789542739dcdc71ebfb756e133d1d7f7ec9f0cf4b1c11fc0375a1d3b99a482d9c973493ca85a6d4b012ab
languageName: node
linkType: hard
"selfsigned@npm:^1.10.8": "selfsigned@npm:^1.10.8":
version: 1.10.14 version: 1.10.14
resolution: "selfsigned@npm:1.10.14" resolution: "selfsigned@npm:1.10.14"