Skip to content

Conversation

@Pytal
Copy link
Contributor

@Pytal Pytal commented Jul 21, 2022

@Pytal Pytal added 2. developing Work in progress accessibility Making sure we design for the widest range of people possible, including those who have disabilities feature: emptycontent Related to the emptycontent component labels Jul 21, 2022
@Pytal Pytal added this to the 6.0.0 milestone Jul 21, 2022
@Pytal Pytal self-assigned this Jul 21, 2022
@Pytal Pytal force-pushed the enh/a11y-empty-content branch from a4b50a5 to 6163cbb Compare July 22, 2022 05:07
@Pytal
Copy link
Contributor Author

Pytal commented Jul 23, 2022

Would it make sense to recommend adding the accessible title prop to EmptyContent icons @nimishavijay?

It would show a tooltip like below whenever you hover over the icon

image

@Pytal Pytal force-pushed the enh/a11y-empty-content branch 2 times, most recently from 63d65e0 to 26884e0 Compare July 23, 2022 00:25
@nimishavijay
Copy link

Would it make sense to recommend adding the accessible title prop to EmptyContent icons?
It would show a tooltip like below whenever you hover over the icon

I would say this image is decorative, as all the information required by a person on this page is given through the text.

If the purpose of a title prop here is to make the page screen-reader friendly, I would day we don't need to include it here because there should be a some supporting text under the icon, and that works for a screenreader also :)

cc @jancborchardt

@jancborchardt
Copy link
Contributor

Yep, as so often the icon is decorative and has its text there as h2, and thus should have empty alt text alt="" to designate that it's decorative and no title to prevent duplication.

- Drop support for class based icons
- Title is now optional

Signed-off-by: Christopher Ng <[email protected]>
@Pytal Pytal force-pushed the enh/a11y-empty-content branch from 26884e0 to 7282eb7 Compare July 26, 2022 00:13
@Pytal Pytal added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Jul 26, 2022
@Pytal Pytal marked this pull request as ready for review July 26, 2022 00:30
@nickvergessen nickvergessen added the 💥 breaking PR that requires a new major version label Jul 26, 2022
@Pytal Pytal added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jul 26, 2022
@Pytal Pytal merged commit 6b6b3a4 into master Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish accessibility Making sure we design for the widest range of people possible, including those who have disabilities 💥 breaking PR that requires a new major version feature: emptycontent Related to the emptycontent component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BITV/A11Y] EmptyContent accessibility

7 participants