Skip to content

Prevent chat tool-call rows from overflowing the message column#57

Merged
mgoldsborough merged 2 commits intomainfrom
fix/chat-tool-row-overflow
Apr 21, 2026
Merged

Prevent chat tool-call rows from overflowing the message column#57
mgoldsborough merged 2 commits intomainfrom
fix/chat-tool-row-overflow

Conversation

@mgoldsborough
Copy link
Copy Markdown
Contributor

Summary

  • Assistant message wrapper was sizing to its widest child because the parent flex column uses items-start. Tool-row summaries use white-space: nowrap, so their intrinsic width blew out the wrapper and the existing row-summary ellipsis never engaged.
  • Added w-full to the wrapper so it pins to the message column width — overflow-hidden on the wrapper plus text-overflow: ellipsis on .tool-accordion__row-summary now take effect.

Test plan

  • Open a chat with a multi-tool-call block where at least one tool has a long first argument (e.g. a long SQL question)
  • Confirm the row summary truncates with an ellipsis at the column edge instead of overflowing
  • Check both full-screen chat and compact/sidebar modes
  • Confirm prose and single-tool headers still render as before

The assistant message wrapper sized to its widest child because the
parent flex column uses items-start. Tool-row summaries use
white-space: nowrap, so their intrinsic width blew out the wrapper
and the existing row-summary ellipsis never engaged. Adding w-full
pins the wrapper to the column width so overflow-hidden + the row
summary's text-overflow: ellipsis take effect.
@mgoldsborough mgoldsborough merged commit 17d88ef into main Apr 21, 2026
4 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