Skip to content

Conversation

@willcodeforcoffee
Copy link
Collaborator

@willcodeforcoffee willcodeforcoffee commented Dec 12, 2025

This PR adds the documentation for the new Linear Progress control.
Originally all of the documentation for the Circular Progress controls was simply labelled "Progress Indicator", which is confusing when there are two indicators now. So all "Progress Indicator" mentions were renamed to "Circular Progress Indicator" and left beside the "Linear Progress Indicator" for discoverability, and, consistency.

Note: This PR is currently not building because the main repo cannot publish to npm at the moment.
When that is resolved:

  • the @abgov packages need to be updated
  • then this can be rebuilt (successfully) and tested on Netlify.

Related story:

@willcodeforcoffee willcodeforcoffee self-assigned this Dec 12, 2025
@netlify
Copy link

netlify bot commented Dec 12, 2025

Deploy Preview for abgov-ui-component-docs ready!

Name Link
🔨 Latest commit 15a4ba9
🔍 Latest deploy log https://app.netlify.com/projects/abgov-ui-component-docs/deploys/694ae9044f4da700089e9dff
😎 Deploy Preview https://deploy-preview-467--abgov-ui-component-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 37
Accessibility: 93
Best Practices: 100
SEO: 83
PWA: -
View the detailed breakdown and full score reports

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

@willcodeforcoffee willcodeforcoffee force-pushed the eric/feat1908-linear-progress branch from 1c5f96d to d78121b Compare December 18, 2025 23:35
@willcodeforcoffee willcodeforcoffee marked this pull request as ready for review December 18, 2025 23:45
Copy link
Collaborator

@twjeffery twjeffery left a comment

Choose a reason for hiding this comment

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

Thanks for adding this @willcodeforcoffee

A few small issues I see that need to be adjusted. I'll list them below, let me know if you have any questions and/or want to discuss:

  1. temporary notification "New" badge now bleeding off right of side menu
Image
  1. linear progress indicator should have "New" badge

  2. the code snippet for the 3rd example (Automatically increasing progress) is not showing a show more button for some reason, so you can't see all the code. My only clue to why is perhaps because you are not using the sandbox in the same way as other pages?

Image

Example of show more button:
Image

Copy link
Collaborator

@twjeffery twjeffery left a comment

Choose a reason for hiding this comment

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

Thanks @willcodeforcoffee Looks good to me!

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.

Linear progress indicator documentation

3 participants