Skip to content

Revert "sessions - improve session hover title rendering and persistence"#299168

Merged
bpasero merged 1 commit intomainfrom
revert-298950-ben/copilot-worktree-2026-03-03T11-10-33
Mar 4, 2026
Merged

Revert "sessions - improve session hover title rendering and persistence"#299168
bpasero merged 1 commit intomainfrom
revert-298950-ben/copilot-worktree-2026-03-03T11-10-33

Conversation

@bpasero
Copy link
Member

@bpasero bpasero commented Mar 4, 2026

Reverts #298950

Copilot AI review requested due to automatic review settings March 4, 2026 10:06
@bpasero bpasero enabled auto-merge (squash) March 4, 2026 10:06
@bpasero bpasero self-assigned this Mar 4, 2026
@bpasero bpasero added this to the 1.111.0 milestone Mar 4, 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 reverts #298950, which had introduced improvements to session hover title rendering (using renderAsPlaintext on the markdown label for a simple hover) and persistence. The revert rolls back the rename from disableHoveruseSimpleHover and removes the plaintext hover rendering that was added in that PR, restoring the original early-return behavior when hover is disabled.

Changes:

  • Renames the option useSimpleHover back to disableHover in all relevant interfaces and usages.
  • Removes the plaintext-markdown hover rendering logic that was introduced in #298950, restoring the simple early-return when hover is disabled.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
agentSessionsViewer.ts Reverts IAgentSessionRendererOptions.useSimpleHoverdisableHover; restores renderHover to return early instead of showing a plaintext hover
agentSessionsControl.ts Reverts IAgentSessionsControlOptions.useSimpleHoverdisableHover
sessionsViewPane.ts Reverts usage site: useSimpleHover: truedisableHover: true
agentSessionsViewer.fixture.ts Reverts test fixture: useSimpleHover: truedisableHover: true

@bpasero bpasero merged commit 34659c0 into main Mar 4, 2026
24 checks passed
@bpasero bpasero deleted the revert-298950-ben/copilot-worktree-2026-03-03T11-10-33 branch March 4, 2026 11:51
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