From 8d99a97a9eac908ec78c68fb03a8fb88f00acc3d Mon Sep 17 00:00:00 2001
From: "mr. m" <mr.m@tuta.com>
Date: Mon, 19 May 2025 13:31:18 +0200
Subject: [PATCH] chore: Small details to split view, b=(no-bug), c=split-view,
 tabs

---
 src/zen/split-view/ZenViewSplitter.mjs  | 5 +++--
 src/zen/split-view/zen-decks.css        | 4 ++--
 src/zen/tabs/zen-tabs/vertical-tabs.css | 2 +-
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/zen/split-view/ZenViewSplitter.mjs b/src/zen/split-view/ZenViewSplitter.mjs
index 02b9cee9a..136ac90b0 100644
--- a/src/zen/split-view/ZenViewSplitter.mjs
+++ b/src/zen/split-view/ZenViewSplitter.mjs
@@ -147,7 +147,7 @@ class ZenViewSplitter extends ZenDOMOperatedFeature {
     if (groupIndex < 0) {
       return;
     }
-    this.removeTabFromGroup(tab, groupIndex, event.forUnsplit);
+    this.removeTabFromGroup(tab, groupIndex, true);
   }
 
   /**
@@ -188,6 +188,7 @@ class ZenViewSplitter extends ZenDOMOperatedFeature {
         this.resetTabState(remainingTab, forUnsplit);
       }
       this.removeGroup(groupIndex);
+      gBrowser.selectedTab = remainingTabs[remainingTabs.length - 1];
     } else {
       const node = this.getSplitNodeFromTab(tab);
       const toUpdate = this.removeNode(node);
@@ -1364,7 +1365,7 @@ class ZenViewSplitter extends ZenDOMOperatedFeature {
     }
     const container = event.currentTarget.closest('.browserSidebarContainer');
     const tab = window.gBrowser.tabs.find(
-      (t) => t.linkedBrowser.closest('.browserSidebarContainer') === container
+      (t) => t.linkedBrowser?.closest('.browserSidebarContainer') === container
     );
     if (tab) {
       window.gBrowser.selectedTab = tab;
diff --git a/src/zen/split-view/zen-decks.css b/src/zen/split-view/zen-decks.css
index 124fa908f..b85151a90 100644
--- a/src/zen/split-view/zen-decks.css
+++ b/src/zen/split-view/zen-decks.css
@@ -41,7 +41,7 @@
   }
 }
 
-#tabbrowser-tabpanels[zen-split-view='true']:not([zen-split-resizing]) > [zen-split-anim='true'] {
+#tabbrowser-tabpanels[zen-split-view='true']:not([zen-split-resizing]) > [zen-split='true'] {
   transition: inset 0.09s ease-out !important;
   & browser {
     transition: opacity 0.2s ease-out !important;
@@ -132,7 +132,7 @@
   align-items: center;
   position: fixed;
   padding: 0.4rem 0.6rem 0.2rem 0.6rem;
-  border-radius: 8px;
+  border-radius: var(--zen-native-content-radius);
   background-color: light-dark(rgba(255, 255, 255, 1), rgba(0, 0, 0, 1));
   box-shadow: 0 0 0 1px var(--button-primary-border-color);
   gap: 0.8rem;
diff --git a/src/zen/tabs/zen-tabs/vertical-tabs.css b/src/zen/tabs/zen-tabs/vertical-tabs.css
index 914085425..de3bc7751 100644
--- a/src/zen/tabs/zen-tabs/vertical-tabs.css
+++ b/src/zen/tabs/zen-tabs/vertical-tabs.css
@@ -1269,7 +1269,7 @@
   position: absolute;
 
   &[hidden='true'] {
-    --hidden-essentials-width: var(--zen-sidebar-width);
+    --hidden-essentials-width: calc(var(--zen-sidebar-width) + var(--zen-toolbox-padding));
     max-width: var(
       --hidden-essentials-width
     ) !important; /* To still allow essentials to grid the tabs */