Skip to content

Improve new session hover functionality#315222

Merged
benibenj merged 1 commit intomainfrom
benibenj/envious-gibbon
May 8, 2026
Merged

Improve new session hover functionality#315222
benibenj merged 1 commit intomainfrom
benibenj/envious-gibbon

Conversation

@benibenj
Copy link
Copy Markdown
Contributor

@benibenj benibenj commented May 8, 2026

Copilot Generated Description: Update the hover behavior for the new session button to display the keybinding label and adjust the hover position.

Copilot AI review requested due to automatic review settings May 8, 2026 09:49
@benibenj benibenj enabled auto-merge May 8, 2026 09:49
@benibenj benibenj self-assigned this May 8, 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 the tooltip/hover behavior for the “New Session” button in the Agents Sessions view so the hover is anchored to the button (not the mouse cursor) and appears below it, while showing the keybinding label when available.

Changes:

  • Switch the “New Session” hover from setupDelayedHoverAtMouse to setupDelayedHover (element-anchored hover).
  • Make the hover compact and explicitly position it below the button via HoverPosition.BELOW.
Show a summary per file
File Description
src/vs/sessions/contrib/sessions/browser/views/sessionsView.ts Adjusts the new session button hover to be compact and positioned below the button (element-anchored) while including the keybinding label.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@benibenj benibenj merged commit 760844f into main May 8, 2026
30 checks passed
@benibenj benibenj deleted the benibenj/envious-gibbon branch May 8, 2026 10:07
@vs-code-engineering vs-code-engineering Bot added this to the 1.120.0 milestone May 8, 2026
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