From 6048fcb0530f38059b6b1f09bc58ec9f93e15eeb Mon Sep 17 00:00:00 2001 From: David Cook Date: Wed, 1 Apr 2026 11:08:54 +1100 Subject: [PATCH] Define function as member arrow function This way it behaves as an instance method, and we don't have to pass in the object. --- .../vertical_ellipsis_menu_controller.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/components/vertical_ellipsis_menu_component/vertical_ellipsis_menu_controller.js b/app/components/vertical_ellipsis_menu_component/vertical_ellipsis_menu_controller.js index dbbe442c84..4d47a9683c 100644 --- a/app/components/vertical_ellipsis_menu_component/vertical_ellipsis_menu_controller.js +++ b/app/components/vertical_ellipsis_menu_component/vertical_ellipsis_menu_controller.js @@ -8,7 +8,7 @@ export default class extends Controller { window.addEventListener("click", this.#hideIfClickedOutside); // Close menu when making a selection - this.contentTarget.addEventListener("click", this.#selected.bind(this)); + this.contentTarget.addEventListener("click", this.#selected); } disconnect() { @@ -19,7 +19,7 @@ export default class extends Controller { this.#toggleShow(); } - #selected() { + #selected = () => { this.contentTarget.classList.add("selected"); }