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
[Component] Alerts Component (Replaces Notifications) #263
Conversation
Changes: - Moved the Label component into the verified state - Changed various references to use Label instead of label
Changes: - Simplified the TextInput component to an atom, removed baked-in Notification - Updated Storybook description to match CFPB design - Updated Storybook stories to match CFPB TextInput examples
Changes: - The type property (success, warning, error, info) is now -- status - Allowed the field level component be usable by itself (unlinked) - Updated the Storybook Stories showing field level usable separately in conjunction with TextInput - Updated the Storybook stories to have parity with the DS equivalent - Updated the tests where the field level component is separate
Changes: - The type property (success, warning, error, info) is now -- status - Allowed the field level component be usable by itself (unlinked) - Updated the Storybook Stories showing field level usable separately in conjunction with TextInput - Updated the Storybook stories to have parity with the DS equivalent - Updated the tests where the field level component is separate
✅ Deploy Preview for cfpb-design-system-react ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Looking pretty good. One issue with the blank status.
Also, I think the majority of latest PR feedback for the TextIntroduction apply here.
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.
👍🏾
Closes #240
Changes
Verification Checklist
See #240 For Verification Checklist
Accessibility Tests (WAVE)
How to test this PR
yarn test Alert
Screenshots