-
Notifications
You must be signed in to change notification settings - Fork 115
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
chore: update status pattern #4110
base: main
Are you sure you want to change the base?
Conversation
Run & review this pull request in StackBlitz Codeflow. |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 0643cb2. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
Size Change: 0 B Total Size: 1.1 MB ℹ️ View Unchanged
|
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. Latest deployment of this branch, based on commit 0643cb2:
|
Paste Run #8886
Run Properties:
|
Project |
Paste
|
Run status |
Passed #8886
|
Run duration | 02m 29s |
Commit |
0643cb2525: chore: feedback from pr
|
Committer | “nora |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
67
|
394d15e
to
0643cb2
Compare
@@ -154,7 +182,7 @@ Non-text status indicators (such as icons or status dots) should be paired with | |||
possible and should always be given a `title` to give context and meaning to the icon or shape. | |||
Check out the [icon usage guidelines](/components/icon/usage-guidelines) for more detailed accessibility information. | |||
|
|||
## Variations | |||
## Status types | |||
|
|||
### Process status | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update the text here to just be:
Process statuses indicate an object’s progress through a set of stages. Process statuses should always be paired with text.
- Otherwise, use the default [Status Badge](/components/status-badge). | ||
Placement of the status in a Table should depend on the importance of the status to the given user flow and should be tested with users. | ||
For statuses that are not as critical to a user’s workflow, use [plain text statuses](#plain-text-status). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Otherwise, use the default [Status Badge](/components/status-badge). | |
Placement of the status in a Table should depend on the importance of the status to the given user flow and should be tested with users. | |
For statuses that are not as critical to a user’s workflow, use [plain text statuses](#plain-text-status). | |
- Otherwise, use the default [Status Badge](/components/status-badge). | |
Placement of the status in a Table should depend on the importance of the status to the given user flow and should be tested with users. | |
For statuses that are not as critical to a user’s workflow, use [plain text statuses](#plain-text-status). |
Updates doc: https://docs.google.com/document/d/1kDSWZrUBIR0ZrliM80_IjKCM8rMIDiISuxHTlQ-1xvs/edit?tab=t.0