Skip to content

fix(web): polish recap header toggle#96

Merged
Jay1 merged 1 commit into
mainfrom
jcode/settings-goal-generation-model
Jun 12, 2026
Merged

fix(web): polish recap header toggle#96
Jay1 merged 1 commit into
mainfrom
jcode/settings-goal-generation-model

Conversation

@Jay1

@Jay1 Jay1 commented Jun 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • tighten Recap toggle typography to match adjacent Hand off header control
  • add breathing room around the Recap label
  • reduce Recap icon emphasis in the closed/default state

Verification

  • bunx oxfmt@0.52.0 --check apps/web/src/components/chat/ChatHeader.tsx
  • safe-run --profile test -- bun run --cwd apps/web test src/components/chat/ChatHeader.test.ts
  • safe-run --profile browser -- bun run --cwd apps/web test:browser src/components/chat/ChatHeader.browser.tsx
  • safe-run --profile build -- bun run --cwd apps/web typecheck
  • visual QA oracle pass A: PASS
  • visual QA oracle pass B: PASS

Summary by CodeRabbit

  • Style
    • Improved the visual styling of the Recap toggle button with enhanced spacing and text presentation.
    • Refined the icon sizing for better visual consistency within the interface.

@coderabbitai

coderabbitai Bot commented Jun 12, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 1e6c1b88-1f27-49da-bfc6-06d9ebc1531f

📥 Commits

Reviewing files that changed from the base of the PR and between 91bf26a and 6ffb2a8.

📒 Files selected for processing (1)
  • apps/web/src/components/chat/ChatHeader.tsx

📝 Walkthrough

Walkthrough

The PR updates the "Recap" toggle button in ChatHeader with refined styling: the Toggle component's className now applies explicit horizontal padding using CSS calc expressions and text-length variables, and the ListTodoIcon size is reduced from 3.5 to 3.

Changes

Recap Toggle Button Styling

Layer / File(s) Summary
Recap toggle styling update
apps/web/src/components/chat/ChatHeader.tsx
The "Recap" Toggle className is updated to use explicit horizontal padding and CSS text-length variables. The ListTodoIcon size is reduced from size-3.5 to size-3.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • Jay1/jcode#92: Introduces the "Recap" toggle wired to threadRecapOpen and onToggleThreadRecap; this PR refines its visual styling.

Suggested labels

vouch:trusted, size:XS

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix(web): polish recap header toggle' is specific and directly describes the main change—styling improvements to the Recap toggle button.
Description check ✅ Passed The description covers the key sections with clear summary of changes, detailed verification steps performed, and addresses all critical information needed for review.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added size:XS vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Jun 12, 2026
@Jay1 Jay1 merged commit 05a7d40 into main Jun 12, 2026
8 checks passed
@Jay1 Jay1 deleted the jcode/settings-goal-generation-model branch June 12, 2026 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant