Skip to content

better styling for rich elements hover#296168

Merged
justschen merged 2 commits intomainfrom
justin/sawk
Feb 19, 2026
Merged

better styling for rich elements hover#296168
justschen merged 2 commits intomainfrom
justin/sawk

Conversation

@justschen
Copy link
Copy Markdown
Collaborator

@justschen justschen commented Feb 19, 2026

makes it left right scrollable, better show more... button. also re-orders a little bit and makes it bigger

Screen.Recording.2026-02-18.at.5.46.15.PM.mov

Copilot AI review requested due to automatic review settings February 19, 2026 01:22
@justschen justschen enabled auto-merge (squash) February 19, 2026 01:23
@vs-code-engineering vs-code-engineering Bot added this to the February 2026 milestone Feb 19, 2026
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

This PR improves the styling and user experience of rich element hovers in the chat interface by replacing native CSS scrolling with VS Code's styled scrollbars and refining the visual presentation.

Changes:

  • Replaced native CSS horizontal scrolling with DomScrollableElement for element and path code sections to provide consistent VS Code-styled scrollbars
  • Updated CSS to hide native overflow and remove default underline from "Show More" button (underline still appears on hover)
  • Enhanced "Show More" button text with ellipsis for better UX indication

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/vs/workbench/contrib/chat/browser/widget/media/chat.css Updated overflow and text-decoration styles for hover elements to support DomScrollableElement implementation
src/vs/workbench/contrib/chat/browser/attachments/chatAttachmentWidgets.ts Wrapped element and path code blocks in DomScrollableElement, added scanDomNode initialization, and updated button text

@justschen justschen merged commit 243a432 into main Feb 19, 2026
18 checks passed
@justschen justschen deleted the justin/sawk branch February 19, 2026 02:37
@vs-code-engineering vs-code-engineering Bot locked and limited conversation to collaborators Apr 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants