Skip to content

Conversation

@jrieken
Copy link
Member

@jrieken jrieken commented Jan 29, 2026

No description provided.

Copilot AI review requested due to automatic review settings January 29, 2026 15:54
@jrieken jrieken enabled auto-merge (squash) January 29, 2026 15:54
@jrieken jrieken self-assigned this Jan 29, 2026
@vs-code-engineering
Copy link

📬 CODENOTIFY

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

@bpasero

Matched files:

  • src/vs/workbench/browser/parts/editor/breadcrumbsControl.ts

@vs-code-engineering vs-code-engineering bot added this to the January 2026 milestone Jan 29, 2026
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 fixes the positioning of the breadcrumbs dropdown picker by correcting which DOM element is used to calculate the anchor position. Previously, the code incorrectly used event.node.firstChild to get the position, which would return the first child element inside the breadcrumb item container. The fix correctly uses event.node directly, which is the breadcrumb item container itself.

Changes:

  • Fixed breadcrumbs picker positioning by using the correct DOM element reference

@jrieken jrieken merged commit d9e4d02 into main Jan 29, 2026
27 of 28 checks passed
@jrieken jrieken deleted the joh/continuing-tahr branch January 29, 2026 16:36
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