Skip to content

fix(controls): Fix ProgressRing not restarting properly when visibily changes#1678

Open
Nuklon wants to merge 2 commits intolepoco:mainfrom
Nuklon:patch-42
Open

fix(controls): Fix ProgressRing not restarting properly when visibily changes#1678
Nuklon wants to merge 2 commits intolepoco:mainfrom
Nuklon:patch-42

Conversation

@Nuklon
Copy link
Copy Markdown
Collaborator

@Nuklon Nuklon commented Mar 24, 2026

Pull request type

Please check the type of change your PR introduces:

  • Update
  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes

What is the current behavior?

If you hide an indeterminate progress ring (e.g., after loading is done), and then show it again, the progress ring doesn't restart the animation properly, rendering a partial progress ring (+ looks like broken animation) due to the angle not getting reset.

What is the new behavior?

Restart the animation after visibily changed (reuses the IsEnabled path).

@Nuklon Nuklon requested a review from pomianowski as a code owner March 24, 2026 12:31
@github-actions github-actions bot added controls Changes to the appearance or logic of custom controls. styles Topic is related to styles PR Pull request release labels Mar 24, 2026
@Nuklon Nuklon marked this pull request as draft March 24, 2026 12:47
@Nuklon Nuklon marked this pull request as ready for review March 24, 2026 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

controls Changes to the appearance or logic of custom controls. PR Pull request release styles Topic is related to styles

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant