From f9e909c078ac3a893b69a397b4425476f488d40d Mon Sep 17 00:00:00 2001 From: Federico Mastrini Date: Mon, 10 Aug 2026 16:10:56 +0200 Subject: [PATCH 1/5] fix(frontend): align expanded title-bar breadcrumbs - Increase expanded title-bar leading inset to match page content - Prevent accidental body selection while preserving input text selection --- .../components/__tests__/application-title-bar.test.tsx | 4 ++-- apps/frontend/src/components/application-title-bar.tsx | 2 +- apps/frontend/src/styles.css | 8 +++++++- 3 files changed, 10 insertions(+), 4 deletions(-) diff --git a/apps/frontend/src/components/__tests__/application-title-bar.test.tsx b/apps/frontend/src/components/__tests__/application-title-bar.test.tsx index 51594a82..22de618a 100644 --- a/apps/frontend/src/components/__tests__/application-title-bar.test.tsx +++ b/apps/frontend/src/components/__tests__/application-title-bar.test.tsx @@ -74,7 +74,7 @@ describe("ApplicationTitleBar", () => { ).toBeNull(); }); - it("keeps title bar free of chrome inset when sidebar is expanded", () => { + it("aligns expanded title-bar breadcrumbs with page content", () => { renderTitleBar("tauri", ); const banner = screen.getByRole("banner"); const leading = banner.querySelector('[data-slot="title-bar-leading"]'); @@ -82,7 +82,7 @@ describe("ApplicationTitleBar", () => { const routeAction = screen.getByRole("button", { name: "Route action" }); // Expanded desktop: fixed toggle sits over the sidebar, not the content title bar. - expect(leading?.style.paddingLeft).toBe("0.5rem"); + expect(leading?.style.paddingLeft).toBe("1.5rem"); expect(banner.querySelector("[data-tauri-drag-region]")).toBeNull(); expect(dragRegion).not.toBeNull(); if (!dragRegion) { diff --git a/apps/frontend/src/components/application-title-bar.tsx b/apps/frontend/src/components/application-title-bar.tsx index 88fa6219..81b1f2de 100644 --- a/apps/frontend/src/components/application-title-bar.tsx +++ b/apps/frontend/src/components/application-title-bar.tsx @@ -114,7 +114,7 @@ export function ApplicationTitleBar({ buildTarget }: ApplicationTitleBarProps) { ? hasNativeMacWindowChrome ? NATIVE_CHROME_LEADING_INSET : WEB_CHROME_LEADING_INSET - : "0.5rem", + : "1.5rem", }; return ( diff --git a/apps/frontend/src/styles.css b/apps/frontend/src/styles.css index ca1088bd..962d1b09 100644 --- a/apps/frontend/src/styles.css +++ b/apps/frontend/src/styles.css @@ -167,7 +167,13 @@ @apply border-border outline-ring/50; } body { - @apply bg-background text-foreground; + @apply select-none bg-background text-foreground; + } + input, + textarea, + [contenteditable="true"], + [contenteditable="plaintext-only"] { + user-select: text; } [data-sonner-toaster] { --width: 356px; From 065d7c59518fc1841d6b028e58f53ea7e74aa02b Mon Sep 17 00:00:00 2001 From: Federico Mastrini Date: Mon, 10 Aug 2026 16:53:05 +0200 Subject: [PATCH 2/5] fix(frontend): remove sidebar rail from root layout - Drop `SidebarRail` from the app sidebar - Align root layout spacing with the breadcrumb area --- apps/frontend/src/routes/__root.tsx | 2 -- 1 file changed, 2 deletions(-) diff --git a/apps/frontend/src/routes/__root.tsx b/apps/frontend/src/routes/__root.tsx index 1cc4f54c..6ef22dac 100644 --- a/apps/frontend/src/routes/__root.tsx +++ b/apps/frontend/src/routes/__root.tsx @@ -29,7 +29,6 @@ import { SidebarMenuSubButton, SidebarMenuSubItem, SidebarProvider, - SidebarRail, SidebarTrigger, useSidebar, } from "@/components/ui/sidebar"; @@ -281,7 +280,6 @@ function AppSidebar({ buildTarget: sidebarBuildTarget }: AppSidebarProps) { - ); } From 54f580fd34bed4d6743aa25ee768e3e4b5adce14 Mon Sep 17 00:00:00 2001 From: Federico Mastrini Date: Mon, 10 Aug 2026 16:59:36 +0200 Subject: [PATCH 3/5] fix: tighten sidebar brand mark spacing - Remove the extra horizontal gap in the brand mark - Keep the optical vertical nudge intact --- apps/frontend/src/routes/__root.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/frontend/src/routes/__root.tsx b/apps/frontend/src/routes/__root.tsx index 6ef22dac..4e00f036 100644 --- a/apps/frontend/src/routes/__root.tsx +++ b/apps/frontend/src/routes/__root.tsx @@ -132,7 +132,7 @@ function SidebarBrandMark({ className }: { className?: string }) { )} > {/* Optical nudge: text-lg metrics sit slightly high in the 48px strip. */} - + {/* size-8 matches icon menu buttons in the rail. */} 財 From f8d797ea226747d8ec1cd935341693aff670eac5 Mon Sep 17 00:00:00 2001 From: Federico Mastrini Date: Mon, 10 Aug 2026 17:25:26 +0200 Subject: [PATCH 4/5] fix(frontend): align title bar and screen padding - Reduce title bar leading inset and action padding - Match screen content horizontal padding to the new spacing --- .../src/components/__tests__/application-title-bar.test.tsx | 2 +- apps/frontend/src/components/application-title-bar.tsx | 4 ++-- apps/frontend/src/components/screen-base.tsx | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/apps/frontend/src/components/__tests__/application-title-bar.test.tsx b/apps/frontend/src/components/__tests__/application-title-bar.test.tsx index 22de618a..662b908e 100644 --- a/apps/frontend/src/components/__tests__/application-title-bar.test.tsx +++ b/apps/frontend/src/components/__tests__/application-title-bar.test.tsx @@ -82,7 +82,7 @@ describe("ApplicationTitleBar", () => { const routeAction = screen.getByRole("button", { name: "Route action" }); // Expanded desktop: fixed toggle sits over the sidebar, not the content title bar. - expect(leading?.style.paddingLeft).toBe("1.5rem"); + expect(leading?.style.paddingLeft).toBe("1rem"); expect(banner.querySelector("[data-tauri-drag-region]")).toBeNull(); expect(dragRegion).not.toBeNull(); if (!dragRegion) { diff --git a/apps/frontend/src/components/application-title-bar.tsx b/apps/frontend/src/components/application-title-bar.tsx index 81b1f2de..93c4f305 100644 --- a/apps/frontend/src/components/application-title-bar.tsx +++ b/apps/frontend/src/components/application-title-bar.tsx @@ -114,7 +114,7 @@ export function ApplicationTitleBar({ buildTarget }: ApplicationTitleBarProps) { ? hasNativeMacWindowChrome ? NATIVE_CHROME_LEADING_INSET : WEB_CHROME_LEADING_INSET - : "1.5rem", + : "1rem", }; return ( @@ -145,7 +145,7 @@ export function ApplicationTitleBar({ buildTarget }: ApplicationTitleBarProps) {
{actions} -
{children}
+
{children}
); From 3f3ea819b6a8e39bfb73ebc4867f962164cefaf6 Mon Sep 17 00:00:00 2001 From: Federico Mastrini Date: Mon, 10 Aug 2026 17:30:45 +0200 Subject: [PATCH 5/5] fix(frontend): adjust scroll area scrollbar size - Set vertical and horizontal scrollbar thickness to 0.375rem --- apps/frontend/src/styles.css | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/apps/frontend/src/styles.css b/apps/frontend/src/styles.css index 962d1b09..7e8e711f 100644 --- a/apps/frontend/src/styles.css +++ b/apps/frontend/src/styles.css @@ -12,6 +12,14 @@ border-radius: var(--radius-xl); } +[data-slot="scroll-area-scrollbar"][data-orientation="vertical"] { + width: 0.375rem; +} + +[data-slot="scroll-area-scrollbar"][data-orientation="horizontal"] { + height: 0.375rem; +} + @custom-variant dark (&:is(.dark *)); @theme inline {