Skip to content

Conversation

@connor4312
Copy link
Member

Adds support for custom hover delays through a new 'delay' property in IHoverLifecycleOptions. This allows the agent sessions component to use a shorter delay when showing hover information for sessions requiring input, improving the user experience by providing faster feedback.

Changes:

  • Adds optional 'delay' parameter to IHoverLifecycleOptions interface
  • Updates HoverService to respect custom delay values
  • Applies shorter delay in agent sessions viewer for input-requiring sessions
  • Adds comprehensive test coverage for the new delay functionality

(Commit message generated by Copilot)

…g input

Adds support for custom hover delays through a new 'delay' property in
IHoverLifecycleOptions. This allows the agent sessions component to use a
shorter delay when showing hover information for sessions requiring input,
improving the user experience by providing faster feedback.

Changes:
- Adds optional 'delay' parameter to IHoverLifecycleOptions interface
- Updates HoverService to respect custom delay values
- Applies shorter delay in agent sessions viewer for input-requiring sessions
- Adds comprehensive test coverage for the new delay functionality

(Commit message generated by Copilot)
Copilot AI review requested due to automatic review settings January 26, 2026 20:59
@connor4312 connor4312 enabled auto-merge (squash) January 26, 2026 20:59
@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/contrib/chat/browser/agentSessions/agentSessionsViewer.ts

Copy link
Member

@Tyriar Tyriar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When this has come up in the past we have avoided adding this to keep things consistent. Why is it too slow now? If anything it feels too fast on Windows as the popup is huge.

This comment was marked as resolved.

@connor4312
Copy link
Member Author

connor4312 commented Jan 26, 2026

@Tyriar this is only faster for sessions that are pending input where we know the intention of the user is generally to interact with the session. This was a feedback item that came up independently from both Kai and JC 🙂 (who are both macOS users where the default delay is 1.5s to match the OS behavior)

@connor4312 connor4312 requested a review from Tyriar January 26, 2026 23:37
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.

4 participants