Skip to content

Comments

Update viewpane filter container to use :has selector for focus state#297383

Merged
mrleemurray merged 2 commits intomainfrom
mrleemurray/sure-maroon-vole
Feb 24, 2026
Merged

Update viewpane filter container to use :has selector for focus state#297383
mrleemurray merged 2 commits intomainfrom
mrleemurray/sure-maroon-vole

Conversation

@mrleemurray
Copy link
Contributor

Refactor the viewpane filter container to utilize the :has selector for improved focus state handling. This change enhances the user experience by ensuring the focus state is applied correctly when an input box within the container is focused. Testing can be done by interacting with the viewpane filter and observing the focus behavior.

Addresses: #277988

Copilot AI review requested due to automatic review settings February 24, 2026 16:10
@mrleemurray mrleemurray self-assigned this Feb 24, 2026
@mrleemurray mrleemurray added this to the February 2026 milestone Feb 24, 2026
@vs-code-engineering
Copy link

vs-code-engineering bot commented Feb 24, 2026

📬 CODENOTIFY

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

@bpasero

Matched files:

  • src/vs/workbench/browser/parts/views/media/views.css

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

Refactors the panel header viewpane filter container’s “expanded” focus styling to use a :has(...) selector, aiming to prevent the filter input from jumping when clicked (issue #277988).

Changes:

  • Replace :active/:focus-within-based expansion with :has(.monaco-inputbox .input:focus) for the viewpane filter container in panel titles.

@mrleemurray mrleemurray merged commit 67c59a1 into main Feb 24, 2026
20 checks passed
@mrleemurray mrleemurray deleted the mrleemurray/sure-maroon-vole branch February 24, 2026 16: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.

2 participants