Skip to content

fix(web): align the sidebar T3 Code lockup on a shared baseline #7312

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
meisolated wants to merge 5 commits into
base: main
Choose a base branch
from
+8 −11
Open

fix(web): align the sidebar T3 Code lockup on a shared baseline #7312

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
8c1e6b2
fix(web): align the sidebar T3 Code lockup on a shared baseline
meisolated Aug 17, 2026
54758b7
refactor(web): drop the sidebar brand comment
meisolated Aug 17, 2026
0b8bf89
test(web): follow the sidebar lockup class change
meisolated Aug 17, 2026
f3713a0
Merge branch 'main' into fix/sidebar-brand-baseline
meisolated Aug 18, 2026
ce1b7a8
Merge branch 'main' into fix/sidebar-brand-baseline
meisolated Aug 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 5 additions & 10 deletions apps/web/src/components/sidebar/SidebarChrome.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,19 +84,14 @@ function SidebarBrand({ onBackdrop }: { onBackdrop: boolean }) {
<Link
aria-label="Go to threads"
className={cn(
"relative z-10 ml-[var(--workspace-titlebar-content-left)] hidden h-7 w-fit min-w-0 shrink-0 items-center gap-1 overflow-hidden rounded-md outline-hidden ring-ring focus-visible:ring-2 md:flex",
"relative z-10 ml-[var(--workspace-titlebar-content-left)] hidden h-7 w-fit min-w-0 shrink-0 items-center overflow-hidden rounded-md outline-hidden ring-ring focus-visible:ring-2 md:flex",
Comment thread
macroscopeapp[bot] marked this conversation as resolved.
onBackdrop ? "text-white" : "text-foreground",
)}
to="/"
>
<T3Wordmark />
<span
className={cn(
"-translate-y-px truncate text-sm font-medium tracking-tight",
onBackdrop ? "text-white/70" : "text-muted-foreground",
)}
>
Code
<span className="truncate text-sm font-medium tracking-tight">
<T3Wordmark />{" "}
<span className={onBackdrop ? "text-white/70" : "text-muted-foreground"}>Code</span>
</span>
</Link>
);
Expand All @@ -106,7 +101,7 @@ function T3Wordmark() {
return (
<svg
aria-label="T3"
className="h-2.5 w-auto shrink-0"
className="inline h-2.5 w-auto align-baseline"
Comment thread
macroscopeapp[bot] marked this conversation as resolved.
viewBox="15.5309 37 94.3941 56.96"
xmlns="http://www.w3.org/2000/svg"
>
Expand Down
4 changes: 3 additions & 1 deletion apps/web/src/components/threadSidebarWidth.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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);
});
Expand Down
Loading