Skip to content

Fix unexpected layout and focus change when PR state changes#8826

Merged
alexr00 merged 1 commit into
mainfrom
alexr00/surprised-dragon
Jul 3, 2026
Merged

Fix unexpected layout and focus change when PR state changes#8826
alexr00 merged 1 commit into
mainfrom
alexr00/surprised-dragon

Conversation

@alexr00

@alexr00 alexr00 commented Jul 3, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings July 3, 2026 09:20
@alexr00 alexr00 enabled auto-merge (squash) July 3, 2026 09:20
@alexr00 alexr00 self-assigned this Jul 3, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 adjusts ReviewManager’s background polling behavior to avoid triggering UI layout changes (and related focus shifts) when the pull request state is refreshed due to polling.

Changes:

  • Update the polling refresh path to call updateState(true, false) (silent refresh, no layout update) instead of updateState().
Show a summary per file
File Description
src/view/reviewManager.ts Make polling-based state refresh silent and skip layout updates to prevent unintended UI/focus changes.

Review details

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Low

@alexr00 alexr00 merged commit 55e6a06 into main Jul 3, 2026
8 checks passed
@alexr00 alexr00 deleted the alexr00/surprised-dragon branch July 3, 2026 09:47
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