From 1b5eea47505fa0f3f719dd8867856fda452dd8be Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jan=20B=C3=B6hmer?= Date: Sun, 28 Jan 2024 21:57:42 +0100 Subject: [PATCH] Register the unselect handler on turbo:load instead of visit, to avoid problems that the newly clicked treeview node gets unselected too. --- assets/controllers/elements/tree_controller.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/assets/controllers/elements/tree_controller.js b/assets/controllers/elements/tree_controller.js index bc829ff5..bb64839c 100644 --- a/assets/controllers/elements/tree_controller.js +++ b/assets/controllers/elements/tree_controller.js @@ -103,7 +103,6 @@ export default class extends Controller { this._registerURLWatcher(node); } }, - //onNodeContextmenu: contextmenu_handler, }, [BS5Theme, BS53Theme, FAIconTheme]); this.treeTarget.addEventListener(EVENT_INITIALIZED, (event) => { @@ -138,12 +137,13 @@ export default class extends Controller { node.setSelected(false, {silent: true, ignorePreventUnselect: true}); //Unregister the watcher - document.removeEventListener('turbo:visit', unselectNode); + document.removeEventListener('turbo:load', unselectNode); } }; //Register the watcher via hotwire turbo - document.addEventListener('turbo:visit', unselectNode); + //We must just load to have the new url in window.location + document.addEventListener('turbo:load', unselectNode); } _onContextMenu(event)