demo(icon): Add an accessibility demo page for icon #6348
Conversation
googlebot
added
the
cla: yes
tinayuangao
added
Accessibility
googlebot
added
the
cla: yes
| @@ -0,0 +1,14 @@ | |||
| <section> | |||
| <h2>Home icon (Ligature from Material Icons font)</h2> | |||
There was a problem hiding this comment.
I think we should have examples for each of the accessibility use cases listed here: https://material.angular.io/components/icon/overview
There was a problem hiding this comment.
Thanks! Changed to uses cases listed in https://material.angular.io/components/icon/overview
| @@ -0,0 +1,17 @@ | |||
| <section> | |||
| <h2>Fingerprint icon (Decorative icons)</h2> | |||
There was a problem hiding this comment.
We should make this something like
<h2>Fingerprint status</h2>
<p>Showing a status message with a purely decorative icon</p>
<md-icon color="primary">fingerprint</md-icon>
Your fingerprint was recorded!
(similar structure for other examples)| <h2>Delete icon (Interactive icons) </h2> | ||
| <button md-icon-button aria-label="Delete" (click)="deleteIcon()"> | ||
| <md-icon color="accent">delete</md-icon> | ||
| </button> |
There was a problem hiding this comment.
We should have an example with <a md-icon-button> as well
|
Added another example with |
jelbourn
added
pr: lgtm
action: merge
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
4 participants
No description provided.