-
Notifications
You must be signed in to change notification settings - Fork 85
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
[WNMGDS-2864] Add utility actions pattern guidance page #3198
Merged
jack-ryan-nava-pbc
merged 30 commits into
main
from
jryan/wnmgds-2864-utility-actions-guidance
Sep 10, 2024
Merged
[WNMGDS-2864] Add utility actions pattern guidance page #3198
jack-ryan-nava-pbc
merged 30 commits into
main
from
jryan/wnmgds-2864-utility-actions-guidance
Sep 10, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pwolfert
reviewed
Aug 12, 2024
This reverts commit fd0f3bc.
* Add PrintIcon component. * Removed fillColor attribute from PrintIcon componet and removed lint from incons.stories.tsx
* Adds Email and RSS icons. * Adds Link and What's New Icons * Exchange (') for curly (’)
zarahzachz
reviewed
Aug 19, 2024
kim-cmsds
reviewed
Aug 19, 2024
kim-cmsds
reviewed
Aug 19, 2024
kim-cmsds
reviewed
Aug 19, 2024
kim-cmsds
reviewed
Aug 19, 2024
* Run prettier on json files * Update lint-staged config to format json files
* Export vars and function needed in release script * Reorder imports and add new reviewers * Add new utility functions used by bumpVersions * Handle staging files properly * Export correct function from versions * export readJson so maybe we capture new line with read file' ' * big refactor add comments break up inline for loop and associated functions * Wrap removal of top level package json in try catch and uncomment woopsie * Add maybe more helpful comment * Further refactoring * Looks like a big refactor but basically moved all code to a new file * Adding functionality so we don't lose newline at end of json files * [NO-TICKET] Automatically format json files (#3206) * Run prettier on json files * Update lint-staged config to format json files * Undo new line addition when writing json * Fix typo * Move version number into our main function to make usage clearer --------- Co-authored-by: Patrick Wolfert <patrick.wolfert@adhocteam.us>
* Publish - @cmsgov/design-system@11.0.0-beta.0 - @cmsgov/ds-cms-gov@11.0.0-beta.0 - @cmsgov/ds-healthcare-gov@15.0.0-beta.0 - @cmsgov/ds-medicare-gov@13.0.0-beta.0 * [NO-TICKET] Manually updates "@cmsgov/design-system" version in package dependencies (#3201) manually updates dependency of "@cmsgov/design-system" version * Reverts design-system-tokens version back to 0.0.1 --------- Co-authored-by: Kim Niedermaier <kimniedermaier+cmsds@navapbc.com> Co-authored-by: Patrick Wolfert <patrick.wolfert@adhocteam.us>
* Basic migration script for v11 variable/class changes * These two variables weren't removed! We need to update our release notes, because that was a mistake * Fix syntax error (stray single quote) * Fix regex group not getting printed in the replacement text * Mention the migration script in the blog post and explain how to use it
…gh `HelpDrawer` (#3210) Fix `onAnalyticsEvent` prop bleeding into `dialog` DOM elements through `HelpDrawer` Without this fix, the unit test I added will print errors to the console like this: ``` console.error Warning: Unknown event handler property `onAnalyticsEvent`. It will be ignored. at dialog at NativeDialog (/Users/patrickwolfert/Projects/design-system/packages/design-system/src/components/NativeDialog/NativeDialog.tsx:38:3) at Drawer (/Users/patrickwolfert/Projects/design-system/packages/design-system/src/components/Drawer/Drawer.tsx:73:5) at HelpDrawer (/Users/patrickwolfert/Projects/design-system/packages/design-system/src/components/HelpDrawer/HelpDrawer.tsx:13:11) ```
Be more informative about what `defaultMenuLinks` returns in the types Previously the return type was `{}`, which is useless to teams trying to use that functions themselves
update v11 blog post to remove reference to removing the privacy settings dialog
* filter out the core design system * Add comment and make conditional more readable * make conditional more readable
* Initial set of default values added to props tables * Updated storybook snaps
* Update all themes to make inline tool tips black * Udpate playwright vrt snapshots * Updating css snapshots * Updated css themes and custom tooltip scss styling to maybe no avail * setting tooltip color to match base color of surrounding text * Restore weirdly changed snapshots curious if CI browser tests pass * [WNMGDS-2875] Fix Release script take 2 (#3205) * Export vars and function needed in release script * Reorder imports and add new reviewers * Add new utility functions used by bumpVersions * Handle staging files properly * Export correct function from versions * export readJson so maybe we capture new line with read file' ' * big refactor add comments break up inline for loop and associated functions * Wrap removal of top level package json in try catch and uncomment woopsie * Add maybe more helpful comment * Further refactoring * Looks like a big refactor but basically moved all code to a new file * Adding functionality so we don't lose newline at end of json files * [NO-TICKET] Automatically format json files (#3206) * Run prettier on json files * Update lint-staged config to format json files * Undo new line addition when writing json * Fix typo * Move version number into our main function to make usage clearer --------- Co-authored-by: Patrick Wolfert <patrick.wolfert@adhocteam.us> * Undoing accidental hover styling change --------- Co-authored-by: Patrick Wolfert <patrick.wolfert@adhocteam.us>
This reverts commit fd0f3bc.
andream-cmsds
approved these changes
Sep 9, 2024
### Usage | ||
|
||
- Utility actions are ghost buttons for available actions that do not take priority over primary or secondary call-to-actions (CTAs). | ||
- Placement should be in the page header and top, right aligned. |
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.
Just a suggestion, but it might be clearer to write "top-right aligned" vs "top, right aligned."
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.
I agree, but since this is approved copy we'll keep it as is. Thanks!
tamara-corbalt
approved these changes
Sep 10, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Impacts: Documentation
Indicates that this item relates to documentation
Type: Added
Indicates a new feature.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Ticket
Added new guidance page for the utility actions pattern.
Note
The design will differ on this branch because the EmailIcon is being imported from the local Doc Site component rather than the Design System. A ticket is up to fix this issue.
How to test
Checklist
[WNMGDS-####] Title
or [NO-TICKET] if this is unticketed work.Type
(only one) label for this PR, if it is a breaking change, label should only beType: Breaking
Impacts
, multiple can be selected.This is a change to documentation:
Screenshots:
With EmailIcon collision:
![Screenshot 2024-09-09 at 9 58 51 AM](https://private-user-images.githubusercontent.com/176332468/365758444-b40df53f-79cf-44f0-bad2-e98392b2a677.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjYxMjUzOTEsIm5iZiI6MTcyNjEyNTA5MSwicGF0aCI6Ii8xNzYzMzI0NjgvMzY1NzU4NDQ0LWI0MGRmNTNmLTc5Y2YtNDRmMC1iYWQyLWU5ODM5MmIyYTY3Ny5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwOTEyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDkxMlQwNzExMzFaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT05ZGU0OGRlZWI3YjZhYzI0NDExZjU0YTQyNThlMGIzNTBhN2FmOGRlY2M2YjkwMmZkNmYzYmU0MzAxMzBlZmZhJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.GvMkUlBR6YtW91GV94kgNGIJWUzAk7vBNz6W-IJkMsA)
Without EmailIcon collision:
![Screenshot 2024-09-09 at 12 39 41 PM](https://private-user-images.githubusercontent.com/176332468/365758492-2a11cbdd-7325-4f5d-a4eb-3dcda47ed279.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjYxMjUzOTEsIm5iZiI6MTcyNjEyNTA5MSwicGF0aCI6Ii8xNzYzMzI0NjgvMzY1NzU4NDkyLTJhMTFjYmRkLTczMjUtNGY1ZC1hNGViLTNkY2RhNDdlZDI3OS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwOTEyJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDkxMlQwNzExMzFaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT00MmI3NDcwNzI2YmMzNTBiMDJjODYzZGNkY2JjMDViZDI1M2VhM2U3MmE5ZGNjN2ZhZmE4OTkwZWNiNmJhYzIwJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.ctBtHN2FMtNLpFR9_uiQ7g841dMQkLXaVUV6ExuCYlI)