From c28311ceae6ad2eac7bf3ae848059d37e1352545 Mon Sep 17 00:00:00 2001
From: cc <codecolorist@gmail.com>
Date: Tue, 14 Apr 2026 17:56:36 +0200
Subject: [PATCH] Move navigation links to top right, fix base path

- Links now on same row as breadcrumb/version switcher
- Removed addBasePath from Link hrefs (Next.js handles it)
---
 src/app/os/layout.tsx | 74 ++++++++++++++++++++++---------------------
 1 file changed, 38 insertions(+), 36 deletions(-)

diff --git a/src/app/os/layout.tsx b/src/app/os/layout.tsx
index 51c8868..ad6bd0b 100644
--- a/src/app/os/layout.tsx
+++ b/src/app/os/layout.tsx
@@ -39,44 +39,46 @@ export default function OSDetailLayout({
 
   return (
     <div className="p-4 md:p-8" suppressHydrationWarning>
-      <header className="mb-6 space-y-3">
+      <header className="mb-6">
         <div className="flex flex-col sm:flex-row sm:items-center gap-4">
-          <Breadcrumb>
-            <BreadcrumbList>
-              <BreadcrumbItem>
-                <BreadcrumbLink href={addBasePath("/")}>Home</BreadcrumbLink>
-              </BreadcrumbItem>
-              <BreadcrumbSeparator />
-              <BreadcrumbItem>
-                <span className="text-muted-foreground">
-                  {os?.split("/")[0]}
-                </span>
-              </BreadcrumbItem>
-            </BreadcrumbList>
-          </Breadcrumb>
-          <VersionSwitcher currentOs={os} />
-        </div>
+          <div className="flex items-center gap-4">
+            <Breadcrumb>
+              <BreadcrumbList>
+                <BreadcrumbItem>
+                  <BreadcrumbLink href={addBasePath("/")}>Home</BreadcrumbLink>
+                </BreadcrumbItem>
+                <BreadcrumbSeparator />
+                <BreadcrumbItem>
+                  <span className="text-muted-foreground">
+                    {os?.split("/")[0]}
+                  </span>
+                </BreadcrumbItem>
+              </BreadcrumbList>
+            </Breadcrumb>
+            <VersionSwitcher currentOs={os} />
+          </div>
 
-        <nav className="flex items-center gap-4 text-sm">
-          <Link
-            href={addBasePath(`/os/keys?os=${os}`)}
-            className={currentPage === "keys" ? "font-medium" : "text-muted-foreground hover:text-foreground"}
-          >
-            Entitlement Keys
-          </Link>
-          <Link
-            href={addBasePath(`/os/files?os=${os}`)}
-            className={currentPage === "files" ? "font-medium" : "text-muted-foreground hover:text-foreground"}
-          >
-            Browse Files
-          </Link>
-          {currentPage === "find" && (
-            <span className="font-medium">Search Results</span>
-          )}
-          {currentPage === "bin" && (
-            <span className="font-medium">Binary Detail</span>
-          )}
-        </nav>
+          <nav className="flex items-center gap-4 text-sm sm:ml-auto">
+            <Link
+              href={`/os/keys?os=${os}`}
+              className={currentPage === "keys" ? "font-medium" : "text-muted-foreground hover:text-foreground"}
+            >
+              Entitlement Keys
+            </Link>
+            <Link
+              href={`/os/files?os=${os}`}
+              className={currentPage === "files" ? "font-medium" : "text-muted-foreground hover:text-foreground"}
+            >
+              Browse Files
+            </Link>
+            {currentPage === "find" && (
+              <span className="font-medium">Search Results</span>
+            )}
+            {currentPage === "bin" && (
+              <span className="font-medium">Binary Detail</span>
+            )}
+          </nav>
+        </div>
       </header>
 
       <div>{children}</div>