fix(site): fix exec tool layout shift on command expand#23739
Merged
DanielleMaywood merged 1 commit intomainfrom Mar 28, 2026
Merged
fix(site): fix exec tool layout shift on command expand#23739DanielleMaywood merged 1 commit intomainfrom
DanielleMaywood merged 1 commit intomainfrom
Conversation
- Remove conditional items-center/items-start toggle on the exec tool header that caused the command text to jump vertically when expanding. Use items-start unconditionally so nothing shifts. - Align horizontal padding from px-2.5 to px-3 across ExecuteTool and ProcessOutputTool to match other bordered tool components. - Add py-2 vertical padding on the exec tool header for consistency with other bordered tools. - Shrink the CopyButton in the exec header from size-8 to size-6 with negative margin so it no longer stretches the row height beyond the text line. - Add hover:bg-surface-tertiary to the CopyButton to match the edit button hover style on user messages. - Fix user message edit button using negative margin instead of positive top margin to prevent it from stretching the message row.
kylecarbs
approved these changes
Mar 28, 2026
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
The exec tool command header toggled between
items-centeranditems-startwhen expanding, causing the text to jump vertically. The CopyButton (size-8, 32px) also stretched the header row beyond the text line-height.Use
items-startunconditionally so nothing shifts on expand. Shrink the CopyButton tosize-6with negative margin to match the text line-height. Align padding (px-3,py-2) with other bordered tool components. Addhover:bg-surface-tertiaryto the CopyButton for consistency with the edit button hover style on user messages.Same negative-margin fix applied to the user message edit button.