-
Notifications
You must be signed in to change notification settings - Fork 21
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
docs(icon): remove sets demo #2088
base: main
Are you sure you want to change the base?
Conversation
Users should instead see https://ux.redhat.com/foundations/typography
|
✅ Deploy Preview for red-hat-design-system ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Size Change: 0 B Total Size: 208 kB ℹ️ View Unchanged
|
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.
We should add an alert at the top of the rh-icon
demo page rerouting people to the iconography page. There has been confusion about where to find the icons with the users I've worked with, this would be helpful.
<rh-alert state="info">
<h2 slot="header">Looking for the icons?</h2>
<p>All the icons you might need are right at your fingertips! Head over to our <a href="/foundations/iconography/">iconongraphy</a> foundations page to explore the full collection.</p>
</rh-alert>
or something to this effect
@bennypowers I went ahead and dropped a commit adding that alert with the link to the iconography page for your review code wise. @marionnegp what are your thoughts on this? Let me know if you want the language changed. |
Users should instead see https://ux.redhat.com/foundations/typography
What I did