fix(site): add padding below thinking-only assistant messages#24140
Draft
fix(site): add padding below thinking-only assistant messages#24140
Conversation
Reasoning content populates parsed.reasoning but not parsed.markdown, so hasCopyableContent evaluates to false and the action bar is not rendered. The action bar provides ~24px of invisible layout space below each message; without it, thinking-only assistant messages sit flush against the next user bubble. Widen the render condition to always show the action bar for assistant messages (empty when there is nothing to copy) and add min-h-6 so the div maintains height even without child buttons.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reasoning content populates
parsed.reasoningbut notparsed.markdown, sohasCopyableContentevaluates tofalseand the action bar is not rendered. The action bar provides ~24px of invisible layout space below each message; without it, thinking-only assistant messages sit flush against the next user bubble.Widen the render condition to always show the action bar for assistant messages (empty when there is nothing to copy) and add
min-h-6so the div maintains height even without child buttons.Implementation notes
Root cause —
ConversationTimeline.tsxline 636:For a thinking-only assistant message
parsed.markdownis"", sohasCopyableContentisfalseandisUserisfalse. The entire action bar div is omitted from the DOM, removing the ~24px spacer that normal messages get.Fix — two changes on that same block:
|| !isUserso the action bar always renders for assistant messages.min-h-6on the div so it keeps 24px height even when empty (no CopyButton).