Skip to content

Fix session filter message layout causing tree overflow - #298987

Closed
Benjamin Pasero (bpasero) with Copilot wants to merge 8 commits into
mainfrom
copilot/sub-pr-298979
Closed

Fix session filter message layout causing tree overflow#298987
Benjamin Pasero (bpasero) with Copilot wants to merge 8 commits into
mainfrom
copilot/sub-pr-298979

Conversation

Copilot AI commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

.agent-sessions-empty-filter-message was appended as a flow sibling of the tree inside .agent-sessions-viewer, so when visible it consumed vertical space while layout() still passed the full height to the tree — causing overflow/clamp.

Changes

  • agentsessionsviewer.css: Add position: relative to .agent-sessions-viewer; make .agent-sessions-empty-filter-message position: absolute (top/left/right: 0) so it renders as an overlay without affecting flow layout. The message is only shown when the tree is empty, so overlaying the tree area is correct.

The layout() call in agentSessionsControl.ts already passes the full height to the tree and requires no changes.


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…tsessionsviewer.css

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…onsControl.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: bpasero <900690+bpasero@users.noreply.github.com>
Copilot AI changed the title [WIP] Update session filtering visibility based on PR feedback Fix session filter message layout causing tree overflow Mar 3, 2026
Base automatically changed from ben/modest-swordtail to main March 3, 2026 17:27
@bpasero
Benjamin Pasero (bpasero) deleted the copilot/sub-pr-298979 branch March 26, 2026 15:19
@vs-code-engineering vs-code-engineering Bot locked and limited conversation to collaborators Apr 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants