Skip to content

Adjust padding for progress row in chat widget - #331982

Merged
Lee Murray (mrleemurray) merged 2 commits into
mainfrom
mrleemurray/accused-aquamarine-boar
Aug 23, 2026
Merged

Adjust padding for progress row in chat widget#331982
Lee Murray (mrleemurray) merged 2 commits into
mainfrom
mrleemurray/accused-aquamarine-boar

Conversation

@mrleemurray

Copy link
Copy Markdown
Contributor

Modify the padding for the progress row to ensure consistent scroll anchoring in the chat widget. This change enhances the user experience by maintaining the position of the scroll when new messages are loaded.

Fixes: #331791 (comment)

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

Restores consistent chat scroll anchoring by preserving the progress row’s 37px structural padding.

Changes:

  • Adds the 1px stroke token to the 36px spacing token.

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nice improvement. Clean separation of concerns.

@mrleemurray
Lee Murray (mrleemurray) merged commit cac1904 into main Aug 23, 2026
27 checks passed
@mrleemurray
Lee Murray (mrleemurray) deleted the mrleemurray/accused-aquamarine-boar branch August 23, 2026 06:45
@vs-code-engineering vs-code-engineering Bot added this to the 1.135.0 milestone Aug 23, 2026
@wenma531

Copy link
Copy Markdown

1px stroke token to the 36px spacing

@wenma531

Copy link
Copy Markdown

padding for the progress row to ensure consistent scroll anchoring in the chat widget.

@wenma531

Copy link
Copy Markdown

the progress row to ensure

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.

6 participants