Skip to content

preserve agent session user filter when clicking notification filter#289506

Merged
joshspicer merged 2 commits intomainfrom
joshspicer/filter
Jan 21, 2026
Merged

preserve agent session user filter when clicking notification filter#289506
joshspicer merged 2 commits intomainfrom
joshspicer/filter

Conversation

@joshspicer
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings January 21, 2026 22:39
@vs-code-engineering
Copy link
Copy Markdown
Contributor

vs-code-engineering bot commented Jan 21, 2026

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

@bpasero

Matched files:

  • src/vs/workbench/contrib/chat/browser/agentSessions/experiments/agentTitleBarStatusWidget.ts

mjbvz
mjbvz previously approved these changes Jan 21, 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 pull request attempts to preserve the user's filter state when toggling notification filters (unread/in-progress badges) in the agent sessions view. Before this change, clicking a notification filter would simply clear any existing filter. With this change, the code tries to save the user's filter before applying a badge filter and restore it later.

Changes:

  • Added storage key PREVIOUS_FILTER_STORAGE_KEY to save user's filter before applying badge filters
  • Implemented _saveUserFilter() method to save current filter to storage
  • Implemented _restoreUserFilter() method to restore previously saved filter
  • Updated _openSessionsWithFilter() to save/restore filters when toggling badge filters instead of clearing
  • Updated _clearFilterIfCategoryEmpty() to restore user's filter instead of clearing when a filtered category becomes empty

@joshspicer joshspicer merged commit f03a52a into main Jan 21, 2026
22 checks passed
@joshspicer joshspicer deleted the joshspicer/filter branch January 21, 2026 23:15
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Mar 7, 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.

4 participants