fix(ui): prevent horizontal scrollbar from long tool call rows#425
fix(ui): prevent horizontal scrollbar from long tool call rows#425
Conversation
Clip tool card overflow and propagate min-width constraints so tool-args-preview text truncation works correctly through the flex layout chain. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 057e54b338
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
|
||
| .tool-card { | ||
| overflow: visible; | ||
| overflow: hidden; |
There was a problem hiding this comment.
Changing .tool-card to overflow: hidden clips the disclosure chevron on every expandable tool row. .tool-caret is intentionally pulled 8px to the left with margin-left: -8px (SessionModal.svelte:1596-1600), so once the card becomes a clipping container the caret is rendered outside the card and disappears. That removes the main visual cue that a tool call can be expanded/collapsed.
Useful? React with 👍 / 👎.
|
|
||
| .modal-content { | ||
| flex: 1; | ||
| overflow-x: hidden; |
There was a problem hiding this comment.
overflow-x: hidden on .modal-content makes some supported assistant markdown impossible to inspect. This component enables GFM (marked.setOptions({ breaks: true, gfm: true }) at lines 74-75), but within .markdown-content only pre blocks get their own horizontal scroller (SessionModal.svelte:2047-2052). Wide GFM tables and similar blocks currently depend on the modal’s outer x-scroll, so after this change their rightmost columns are clipped with no way to reach them.
Useful? React with 👍 / 👎.
1 participant
Summary
overflow-x: hiddento.modal-contentto prevent horizontal scrollbar when tool call rows are wider than the modal.tool-cardoverflow fromvisibletohiddenand addsmin-width: 0to.tool-headerso long content truncates instead of overflowingTest plan
🤖 Generated with Claude Code