Skip to content

fix(comments): chip sits beside the send button; message marker moves to the bottom-right - #288

Merged
bk-agent-01 merged 2 commits into
stagefrom
t3code/comments-chip-inside
Oct 8, 2026
Merged

bk-agent-01 merged 2 commits into
stagefrom
t3code/comments-chip-inside

Conversation

@bk-agent-01

Copy link
Copy Markdown
Collaborator

Problem

The review-comment chip from #286 sat on the top edge of the chat box. Banners above the box (for example "Resume with less context") covered it. In the one-line chat box, it hung over the chat.

The "💬 N" marker on an agent message sat at the message's top-right corner. When a message starts with a code block, it covered the block's wrap and copy buttons.

Fix

  • The chip now renders inline in the chat box's action row, just before the send button. Both chat box layouts keep this row. So the chip is always inside the box and never covers the draft.
  • The message marker moves to the bottom-right corner of the message.

All changes are in fork files, plus a move of the one marked seam line in ChatComposer.tsx.

Evidence

CI runs the checks. No screenshots: the preview browser cannot run on this host.

🤖 Generated with Claude Code

… to the bottom-right

The composer chip straddled the chat box's top edge, where banners above the
box covered it and, in the one-line composer, it hung over the chat. It now
renders inline in the composer's action row, just before the send button,
which both composer layouts keep, so it is always inside the box and never
covers the draft.

The per-message comment marker sat at the message's top-right, on top of a
leading code block's wrap and copy buttons; it now sits at the bottom-right.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S labels Oct 8, 2026
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@bk-agent-01
bk-agent-01 merged commit d980090 into stage Oct 8, 2026
25 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 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