Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Adding ProgressRing extension #1002

Merged
merged 7 commits into from
Feb 2, 2024
Merged

Conversation

nickrandolph
Copy link
Contributor

Fixes: https://github.com/unoplatform/uno.chefs/issues/835

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature

What is the current behavior?

ProgressRing contained within the loading template doesn't ever stop (ie IsActive is true)

What is the new behavior?

New ProgressRingExtensions.IsActive can be used to start/stop all nested ProgressRings
In VisualState for Loading/Loaded any ProgressRings can have IsActive set to true/false

PR Checklist

Please check if your PR fulfills the following requirements:

Other information

Internal Issue (If applicable):

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://polite-field-01aa64f0f-1002.eastus2.azurestaticapps.net

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://polite-field-01aa64f0f-1002.eastus2.azurestaticapps.net

Copy link
Contributor

@Xiaoy312 Xiaoy312 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(comment required)

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://polite-field-01aa64f0f-1002.eastus2.azurestaticapps.net

@lukeblevins lukeblevins self-assigned this Feb 2, 2024
Copy link

github-actions bot commented Feb 2, 2024

Azure Static Web Apps: Your stage site is ready! Visit it here: https://polite-field-01aa64f0f-1002.eastus2.azurestaticapps.net

Copy link

github-actions bot commented Feb 2, 2024

Azure Static Web Apps: Your stage site is ready! Visit it here: https://polite-field-01aa64f0f-1002.eastus2.azurestaticapps.net

Copy link

github-actions bot commented Feb 2, 2024

Azure Static Web Apps: Your stage site is ready! Visit it here: https://polite-field-01aa64f0f-1002.eastus2.azurestaticapps.net

Copy link

github-actions bot commented Feb 2, 2024

Azure Static Web Apps: Your stage site is ready! Visit it here: https://polite-field-01aa64f0f-1002.eastus2.azurestaticapps.net

@lukeblevins lukeblevins self-requested a review February 2, 2024 20:29
@nickrandolph nickrandolph merged commit c36764b into main Feb 2, 2024
26 checks passed
@nickrandolph nickrandolph deleted the dev/nr/stopprogressring branch February 2, 2024 22:53
@kazo0
Copy link
Contributor

kazo0 commented Feb 2, 2024

@nickrandolph should this be backported?

@nickrandolph
Copy link
Contributor Author

It can be, but it's new so can go into 5.2 unless needed prior

@nickrandolph
Copy link
Contributor Author

@kazo0 I think given the performance implication this has (ie progress ring continues to run after splash screen), I'll back port it.

@nickrandolph
Copy link
Contributor Author

@mergify backport release/stable/5.1

Copy link
Contributor

mergify bot commented Feb 5, 2024

backport release/stable/5.1

✅ Backports have been created

nickrandolph added a commit that referenced this pull request Feb 5, 2024
….1/pr-1002

feat: Adding ProgressRing extension (backport #1002)
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.

None yet

4 participants