From 98657e444e95c7ba3c9c6d6c6e8ef2b4da984371 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andre=20Schr=C3=B6der?= Date: Sun, 20 Dec 2020 18:41:45 +0100 Subject: [PATCH] Fix breaking containers when adding tab with local file See https://github.com/mozilla/multi-account-containers/issues/1717 for a description of the underlying issue: > Hide/Show tabs in this container actually closes and reopens tabs > instead of hiding/showing, meaning it cannot reopen (show) privileged > tabs (file, chrome, about, etc.) This commit fixes #918 and #1817 --- src/js/background/backgroundLogic.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/js/background/backgroundLogic.js b/src/js/background/backgroundLogic.js index 7643221..a0db924 100644 --- a/src/js/background/backgroundLogic.js +++ b/src/js/background/backgroundLogic.js @@ -93,7 +93,8 @@ const backgroundLogic = { // We can't open these we just have to throw them away if (protocol === "about:" || protocol === "chrome:" - || protocol === "moz-extension:") { + || protocol === "moz-extension:" + || protocol === "file:") { return false; } return true;