From d48926eb2a782d4149d480519578e2de1153e895 Mon Sep 17 00:00:00 2001
From: Steven H <steven@gitbook.io>
Date: Tue, 5 Nov 2024 11:49:33 +0000
Subject: [PATCH] Fix an issue where the space dropdown was shown under the
 site sections in Safari. (#2567)

---
 .changeset/shiny-chefs-compare.md             |  5 ++
 .../SiteSectionTabs/SiteSectionTabs.tsx       | 89 ++++++++++---------
 2 files changed, 52 insertions(+), 42 deletions(-)
 create mode 100644 .changeset/shiny-chefs-compare.md

diff --git a/.changeset/shiny-chefs-compare.md b/.changeset/shiny-chefs-compare.md
new file mode 100644
index 000000000..d617799d8
--- /dev/null
+++ b/.changeset/shiny-chefs-compare.md
@@ -0,0 +1,5 @@
+---
+'gitbook': minor
+---
+
+Fix an issue where the space dropdown was shown under the site sections in Safari.
diff --git a/packages/gitbook/src/components/SiteSectionTabs/SiteSectionTabs.tsx b/packages/gitbook/src/components/SiteSectionTabs/SiteSectionTabs.tsx
index 04f81f0da..31bfd0c56 100644
--- a/packages/gitbook/src/components/SiteSectionTabs/SiteSectionTabs.tsx
+++ b/packages/gitbook/src/components/SiteSectionTabs/SiteSectionTabs.tsx
@@ -63,7 +63,7 @@ export function SiteSectionTabs(props: {
         <nav
             aria-label="Sections"
             ref={navRef}
-            className="flex flex-nowrap items-center mb-px"
+            className="flex flex-nowrap items-center mb-px max-w-screen-2xl mx-auto page-full-width:max-w-full"
             style={
                 {
                     '--tab-opacity': `${opacity}`,
@@ -72,47 +72,52 @@ export function SiteSectionTabs(props: {
                 } as React.CSSProperties
             }
         >
-            <div
-                className={tcls(
-                    'relative',
-                    'flex',
-                    'gap-2',
-                    'bg-transparent',
-
-                    // Horizontal padding, which is the layout padding minus the padding of the tabs themselves.
-                    'px-1',
-                    'sm:px-3',
-                    'md:px-5',
-
-                    /* add a pseudo element for active tab indicator */
-                    'after:block',
-                    "after:content-['']",
-                    'after:origin-left',
-                    'after:absolute',
-                    'after:-bottom-px',
-                    'after:left-0',
-                    'after:opacity-[--tab-opacity]',
-                    'after:scale-x-[--tab-scale]',
-                    'after:[transition:_opacity_150ms_25ms,transform_150ms]',
-                    'after:motion-reduce:transition-none',
-                    'after:translate-x-[var(--tab-start)]',
-                    'after:will-change-transform',
-                    'after:h-0.5',
-                    'after:w-[100px]',
-                    'after:bg-primary',
-                    'dark:after:bg-primary-400',
-                )}
-                role="tablist"
-            >
-                {tabs.map((tab, index) => (
-                    <Tab
-                        active={currentIndex === index}
-                        key={index + tab.path}
-                        label={tab.label}
-                        href={tab.path}
-                        ref={currentIndex === index ? currentTabRef : null}
-                    />
-                ))}
+            <div className="flex flex-col bg-transparent">
+                {/* An element for the tabs which includes the page padding */}
+                <div
+                    role="tablist"
+                    className={tcls(
+                        'flex flex-row gap-2',
+                        // Horizontal padding, which is the layout padding minus the padding of the tabs themselves.
+                        'px-1',
+                        'sm:px-3',
+                        'md:px-5',
+                    )}
+                >
+                    {tabs.map((tab, index) => (
+                        <Tab
+                            active={currentIndex === index}
+                            key={index + tab.path}
+                            label={tab.label}
+                            href={tab.path}
+                            ref={currentIndex === index ? currentTabRef : null}
+                        />
+                    ))}
+                </div>
+                {/* A container for a pseudo element for active tab indicator. A container is needed so we can set
+                    a relative position without breaking the z-index of other parts of the header. */}
+                <div
+                    className={tcls(
+                        'flex',
+                        'relative',
+                        'after:block',
+                        "after:content-['']",
+                        'after:origin-left',
+                        'after:absolute',
+                        'after:-bottom-px',
+                        'after:left-0',
+                        'after:opacity-[--tab-opacity]',
+                        'after:scale-x-[--tab-scale]',
+                        'after:[transition:_opacity_150ms_25ms,transform_150ms]',
+                        'after:motion-reduce:transition-none',
+                        'after:translate-x-[var(--tab-start)]',
+                        'after:will-change-transform',
+                        'after:h-0.5',
+                        'after:w-[100px]',
+                        'after:bg-primary',
+                        'dark:after:bg-primary-400',
+                    )}
+                ></div>
             </div>
         </nav>
     ) : null;