-
Notifications
You must be signed in to change notification settings - Fork 13
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
New page: Tags #1832
Comments
AuditAs a first step, I did an audit of tag components on our website. I identified the use case for each and where on the website the specific tag type is used. Filter tag (or display tag)
Toggle tag
Status tag (or tag label)
Results
@sonnakim |
@sonnakim |
We want to document our topic tag in the DS and think adding it to this Tags page draft would work well. I updated the draft. For some reason, the preview link isn’t working for me. If you follow the draft link and click to edit the page, you’ll be able to see the changes in edit mode. And here’s a screenshot. @anselmbradford feel free to add the topic tag code when you’re ready. Any feedback? @sonnakim @natalia-fitzgerald @anselmbradford Side note: I’m not sure what’s going on with the “undefined” jump link at the top... |
Let's do that in a separate PR.
Yea :\ there's an open issue for that #1496 |
Looks good to me 👍 thanks @jenn-franklin ! |
Thanks! I've published the page and added it to the side nav: https://cfpb.github.io/design-system/components/tags @anselmbradford Ready for you to add the topic tag code. Also, would you mind checking out the filter tag code? It's looking funky. |
This is live: https://cfpb.github.io/design-system/components/tags |
Recently my team updated the CSS styling for the Tag component. In the past this component has only appeared as a part of the Multiselect in the Selects section. We would like create a separate page for the Tag component and add this to the CFPB Design System.
What is the standard you're documenting?
Tags
What section will it live in?
Components
When you're ready to share a draft of the page, or want to raise questions, use this issue to track the discussion.
@sonnakim for awareness
The text was updated successfully, but these errors were encountered: