Skip to content

fix jump in terminal headers#298473

Merged
justschen merged 1 commit intomainfrom
justin/zoroark
Feb 28, 2026
Merged

fix jump in terminal headers#298473
justschen merged 1 commit intomainfrom
justin/zoroark

Conversation

@justschen
Copy link
Copy Markdown
Collaborator

fix #297989

Copilot AI review requested due to automatic review settings February 28, 2026 18:19
@justschen justschen enabled auto-merge (squash) February 28, 2026 18:19
Copy link
Copy Markdown
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

Fixes a UI layout “jump” in chat terminal invocation headers by adjusting styling in the chat thinking content stylesheet (ref: #297989).

Changes:

  • Aligns terminal collapsible header button content using flex layout to prevent height/position shifts.
  • Refactors the .chat-thinking-tool-wrapper / .chat-used-context styling block (structure + additional flex alignment).
Comments suppressed due to low confidence (1)

src/vs/workbench/contrib/chat/browser/widget/chatContentParts/media/chatThinkingContent.css:129

  • This nested selector will likely compile to .monaco-button.monaco-text-button .monaco-button:hover .chat-collapsible-hover-chevron, which won’t match the hover chevron inside the same button. Consider changing it to &:hover .chat-collapsible-hover-chevron (and/or moving the rule outside) so the chevron opacity updates on hover as intended.
					.monaco-button:hover .chat-collapsible-hover-chevron {
						opacity: 1;
					}

@justschen justschen merged commit 39b8df7 into main Feb 28, 2026
24 checks passed
@justschen justschen deleted the justin/zoroark branch February 28, 2026 18:36
DonJayamanne pushed a commit that referenced this pull request Mar 2, 2026
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.

Chat output jumps for terminal invocation

3 participants