Revert "Delegate events to the parent element"

This reverts commit e31c16df43.
This commit is contained in:
David Cook
2025-02-20 12:49:25 +11:00
parent e31c16df43
commit 7acc78b6d8

View File

@@ -7,7 +7,11 @@ export default class extends Controller {
connect() {
this.toggleCheckbox();
this.element.addEventListener("change", this.#toggleChangeListener.bind(this), {passive: true});
this.allTarget.addEventListener("change", this.toggleAll.bind(this));
this.checkboxTargets.forEach((checkbox) => {
checkbox.addEventListener("change", this.toggleCheckbox.bind(this));
});
}
toggleAll() {
@@ -35,15 +39,6 @@ export default class extends Controller {
// private
// Delegate events for targets (this ensures we catch events from newly-added elements after an ajax action)
#toggleChangeListener(event) {
if (event.target == this.allTarget) {
this.toggleAll();
} else if (this.checkboxTargets.includes(event.target)) {
this.toggleCheckbox();
}
}
#checkedCount() {
return this.checkboxTargets.filter((checkbox) => checkbox.checked).length;
}