Skip to content

Polish pass 4: message overlap, fonts, header line, dropdown width#422

Merged
blove merged 5 commits into
mainfrom
claude/sidenav-polish-pass-4
May 18, 2026
Merged

Polish pass 4: message overlap, fonts, header line, dropdown width#422
blove merged 5 commits into
mainfrom
claude/sidenav-polish-pass-4

Conversation

@blove
Copy link
Copy Markdown
Contributor

@blove blove commented May 18, 2026

Summary

  • AI message controls overlap fix: `.chat-message__controls` was `position: absolute; left: 0; bottom: -28px` — combined with sub-project B's 16/12 actions padding, the actions row ended up overlapping the bubble and rendering inside the gutter column. Dropped the absolute positioning so the row flows naturally below the bubble inside `.chat-message__main`, indented past the gutter. Dot, text, and footer now line up.
  • Sidenav font cleanup: Search inherited browser-default 16px; New project rendered in Arial. Both now use `font-family: inherit; font-size: var(--ngaf-chat-font-size-sm)`. New chat sized down from 13px → 14px to match (weight 600 retained for CTA).
  • Top line above New chat removed: dropped `border-bottom` on `.chat-sidenav__header`.
  • More prompts dropdown widened: demo-side `::ng-deep` override on `.chat-select__menu` inside `.welcome-suggestions__row` — `min-width: 320px`, `max-width: 480px`. Model picker untouched.
  • @ngaf/* 0.0.40.

Test plan

  • `pnpm nx test chat` — green
  • `pnpm nx test examples-chat-angular` — 18 tests pass
  • Local chrome-mcp verify on all four fixes: actions now in flow, fonts consistent ui-sans-serif/14px, header line gone, dropdown widens to 350px for the longest label
  • Canonical demo redeploys; all four fixes live on demo.cacheplane.ai

🤖 Generated with Claude Code

@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cacheplane Ready Ready Preview, Comment May 18, 2026 5:47pm

Request Review

@blove blove merged commit c4e57f8 into main May 18, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant