Skip to content

Add refresh button to environment picker - #2016

Merged
Stella Huang (StellaHuang95) merged 2 commits into
microsoft:mainfrom
StellaHuang95:stellahuang-microsoft-issue-630-button-to-refresh-select-environment-dbe675
Nov 13, 2026
Merged

Stella Huang (StellaHuang95) merged 2 commits into
microsoft:mainfrom
StellaHuang95:stellahuang-microsoft-issue-630-button-to-refresh-select-environment-dbe675

Conversation

@StellaHuang95

Copy link
Copy Markdown
Contributor

Summary

  • add a localized refresh button to the environment picker
  • refresh managers without closing the picker and coalesce repeated clicks
  • preserve picker state and scroll position while handling cancellation and failures safely

Validation

  • npm run lint
  • npm run compile-tests
  • npm run unittest (3,536 passing, 9 pending)
  • npm run compile
  • manually verified refresh visibility, in-place loading, restored input, preserved scroll position, and subsequent environment selection

Fixes #630

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@StellaHuang95 Stella Huang (StellaHuang95) added the feature-request Request for new features or functionality label Nov 12, 2026
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@rchiodo

Rich Chiodo (rchiodo) commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

🔒 Automated review in progress — Rich Chiodo (@rchiodo) is auto-reviewing this PR.

@rchiodo

Copy link
Copy Markdown
Contributor

Reviewed commit: f32c3e67ffd3aeb1e7122c0a37e25a6f32f1b4e4

Result: 🔴 could-not-verify

Verification details

Verification: Automated verification is best effort. The PR author remains responsible for testing.

Summary: [unavailable] Container verification could not start and local execution was not authorized for this PR HEAD: Podman machine &#x27;pyrx-automation&#x27; is not running.

Test runs: none recorded.

@rchiodo Rich Chiodo (rchiodo) 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.

Approved via Review Center.

@rchiodo Rich Chiodo (rchiodo) added the review-auto:approved Automated review: no blocking findings (approval posted). label Nov 12, 2026
@StellaHuang95
Stella Huang (StellaHuang95) merged commit b3e5455 into microsoft:main Nov 13, 2026
60 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature-request Request for new features or functionality review-auto:approved Automated review: no blocking findings (approval posted).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Button to refresh select environment

2 participants