Skip to content

Fix excessive padding on terminal tool progress part action bar icons#291648

Merged
meganrogge merged 5 commits intomainfrom
copilot/fix-terminal-tool-padding
Jan 29, 2026
Merged

Fix excessive padding on terminal tool progress part action bar icons#291648
meganrogge merged 5 commits intomainfrom
copilot/fix-terminal-tool-padding

Conversation

Copy link
Contributor

Copilot AI commented Jan 29, 2026

Action bar icons in the terminal tool progress part had excessive horizontal spacing due to .interactive-item-container .progress-container .codicon { margin: 0 .2em; } applying to all codicons, including those inside .monaco-action-bar.

Changes

  • Add CSS rule in chatTerminalToolProgressPart.css to reset margin specifically for the terminal tool progress part action bar, matching standard action bar spacing
.chat-terminal-content-part .chat-terminal-action-bar .monaco-action-bar .codicon {
    margin: 0;
}

This scoped fix ensures only the chat terminal action bar icons are affected, without impacting other monaco action bars elsewhere in the UI.

Original prompt

This section details on the original issue you should resolve

<issue_title>Padding on terminal tool progress part icons is too large</issue_title>
<issue_description>Regressed recently?

Image

Version: 1.109.0-insider (user setup)
Commit: abb70fb
Date: 2026-01-26T05:02:47.498Z
Electron: 39.2.7
ElectronBuildId: 13098910
Chromium: 142.0.7444.235
Node.js: 22.21.1
V8: 14.2.231.21-electron.0
OS: Windows_NT x64 10.0.26200</issue_description>

Comments on the Issue (you are @copilot in this section)

@meganrogge Looks like that's coming from here

Image</comment_new>
<comment_new>@meganrogge
Which is not new...</comment_new>
<comment_new>@Tyriar
It's actually the margin from this:

Image Image

It should have similar spacing to other action bars:

Image</comment_new>


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Reset margin to 0 for .monaco-action-bar .codicon inside .progress-container to match other action bars in VS Code.

Co-authored-by: meganrogge <29464607+meganrogge@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix excessive padding on terminal tool progress part icons Fix excessive padding on terminal tool progress part action bar icons Jan 29, 2026
Copilot AI requested a review from meganrogge January 29, 2026 16:22
@meganrogge
Copy link
Collaborator

@copilot we only want to change this for the chat terminal progress part, not all monaco

Move the codicon margin reset from chat.css to chatTerminalToolProgressPart.css
to only affect the chat terminal action bar, not all monaco action bars.

Co-authored-by: meganrogge <29464607+meganrogge@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 29, 2026

@copilot we only want to change this for the chat terminal progress part, not all monaco

Moved the fix to chatTerminalToolProgressPart.css with a more specific selector targeting only .chat-terminal-content-part .chat-terminal-action-bar .monaco-action-bar .codicon. Commit 51b12ed.

@meganrogge meganrogge marked this pull request as ready for review January 29, 2026 21:17
Copilot AI review requested due to automatic review settings January 29, 2026 21:17
@meganrogge meganrogge enabled auto-merge (squash) January 29, 2026 21:17
@meganrogge meganrogge added this to the January 2026 milestone Jan 29, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes excessive horizontal spacing on action bar icons in the terminal tool progress part. The issue was caused by a broad CSS rule that applied margin: 0 .2em to all codicons inside .progress-container, including those within action bars.

Changes:

  • Added a CSS reset rule to remove the margin from codicons specifically within the terminal tool progress part's action bar, ensuring they have standard action bar spacing

@meganrogge meganrogge closed this Jan 29, 2026
auto-merge was automatically disabled January 29, 2026 21:40

Pull request was closed

@meganrogge meganrogge reopened this Jan 29, 2026
@meganrogge meganrogge enabled auto-merge (squash) January 29, 2026 21:41
@meganrogge meganrogge closed this Jan 29, 2026
auto-merge was automatically disabled January 29, 2026 21:43

Pull request was closed

@meganrogge meganrogge reopened this Jan 29, 2026
@meganrogge meganrogge enabled auto-merge (squash) January 29, 2026 21:44
@meganrogge meganrogge merged commit 07e6e58 into main Jan 29, 2026
22 checks passed
@meganrogge meganrogge deleted the copilot/fix-terminal-tool-padding branch January 29, 2026 22:23
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.

Padding on terminal tool progress part icons is too large

4 participants