rowan/multi-account-containers
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #593 from jonathanKingston/current-tab-window

Browse source 
Fix for current tab showing the wrong window. Fixes #592
This commit is contained in:
luke crouch 2017-06-15 10:30:06 -05:00 • committed by GitHub
commit 59f2b8a764
2 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
webextension/background.js
View file

@ -163,7 +163,6 @@ const assignManager = {
remove = true; remove = true;
} }
await this._setOrRemoveAssignment(tab.id, info.pageUrl, userContextId, remove); await this._setOrRemoveAssignment(tab.id, info.pageUrl, userContextId, remove);
this.calculateContextMenu(tab);
} }
}, },
@ -216,6 +215,8 @@ const assignManager = {
event: `${actionName}-container-assignment`, event: `${actionName}-container-assignment`,
userContextId: userContextId, userContextId: userContextId,
}); });
const tab = await browser.tabs.get(tabId);
this.calculateContextMenu(tab);
}, },
async _getAssignment(tab) { async _getAssignment(tab) {

2
webextension/js/popup.js
View file

@ -134,7 +134,7 @@ const Logic = {
}, },
async currentTab() { async currentTab() {
const activeTabs = await browser.tabs.query({active: true}); const activeTabs = await browser.tabs.query({active: true, windowId: browser.windows.WINDOW_ID_CURRENT});
if (activeTabs.length > 0) { if (activeTabs.length > 0) {
return activeTabs[0]; return activeTabs[0];
} }