fix: snapshots: check if sidebar is open before throwing tabs to it

Author: mbnuqw

src/page.setup/components/snapshots.tab.vue

@@ -144,12 +144,10 @@ function onTabDragStart(e: DragEvent): void {
 }
 
 async function openTab(tab: SnapTabState): Promise<void> {
-  let activePanel
-  try {
-    activePanel = await IPC.sidebar(Windows.id, 'getActivePanelConfig')
-  } catch (err) {
-    Logs.err('snapshots.tab.openTab: Unable to getActivePanelConfig:', err)
-  }
+  const activePanel = await browser.sidebarAction
+    .isOpen({ windowId: Windows.id })
+    .then(isOpen => (isOpen ? IPC.sidebar(Windows.id, 'getActivePanelConfig') : undefined))
+    .catch(() => undefined)
 
   if (activePanel) {
     const item: ItemInfo = {

src/page.setup/components/snapshots.vue

@@ -281,7 +281,10 @@ async function openSelectedTabs(how: SnapOpenType): Promise<void> {
   }
 
   if (how === SnapOpenType.CurrentPanel) {
-    const activePanel = await IPC.sidebar(Windows.id, 'getActivePanelConfig').catch(() => undefined)
+    const activePanel = await browser.sidebarAction
+      .isOpen({ windowId: Windows.id })
+      .then(isOpen => (isOpen ? IPC.sidebar(Windows.id, 'getActivePanelConfig') : undefined))
+      .catch(() => undefined)
     if (Utils.isTabsPanel(activePanel)) {
       await IPC.sidebar(Windows.id, 'openTabs', items, { panelId: activePanel.id })
     } else if (activePanel) {