[pascalandr]

Summary

Large logos follow their title instead of the window, on the plain empty state's ratio (6rem logo above a text-3xl title, 3.2x):

• Home page and branded empty session: 6rem (the home page's 12rem rule from 640 x 721 px goes; the session drops h-48).
• Narrow session column: the title shrinks to 1.5rem, so the logo shows 4.8rem.
• Loading screen: 8rem (128 px) beside its larger 2.8rem title, instead of 180 px.
• dev-docs/UI_CONVENTIONS.md records the rule.

Validation

• Real home page, branded and plain empty states, narrow session column and loading screen in dark and light at 1100 x 760: logo/title 96/30, 96/30, 96/30, 76.8/24 and 128/44.8 px.
• Tried in the owner's installed app through a temporary override.
• npm run typecheck --workspace @codenomad/ui.

[pascalandr]

Review: zero findings

Reviewed commit d71e3c667 against bbda09754. No evidenced P0–P3 problems or required fixes.

• Inspected the real components at seven home-page viewport sizes (390–2560 px wide, including the 640 px / 721 px breakpoints), four session-column widths (280–1100 px), and four loading-screen viewport sizes. Logos rendered at 128 × 128 px; hero spacing, narrow-column fit and loading-screen separation remained intact. The loading image's width="128" height="128" attributes match the CSS and square PNG.
• Found no tests, developer documentation or help pages relying on the removed logo sizes. The commit message accurately describes the change and follows the repository's commit format. No affected help-page content needs updating.
• UI typecheck passed: npm run typecheck from packages/ui.
• Browser tests passed: 23/23, using node --import tsx --test --test-concurrency=1 tests/browser/session-interrupt.test.ts tests/browser/new-instance-launcher.test.ts tests/browser/session-aside.test.ts from packages/ui.
• git diff --check bbda09754...HEAD passed. This review made no repository changes.

[pascalandr]

Round 2 — 8790d7a43

zero findings (P0–P3).

• Checked home hero spacing, normal/narrow branded empty states, the plain empty state and loading screen using the real components and full stylesheet. All 40 layout checks passed across 390×760, 1100×600, 1100×760 and 1600×1000 in light/dark mode.
• Verified the real session shell at 600, 1100 and 1600px viewport widths: narrow columns compute the logo height to 76.7969px beside a 24px title; the medium column computes 96px beside 30px. The narrow selector's specificity (0,3,0) beats h-24 (0,1,0) despite the utility appearing later. Home/plain/normal logos are 96px; loading is 128px beside 44.8px, matching the owner's chosen sizes.
• Searched packages/ui/src for other logo/title placements and sizing rules. All large placements are covered; the native titlebar uses its separate small mark. The commit message, PR body and dev-docs/UI_CONVENTIONS.md:14 match the code. The new documentation describes current behavior and satisfies the repository's no-history rule.

Validation: npm run typecheck --workspace @codenomad/ui passed. node --import tsx --test --test-concurrency=1 tests/browser/session-interrupt.test.ts tests/browser/new-instance-launcher.test.ts tests/browser/session-rendering.test.ts passed 59/59, with no failures or skips. Review-only: no repository files changed or pushed.