Skip to content

Comments

Manually manage the checkboxes#8527

Merged
alexr00 merged 1 commit intomainfrom
alexr00/initial-badger
Feb 17, 2026
Merged

Manually manage the checkboxes#8527
alexr00 merged 1 commit intomainfrom
alexr00/initial-badger

Conversation

@alexr00
Copy link
Member

@alexr00 alexr00 commented Feb 17, 2026

to prevent flickering

to prevent flickering
Copilot AI review requested due to automatic review settings February 17, 2026 11:09
@alexr00 alexr00 enabled auto-merge (squash) February 17, 2026 11:09
@alexr00 alexr00 self-assigned this Feb 17, 2026
@vs-code-engineering vs-code-engineering bot added this to the February 2026 milestone Feb 17, 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 switches the PR-related tree views to manual checkbox state management to reduce checkbox flickering, and updates checkbox handling to optimistically update node state and refresh the tree.

Changes:

  • Enable manageCheckboxStateManually for the PR list and PR changes tree views.
  • Rework checkbox update processing to update node state directly, collect directory descendants, and explicitly refresh affected parts of the tree.
  • Change markFiles calls to suppress viewed-state change events and refresh context after the API call completes.

Reviewed changes

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

File Description
src/view/treeNodes/treeUtils.ts Reworks checkbox processing (descendant collection, manual refresh, and adjusted markFiles calls).
src/view/treeNodes/fileChangeNode.ts Adds a helper to refresh the “file viewed” context after async updates.
src/view/prsTreeDataProvider.ts Enables manual checkbox state management for the PR list tree view.
src/view/prChangesTreeDataProvider.ts Enables manual checkbox state management for the PR changes tree view.

@alexr00 alexr00 merged commit e810f7f into main Feb 17, 2026
12 checks passed
@alexr00 alexr00 deleted the alexr00/initial-badger branch February 17, 2026 12:30
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.

2 participants