Skip to content

fix scrollbar inconsistencies in chat input - #329740

Open
Justin Chen (justschen) wants to merge 1 commit into
mainfrom
justin/zamazenta
Open

fix scrollbar inconsistencies in chat input#329740
Justin Chen (justschen) wants to merge 1 commit into
mainfrom
justin/zamazenta

Conversation

@justschen

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI balanced review requested due to automatic review settings August 8, 2026 03:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Aligns scrollbar sizing and clipping across chat input variants.

Changes:

  • Standardizes editor scrollbar options.
  • Adjusts input spacing and working-state clipping.
  • Documents composer clipping behavior.
Show a summary per file
File Description
chat.css Extends working-state clipping.
LAYOUT.md Documents composer clipping.
newChatInput.ts Configures editor scrollbars.
newChatInSession.css Resets session composer margins.
chatInput.css Moves horizontal inset to margins.

Review details

Tip

Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 5/5 changed files
  • Comments generated: 2
  • Review effort level: Balanced

Comment thread src/vs/sessions/contrib/chat/browser/newChatInput.ts
Comment thread src/vs/workbench/contrib/chat/browser/widget/media/chat.css
@justschen
Justin Chen (justschen) marked this pull request as ready for review August 8, 2026 03:41
@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Screenshot Changes

Base: a2aaf376 Current: df3dfdfb

Changed (2)

sessions/chat/newWidget/newChatWidget/NewSessionComments/Light
Before After
before after
sessions/chat/newWidget/newChatWidget/NewSessionTip/Light
Before After
before after

2 insignificant change(s) omitted (≤20 px, Δ≤2). See CI logs for details.

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.

3 participants