Put if check over whole block

yomidevs · Dec 17, 2024 · a5fac51 · a5fac51
1 parent 9251f28
commit a5fac51
Showing 1 changed file with 15 additions and 13 deletions.
diff --git a/ext/js/background/backend.js b/ext/js/background/backend.js
@@ -1331,19 +1331,21 @@ export class Backend {
         }
 
         try {
-            if (options.general.enableContextMenuScanSelected && chrome.contextMenus) {
-                chrome.contextMenus.create({
-                    id: 'yomitan_lookup',
-                    title: 'Lookup in Yomitan',
-                    contexts: ['selection'],
-                }, () => this._checkLastError(chrome.runtime.lastError));
-                chrome.contextMenus.onClicked.addListener((info) => {
-                    if (info.selectionText) {
-                        this._sendMessageAllTabsIgnoreResponse({action: 'frontendScanSelectedText'});
-                    }
-                });
-            } else {
-                chrome.contextMenus.remove('yomitan_lookup', () => this._checkLastError(chrome.runtime.lastError));
+            if (chrome.contextMenus) {
+                if (options.general.enableContextMenuScanSelected) {
+                    chrome.contextMenus.create({
+                        id: 'yomitan_lookup',
+                        title: 'Lookup in Yomitan',
+                        contexts: ['selection'],
+                    }, () => this._checkLastError(chrome.runtime.lastError));
+                    chrome.contextMenus.onClicked.addListener((info) => {
+                        if (info.selectionText) {
+                            this._sendMessageAllTabsIgnoreResponse({action: 'frontendScanSelectedText'});
+                        }
+                    });
+                } else {
+                    chrome.contextMenus.remove('yomitan_lookup', () => this._checkLastError(chrome.runtime.lastError));
+                }
             }
         } catch (e) {
             log.error(e);