Merge pull request #32 from syt2/31-feature-add-icon-in-menu

add icon in menu
syt2 · Mar 27, 2024 · 2b8466a · 2b8466a
2 parents 0918abd + 4056902
commit 2b8466a
Showing 1 changed file with 3 additions and 7 deletions.
diff --git a/src/modules/addonTable.ts b/src/modules/addonTable.ts
@@ -49,22 +49,19 @@ export class AddonTable {
   static registerInMenuTool() {
     ztoolkit.Menu.register("menuTools", {
       tag: "menuseparator",
-      id: "addon-table-menuseparator-1"
+      id: "addon-table-menuseparator"
     });
     ztoolkit.Menu.register("menuTools", {
       tag: "menuitem",
       id: "addon-table-entrance",
       label: getString("menuitem-addons"),
+      icon: `chrome://${config.addonRef}/content/icons/favicon.svg`,
       commandListener: (event) => {
         (async () => {
           await this.showAddonsWindow({ from: "menu" });
         })();
       },
     });
-    ztoolkit.Menu.register("menuTools", {
-      tag: "menuseparator",
-      id: "addon-table-menuseparator-2"
-    });
   }
 
   /**
@@ -95,9 +92,8 @@ export class AddonTable {
 
   static unregisterAll() {
     document.querySelector("#zotero-toolbaritem-addons")?.remove();
-    ztoolkit.Menu.unregister("addon-table-menuseparator-1");
+    ztoolkit.Menu.unregister("addon-table-menuseparator");
     ztoolkit.Menu.unregister("addon-table-entrance");
-    ztoolkit.Menu.unregister("addon-table-menuseparator-2");
   }
 
   /**