Skip to content

feat: implement comprehensive activityBarTop theming and refine icon rendering#293557

Open
kairosci wants to merge 8 commits intomicrosoft:mainfrom
kairosci:feat/activity-bar-hover-colors
Open

feat: implement comprehensive activityBarTop theming and refine icon rendering#293557
kairosci wants to merge 8 commits intomicrosoft:mainfrom
kairosci:feat/activity-bar-hover-colors

Conversation

@kairosci
Copy link

@kairosci kairosci commented Feb 6, 2026

Closes: #210694

Copilot AI review requested due to automatic review settings February 6, 2026 23:48
@vs-code-engineering
Copy link

vs-code-engineering bot commented Feb 6, 2026

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

@bpasero

Matched files:

  • src/vs/workbench/browser/parts/auxiliarybar/auxiliaryBarPart.ts
  • src/vs/workbench/browser/parts/paneCompositePart.ts
  • src/vs/workbench/browser/parts/sidebar/sidebarPart.ts
  • src/vs/workbench/common/theme.ts

Copy link
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 adds new theme color keys to let themes customize activity bar item hover foreground/background colors, addressing readability issues when hovered items use the same foreground as the bar background.

Changes:

  • Register activityBar.hoverBackground / activityBar.hoverForeground theme colors.
  • Register activityBarTop.hoverBackground / activityBarTop.hoverForeground theme colors.
  • Add theming-participant CSS rules intended to apply the new hover colors to the (left/right) activity bar.

Reviewed changes

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

File Description
src/vs/workbench/common/theme.ts Adds new registered theme colors for activity bar and activity bar top hover states.
src/vs/workbench/browser/parts/activitybar/activitybarPart.ts Attempts to apply the new activity bar hover colors via dynamically injected CSS rules.

@kairosci kairosci force-pushed the feat/activity-bar-hover-colors branch from c69e59e to 1d1a0a9 Compare February 7, 2026 14:27
@kairosci
Copy link
Author

kairosci commented Feb 7, 2026

Given:

{
    "workbench.colorCustomizations": {
    "activityBar.hoverBackground": "#ff0000",      
    "activityBar.hoverForeground": "#00ff00",      
    }
}

I see:

t

this applies to all icons .

@kairosci kairosci marked this pull request as draft February 7, 2026 16:14
@kairosci kairosci force-pushed the feat/activity-bar-hover-colors branch from c294bdc to 2542be3 Compare February 7, 2026 16:17
@kairosci kairosci changed the title feat: add hover colors for activity bar feat: implement comprehensive activityBarTop theming and refine icon rendering Feb 7, 2026
@kairosci kairosci force-pushed the feat/activity-bar-hover-colors branch from 9ad7f67 to d57d671 Compare February 7, 2026 16:34
@kairosci kairosci marked this pull request as ready for review February 7, 2026 17:10
@kairosci kairosci marked this pull request as draft February 7, 2026 17:10
@kairosci kairosci marked this pull request as ready for review February 7, 2026 17:10
@kairosci
Copy link
Author

kairosci commented Feb 7, 2026

Given:

{
    "workbench.colorCustomizations": {
    "activityBar{Top/Bottom}.hoverBackground": "#ff0000",      
    "activityBar{Top/Bottom}.hoverForeground": "#00ff00",      
    }
}

I see:

t

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.

add hoverForeground and hoverBackground to activity bars

2 participants