fix(answer-dock): stop follow-up "Try next" chips overlapping the composer pill - #733
Conversation
…poser pill On phones the answer dock renders the "Try next" follow-up suggestion row directly above the search pill. Its only vertical separator was a negative margin-bottom (-0.125rem), which tucked the chips' rounded underside beneath the pill's top edge and focus glow, reading as a collision on the search bar. Flip that to a small positive gap (0.4375rem) so the chip row and pill read as distinct, intentionally-spaced tiers without pushing the row up into the answer content that shows through the frosted scrim above. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UnnEvbSGcJevNnPKhgb6xo
This pull request has been ignored for the connected project Preview Branches by Supabase. |
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe phone composer follow-up row now uses positive bottom spacing, with updated documentation describing separation from the answer pill and focus glow. ChangesMobile composer spacing
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Summary
margin-bottom(-0.125rem) on.answer-footer-search-edge .answer-suggestion-row-composer-followups, which tucked the chips' rounded underside beneath the pill's top edge and focus glow — reading as a collision on the search bar (the reported issue).0.4375rem/ 7px) so the chip row and the "Ask a follow-up…" pill read as distinct, intentionally-spaced tiers. The gap is deliberately kept modest so the row isn't pushed further up into the answer content that shows through the frosted scrim above.src/app/globals.css. Scoped to the phone answer dock (answer-suggestion-row-composer-followupsissm:hidden), so tablet/desktop and the in-thread follow-up layout are untouched.Verified live against the running app with a Chromium screenshot pass — measured chip-row bottom
748→739vs pill top746, turning a −2px overlap into a +7px gap.Verification
npm run verify:pr-localnpm run verify:uiVerification not run: the automated gate (
scripts/run-playwright.mjs) builds an isolated production app that fails typecheck on a pre-existing, unrelated error (tests/setup/jsdom.setup.tsis missing type declarations for@testing-library/jest-dom/vitest); this change is CSS-only and touches no TypeScript.UI verification not run: the same pre-existing isolated-build typecheck failure blocks
npm run verify:ui. Instead the fix was verified directly in a real browser (Playwright + Chromium againstnpm run ensure), capturing the phone answer dock at the natural and scrolled-to-bottom positions and measuring the chip↔pill geometry (−2px overlap → +7px gap). Existingui-smoke.spec.tsfollow-up tests assert visibility/interaction only (no pixel geometry), so behavior is unaffected.Risk and rollout
<main>scroll reserve, or scrim geometry.-0.125remmargin.Clinical Governance Preflight
Not applicable — this change is purely the vertical spacing of the composer's follow-up suggestion chips. It does not touch ingestion, answer generation, retrieval/ranking, source/citation rendering, document access, privacy, Supabase, production env, or clinical output. No answer content, source metadata, or access behavior changes.
Notes
🤖 Generated with Claude Code
https://claude.ai/code/session_01UnnEvbSGcJevNnPKhgb6xo
Summary by CodeRabbit