Skip to content

Update menu styles and improve sticky scroll appearance#302172

Merged
mrleemurray merged 2 commits intomainfrom
mrleemurray/leading-brown-sparrow
Mar 16, 2026
Merged

Update menu styles and improve sticky scroll appearance#302172
mrleemurray merged 2 commits intomainfrom
mrleemurray/leading-brown-sparrow

Conversation

@mrleemurray
Copy link
Copy Markdown
Contributor

Enhance the menu's visual design by updating styles and improving the appearance of the sticky scroll feature. This includes adjustments to borders, shadows, and selection colors for better consistency and usability.

Copilot AI review requested due to automatic review settings March 16, 2026 16:51
@mrleemurray mrleemurray enabled auto-merge (squash) March 16, 2026 16:51
@mrleemurray mrleemurray self-assigned this Mar 16, 2026
@vs-code-engineering vs-code-engineering bot added this to the 1.112.0 milestone Mar 16, 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 updates VS Code theming and CSS so menus use dedicated menu selection colors and sticky scroll uses a themeable shadow color, aiming for more consistent visuals across themes.

Changes:

  • Switch default menu selection colors to menu.selectionForeground / menu.selectionBackground tokens.
  • Update sticky scroll shadow to use the editorStickyScroll.shadow token (with fixed offsets).
  • Add a CSS border to .monaco-menu using the menu.border token.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/vs/platform/theme/browser/defaultStyles.ts Updates default menu selection colors to menu-specific theme tokens.
src/vs/editor/contrib/stickyScroll/browser/stickyScroll.css Replaces generic shadow variable with sticky-scroll-specific shadow token usage.
src/vs/base/browser/ui/menu/menu.ts Adds a CSS border rule to the menu widget styling.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@mrleemurray mrleemurray merged commit 6647d4f into main Mar 16, 2026
20 checks passed
@mrleemurray mrleemurray deleted the mrleemurray/leading-brown-sparrow branch March 16, 2026 17:56
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