From 5465b08643f565421837221fdb96016cb467b482 Mon Sep 17 00:00:00 2001 From: Tom Schuster Date: Tue, 17 Dec 2024 14:04:13 +0000 Subject: [PATCH] Bug 1918615 - Remove inline event handler from editBMPanel_folderTree. r=mak,places-reviewers Differential Revision: https://phabricator.services.mozilla.com/D232337 --- browser/components/places/content/editBookmark.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/browser/components/places/content/editBookmark.js b/browser/components/places/content/editBookmark.js index 8380a17eb4f2..9ab4e96cc473 100644 --- a/browser/components/places/content/editBookmark.js +++ b/browser/components/places/content/editBookmark.js @@ -1295,7 +1295,6 @@ ChromeUtils.defineLazyGetter(gEditItemOverlay, "_folderTree", () => { is="places-tree" data-l10n-id="bookmark-overlay-folders-tree" editable="true" - onselect="gEditItemOverlay.onFolderTreeSelect();" disableUserActions="true" hidecolumnpicker="true"> @@ -1305,7 +1304,11 @@ ChromeUtils.defineLazyGetter(gEditItemOverlay, "_folderTree", () => { `) ); - return gEditItemOverlay._element("folderTree"); + const folderTree = gEditItemOverlay._element("folderTree"); + folderTree.addEventListener("select", () => + gEditItemOverlay.onFolderTreeSelect() + ); + return folderTree; }); for (let elt of [