From 8c1e6b2f7a5b8f779a51f4f7bd052e9282c4fb64 Mon Sep 17 00:00:00 2001 From: meisolated Date: Mon, 17 Aug 2026 15:04:19 +0530 Subject: [PATCH 1/3] fix(web): align the sidebar T3 Code lockup on a shared baseline The wordmark and "Code" were centered as two separate boxes with a -translate-y-px nudge on the text. Centering aligns boxes, not baselines, so the pair only lined up for a font whose ascent and descent match the default stack: with a custom interface font (Settings -> Appearance) the word drifted ~2px above the mark, and a fixed 1px nudge does not scale with the interface font size either. Put the mark and the word in one line box so the mark rides the text baseline as an inline replaced element, and replace the fixed gap with a real word space that scales with the font. --- .../src/components/sidebar/SidebarChrome.tsx | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/apps/web/src/components/sidebar/SidebarChrome.tsx b/apps/web/src/components/sidebar/SidebarChrome.tsx index f4a98dec86c7..05eafc336b97 100644 --- a/apps/web/src/components/sidebar/SidebarChrome.tsx +++ b/apps/web/src/components/sidebar/SidebarChrome.tsx @@ -83,19 +83,18 @@ function SidebarBrand({ onBackdrop }: { onBackdrop: boolean }) { - - - Code + {/* Mark and word share one line box so the mark rides the text baseline. + Centering the two as separate boxes only lines up for a font whose + ascent and descent match the default stack, and the interface font is + a user setting. */} + + {" "} + Code ); @@ -105,7 +104,7 @@ function T3Wordmark() { return ( From 54758b7e1f8fdcc74462ea3cf24fb7231893e8a7 Mon Sep 17 00:00:00 2001 From: meisolated Date: Mon, 17 Aug 2026 15:10:01 +0530 Subject: [PATCH 2/3] refactor(web): drop the sidebar brand comment --- apps/web/src/components/sidebar/SidebarChrome.tsx | 4 ---- 1 file changed, 4 deletions(-) diff --git a/apps/web/src/components/sidebar/SidebarChrome.tsx b/apps/web/src/components/sidebar/SidebarChrome.tsx index 05eafc336b97..baf261d27627 100644 --- a/apps/web/src/components/sidebar/SidebarChrome.tsx +++ b/apps/web/src/components/sidebar/SidebarChrome.tsx @@ -88,10 +88,6 @@ function SidebarBrand({ onBackdrop }: { onBackdrop: boolean }) { )} to="/" > - {/* Mark and word share one line box so the mark rides the text baseline. - Centering the two as separate boxes only lines up for a font whose - ascent and descent match the default stack, and the interface font is - a user setting. */} {" "} Code From 0b8bf89fb817d98f247f49b3af9bc183819ae205 Mon Sep 17 00:00:00 2001 From: meisolated Date: Mon, 17 Aug 2026 15:17:27 +0530 Subject: [PATCH 3/3] test(web): follow the sidebar lockup class change The wordmark guard pinned the exact class substring, which included the gap-1 the lockup no longer uses now that the space is a real word space. --- apps/web/src/components/threadSidebarWidth.test.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/apps/web/src/components/threadSidebarWidth.test.ts b/apps/web/src/components/threadSidebarWidth.test.ts index e38d5c3749bc..0a75c1fad846 100644 --- a/apps/web/src/components/threadSidebarWidth.test.ts +++ b/apps/web/src/components/threadSidebarWidth.test.ts @@ -41,7 +41,9 @@ describe("thread sidebar width", () => { "utf8", ); - expect(sidebarSource).toContain("hidden h-7 w-fit min-w-0 shrink-0 items-center gap-1"); + expect(sidebarSource).toContain( + "hidden h-7 w-fit min-w-0 shrink-0 items-center overflow-hidden", + ); expect(sidebarSource).toContain("md:flex"); expect(THREAD_SIDEBAR_MIN_WIDTH).toBe(13 * 16); });