Skip to content

Prevent clipped suggest widget list items#327378

Open
hawkticehurst with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-issue-327368
Open

Prevent clipped suggest widget list items#327378
hawkticehurst with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-issue-327368

Conversation

Copilot AI commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Modern UI clipped focused rows at the suggest widget’s rounded edges.

  • Styling
    • Match suggest rows to the overlay radius.
    • Scope the override to Modern UI suggest widgets, leaving other lists unchanged.
ui-diff-20260724-163258

Co-authored-by: hawkticehurst <39639992+hawkticehurst@users.noreply.github.com>

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

Prevents Modern UI suggest-widget focus rows from appearing clipped by rounded overlay edges.

Changes:

  • Adds a Modern UI-scoped radius override for suggest rows.
  • Excludes separator rows.
Comments suppressed due to low confidence (1)

src/vs/workbench/contrib/styleOverrides/browser/media/roundedCorners.css:224

  • This introduces another !important override. It is needed only because the broad list-row declaration at lines 62-64 is itself important; please narrow or restructure that rule so the suggest-specific declaration wins by selector ownership/specificity instead of adding a second important declaration.
	border-radius: var(--vscode-cornerRadius-large) !important;

Comment on lines +223 to +224
.style-override .editor-widget.suggest-widget .monaco-list .monaco-list-row:not(.separator) {
border-radius: var(--vscode-cornerRadius-large) !important;

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed in the latest commit. The rule now only applies cornerRadius-large to the first and last rows (boundary rows), following the notificationsCenter.css pattern. Interior rows keep the Control radius (cornerRadius-small) from the generic list-row rule.

Co-authored-by: hawkticehurst <39639992+hawkticehurst@users.noreply.github.com>
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