Skip to content

fix(web): align tool detail layout - #1204

Merged
tiann merged 2 commits into
tiann:mainfrom
techotaku39:fix/web-tool-detail-spacing
Jul 28, 2026
Merged

fix(web): align tool detail layout#1204
tiann merged 2 commits into
tiann:mainfrom
techotaku39:fix/web-tool-detail-spacing

Conversation

@techotaku39

Copy link
Copy Markdown
Contributor

Summary

Polish the tool detail layout so inline cards and detail dialogs use consistent spacing and alignment across desktop and mobile.

Supersedes #1202, which was temporarily closed for manual validation and could not be reopened after the branch was rebased onto the latest main.

Changes

  • Match the gap between completed tool timing metadata and inline details to the dialog layout, including headers with and without subtitles.
  • Keep the existing spacing for pending tools, which do not show timing metadata.
  • Use the same 16px spacing between inline input and result sections as the detail dialog.
  • Keep tool detail dialog titles left-aligned on mobile without changing other dialogs, including image-share previews.
  • Optically align the close button with tool detail titles through a per-dialog override.
  • Align the generic Tool wrench icon with its label without changing Terminal icons.
  • Add regression coverage for single and grouped tool detail dialogs, inline spacing variants, and the generic Tool icon.

Testing

  • bun typecheck
  • bun run test:cli
  • bun run test:web (181 files, 1560 tests)
  • bun run test:shared (142 tests)
  • bun run build:web
  • git diff --check
  • Manual verification on desktop and mobile using the port 3016 test deployment

AI disclosure

AI-assisted with OpenAI Codex (GPT-5.6).

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Findings
No issues found in the added/modified lines.

Summary
Review mode: initial
No issues found. Residual risk: UI alignment still depends on visual behavior across compact ToolCard variants that are not screenshot-verified here.

Testing
Not run (automation): bun is not installed in this runner (/bin/bash: bun: command not found).

HAPI Bot

@tiann
tiann merged commit 8a77d2f into tiann:main Jul 28, 2026
2 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.

2 participants