fix: Migrate onSelectionChanged to onActivated (#461)

Fixes #153
melink14 · May 8, 2021 · d8a0a9d · d8a0a9d
1 parent 142b3c0
commit d8a0a9d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/extension/background.ts b/extension/background.ts
@@ -1,5 +1,7 @@
 chrome.browserAction.onClicked.addListener(rcxMain.inlineToggle);
-chrome.tabs.onSelectionChanged.addListener(rcxMain.onTabSelect);
+chrome.tabs.onActivated.addListener((activeInfo) => {
+  rcxMain.onTabSelect(activeInfo.tabId);
+});
 chrome.runtime.onMessage.addListener(function (request, sender, response) {
   switch (request.type) {
     case 'enable?':

diff --git a/extension/rikaichan.ts b/extension/rikaichan.ts
@@ -49,7 +49,7 @@ window.rcxMain = {
     return this.dict.init(this.haveNames);
   },
 
-  // The callback for onSelectionChanged
+  // The callback for `onActivated`
   // Just sends a message to the tab to enable itself if it hasn't
   // already
   onTabSelect: function (tabId) {