Bug 1898075 - Check for menu item in mousedown event. r=mtigley,crede…

…ntial-management-reviewers Differential Revision: https://phabricator.services.mozilla.com/D212314
ErichDonGubler · Jun 3, 2024 · c8653c3 · c8653c3
1 parent 030ddd1
commit c8653c3
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/toolkit/components/satchel/megalist/content/MegalistView.mjs b/toolkit/components/satchel/megalist/content/MegalistView.mjs
@@ -457,7 +457,12 @@ export class MegalistView extends MozLitElement {
       },
       { capture: true }
     );
-    popup.addEventListener("mousedown", e => e.preventDefault());
+    popup.addEventListener("mousedown", e => {
+      e.preventDefault();
+      if (e.target.classList != "menuItem") {
+        closeMenu();
+      }
+    });
     for (const command of snapshotData.commands) {
       if (command == "-") {
         const separator = this.ownerDocument.createElement("div");
@@ -468,6 +473,7 @@ export class MegalistView extends MozLitElement {
 
       const menuItem = this.ownerDocument.createElement("button");
       menuItem.setAttribute("role", "menuitem");
+      menuItem.classList.add("menuItem");
       menuItem.setAttribute("data-l10n-id", command.label);
       menuItem.addEventListener("click", e => {
         this.#messageToViewModel("Command", {