Skip to content

Commit 2dd9c52

Browse files
guasamclaude
andcommitted
fix(shell): stop the brand dot reading as a fourth traffic light
hiddenInset paints the native controls across x=20-72, but the titlebar only reserved 64px and the inner row added 12 more, so the dot's ring landed a pixel off the green button -- same row, same size, same circle. macOS already has its left edge anchored, so the dot now renders on win32/linux only, and the inset grows to 80px so the first menu button's hover pill does not take its place against the green button. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent d68dfcd commit 2dd9c52

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

‎app/shell/titlebar.tsx‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,13 @@ export function Titlebar({ title = 'Electron React App' }: { title?: string }) {
3333
<header
3434
className={cn(
3535
'relative flex h-10 shrink-0 items-center border-b border-border bg-card select-none [-webkit-app-region:drag]',
36-
isMac && 'pl-16'
36+
isMac && 'pl-20'
3737
)}
3838
>
3939
<div className="flex items-center gap-2.5 pl-3 [-webkit-app-region:no-drag]">
40-
<span className="size-3.25 rounded-lg bg-brand ring-[3px] ring-brand/20" />
40+
{/* The native traffic lights already anchor the left edge on macOS, where a same-sized dot
41+
beside them just reads as a fourth window control. */}
42+
{!isMac && <span className="size-3.25 rounded-lg bg-brand ring-[3px] ring-brand/20" />}
4143
{menuVisible && <TitlebarMenu />}
4244
</div>
4345

0 commit comments

Comments
 (0)