jpmorganchase / salt-ds

React UI components built with a focus on accessibility, customisation and ease-of-use
https://www.saltdesignsystem.com
Apache License 2.0
109 stars 78 forks source link

Stepped Tracker statuses: warning and error #3584

Open tomhazledine opened 2 weeks ago

tomhazledine commented 2 weeks ago

Adds warning and error statuses to the <TrackerStep> component's state prop.

Full list is now:

(Note that "active" is not a valid status value: the active step is defined by the activeStep prop on the SteppedTracker component.)

Screenshot 2024-06-17 at 12 39 43
vercel[bot] commented 2 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
saltdesignsystem ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 27, 2024 4:38pm
changeset-bot[bot] commented 2 weeks ago

🦋 Changeset detected

Latest commit: d2f49ef21583a23d0f9c9918cc8d70e88f6a5186

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ------------ | ----- | | @salt-ds/lab | Minor |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

ivan-calderon commented 2 weeks ago

Thank you @tomhazledine! Please let me know when you complete the updates in the typography styles we discussed today. I'll be running a formal Design review in Figma on Thursday.

tomhazledine commented 1 week ago

Thank you @tomhazledine! Please let me know when you complete the updates in the typography styles we discussed today. I'll be running a formal Design review in Figma on Thursday.

For any devs reviewing this PR: Ivan's referring to typography changes that will be made in a PR later this week. This PR is unaffected and ready-to-go.

origami-z commented 1 week ago

put back to draft, until naming convosation settles.