Adopt hover for session type picker and mode picker#289242
Merged
Conversation
Contributor
📬 CODENOTIFYThe following users are being notified based on files changed in this PR: @bpaseroMatched files:
|
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request implements hover functionality for chat input pickers, replacing traditional tooltips with rich hover descriptions. The changes improve user experience by providing more detailed contextual information when users interact with session type and mode pickers.
Changes:
- Adopted hover service for actionList items, replacing instant hover with delayed hover for better UX
- Added
getAgentSessionProviderDescriptionhelper function to centralize session provider descriptions - Updated session type picker and mode picker to use hover instead of tooltip properties
- Improved hover lifecycle management in ActionList using MutableDisposable
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| sessionTargetPickerActionItem.ts | Updated to use hoverDescription property and pass hover content via hover object instead of tooltip |
| modePickerActionItem.ts | Changed from tooltip to hover for mode picker actions to provide richer contextual information |
| delegationSessionPickerActionItem.ts | Set tooltip to empty string for "New Chat Session" action (incomplete - missing hover property) |
| agentSessions.ts | Added getAgentSessionProviderDescription function to provide localized hover descriptions for all session providers |
| actionList.ts | Improved hover management by using MutableDisposable and switched from instant to delayed hover for better UX |
justschen
approved these changes
Jan 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Copilot Generated Description:Implement hover functionality for the session type picker and mode picker, providing additional context through hover descriptions. Update related components to utilize the new hover feature. Adjust tooltip handling to accommodate the changes.closes #289041