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

fix(web-components): remove determinate from spinner implementation #31404

Conversation

chrisdholt
Copy link
Member

Previous Behavior

Spinner optionally could be set as a determinate progress indicator.

New Behavior

There is no construct for a determinate spinner, the behavior has been removed.

Related Issue(s)

  • Fixes #

@chrisdholt chrisdholt requested a review from a team as a code owner May 16, 2024 22:42
packages/web-components/src/spinner/spinner.styles.ts Outdated Show resolved Hide resolved
packages/web-components/src/spinner/spinner.template.ts Outdated Show resolved Hide resolved
packages/web-components/src/spinner/spinner.template.ts Outdated Show resolved Hide resolved
packages/web-components/src/spinner/spinner.styles.ts Outdated Show resolved Hide resolved
@chrisdholt chrisdholt force-pushed the users/chhol/remove-determinate-from-spinner branch from 1a9d708 to feee0cb Compare May 16, 2024 22:48
@fabricteam
Copy link
Collaborator

fabricteam commented May 16, 2024

📊 Bundle size report

✅ No changes found

@chrisdholt chrisdholt force-pushed the users/chhol/remove-determinate-from-spinner branch from feee0cb to 97a082f Compare May 16, 2024 22:49
@chrisdholt chrisdholt force-pushed the users/chhol/remove-determinate-from-spinner branch from 4e13b29 to ddd9ceb Compare May 16, 2024 22:58
@chrisdholt chrisdholt force-pushed the users/chhol/remove-determinate-from-spinner branch 2 times, most recently from 8e36b3d to e243e89 Compare May 16, 2024 23:06
@chrisdholt chrisdholt force-pushed the users/chhol/remove-determinate-from-spinner branch from e243e89 to 923fe8e Compare May 16, 2024 23:06
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@chrisdholt chrisdholt merged commit 74312a0 into microsoft:master May 17, 2024
19 checks passed
@chrisdholt chrisdholt deleted the users/chhol/remove-determinate-from-spinner branch May 17, 2024 01:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants