Skip to content

docs(progress-indicator): add progress-indicator - #33

Merged
danielleroux merged 25 commits into
release/v3.2.0from
docs/progress-indicator
Jul 15, 2025
Merged

docs(progress-indicator): add progress-indicator#33
danielleroux merged 25 commits into
release/v3.2.0from
docs/progress-indicator

Conversation

@danielleroux

Copy link
Copy Markdown
Collaborator

🆕 What is the new behavior?

👨‍💻 Help & support

@netlify

netlify Bot commented Jun 16, 2025

Copy link
Copy Markdown

Deploy Preview for industrial-experience ready!

Name Link
🔨 Latest commit 76e50cd
🔍 Latest deploy log https://app.netlify.com/projects/industrial-experience/deploys/687657f33ca73c00080cb6cc
😎 Deploy Preview https://deploy-preview-33--industrial-experience.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

nuke-ellington
nuke-ellington previously approved these changes Jun 18, 2025
@danielleroux
danielleroux changed the base branch from main to release/v3.2.0 June 24, 2025 07:26
@danielleroux
danielleroux marked this pull request as draft June 24, 2025 12:19
@kathrinschalber
kathrinschalber requested a review from flxlst09 June 25, 2025 12:45
@danielleroux
danielleroux changed the base branch from release/v3.2.0 to main June 26, 2025 10:00

@flxlst09 flxlst09 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good to read and easy to understand, thanks! Comments mainly refer to mismatch with the current docu illustrations. The mentioning of the spinner for indeterminate processes is quite late, would be of benefit for users to know about this earlier.

Comment thread docs/components/progress-indicator/guide.md
Comment thread docs/components/progress-indicator/guide.md
Comment thread docs/components/progress-indicator/guide.md Outdated
Comment thread docs/components/progress-indicator/guide.md Outdated
Comment thread docs/components/progress-indicator/guide.md
Comment thread docs/components/progress-indicator/guide.md
@danielleroux
danielleroux changed the base branch from main to release/v3.2.0 June 27, 2025 09:53
Comment thread docs/components/overview.md Outdated
Comment thread docs/components/progress-indicator/guide.md Outdated
Comment thread docs/components/progress-indicator/guide.md Outdated
Comment thread docs/components/progress-indicator/guide.md Outdated
Comment thread docs/components/progress-indicator/guide.md Outdated
Comment thread docs/components/progress-indicator/guide.md Outdated
Comment thread docs/components/progress-indicator/guide.md Outdated
Comment thread docs/components/progress-indicator/guide.md Outdated
Comment thread docs/components/progress-indicator/guide.md Outdated
Comment thread docs/components/progress-indicator/guide.md Outdated
Comment thread docs/components/progress-indicator/guide.md Outdated
Comment thread docs/components/progress-indicator/index.mdx Outdated
Comment thread docs/components/progress-indicator/guide.md Outdated

@tokyojen tokyojen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

OK done - check line 65 as I changed it a lot!

kathrinschalber and others added 3 commits July 2, 2025 09:24
Co-authored-by: tokyojen <143795032+tokyojen@users.noreply.github.com>
Co-authored-by: tokyojen <143795032+tokyojen@users.noreply.github.com>
@kathrinschalber
kathrinschalber marked this pull request as ready for review July 2, 2025 07:29
@kathrinschalber

Copy link
Copy Markdown
Collaborator
  • UX review
  • Dev review
  • Final edit

@danielleroux Figma images have changed too, so please fetch them again

@danielleroux
danielleroux merged commit 7aab802 into release/v3.2.0 Jul 15, 2025
5 checks passed
@danielleroux
danielleroux deleted the docs/progress-indicator branch July 15, 2025 14:57
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.

5 participants