Skip to content
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

Toggle: Usage and Style documentation #3065

Closed
Tracked by #13521
mbgower opened this issue Jul 30, 2022 · 5 comments · Fixed by #4214
Closed
Tracked by #13521

Toggle: Usage and Style documentation #3065

mbgower opened this issue Jul 30, 2022 · 5 comments · Fixed by #4214

Comments

@mbgower
Copy link
Contributor

mbgower commented Jul 30, 2022

A review of the toggle's Usage and Style pages was carried out as part of an accessibility review. Note that there are comments from both Jess Lin and myself (and I sometimes respond to something she said).

Key items to consider:

  • the Usage page lacks an Anatomy section, which would go a long way to answering some questions we had about how a Heading would be used in a toggle (there are no examples). It may also help align terms on the usage and style pages.
  • the usage page does not list all the variants. There are at least 2, possibly 3: the default, a small WITH on/off text, and a small without on/off text. It's worth making it clear that the on/off text is only optional on the Small variant, not the default
  • on the Style page, an 'off' toggle is shown with a darker switch circle, which makes the entire construct potentially fail non-text contrast (since the contrast between the outline and the circle is only 2.5:1). It appears this is an anomaly (does not appear on the React components page for example), but the toggle button presentation on all 4 themes (white, grey 10, etc) should be carried out to ensure that the color of the circle is the same as the page background AND that the outline of the toggle contrasts at least 3:1 against this background.

image

^ bad example

image

^ less poor example (although I'd bump up the grey of the off toggle while leaving the disc the colour of the background)

This document should be opened in Adobe Acrobat Reader (not in a browser) so that all the comments and replies can be reviewed.

@sstrubberg sstrubberg moved this to 🪆 Needs Refined in Design System Dec 14, 2022
@sstrubberg sstrubberg changed the title Toggle Usage and Style documentation needs some changes [a11y]: Toggle Usage and Style documentation needs some changes Feb 6, 2023
@sstrubberg
Copy link
Member

sstrubberg commented Mar 2, 2023

Needs new Usage template treatment.
Style page images need updates.
Style page: we can drop the on/off because we have the tick.

@sstrubberg sstrubberg moved this from 🪆 Needs Refined to ⏱ Backlog in Design System Mar 2, 2023
@sstrubberg sstrubberg added this to the 2023 Q2 milestone Mar 8, 2023
@sstrubberg sstrubberg changed the title [a11y]: Toggle Usage and Style documentation needs some changes Toggle: Usage and Style documentation needs some changes Apr 12, 2023
@sstrubberg sstrubberg changed the title Toggle: Usage and Style documentation needs some changes Toggle: Usage and Style documentation Apr 12, 2023
@thyhmdo
Copy link
Member

thyhmdo commented Jun 7, 2023

Looks like the Toggle doc has been updated, most of issues were already addressed. Designers can double check/consider if these feedback is outdated

  • Adding an Anatomy image
  • Listing all variants (2- Default and Small)
  • Improving the color contrast of the Off toggle\
  • Updating some issues in the Content section
  • Considering the Adding information to Carbon Data table component https://github.ibm.com/IBMa/E2E/issues/4761

@mbgower
Copy link
Contributor Author

mbgower commented Jun 8, 2023

@thyhmdo I ticked off "Improving the contrast" because the poor example has been removed from the style page.

@thyhmdo
Copy link
Member

thyhmdo commented Jun 9, 2023

Thank you! @mbgower

@sstrubberg sstrubberg modified the milestones: 2023 Q2, 2023 Q3 Jun 28, 2023
@mbgower
Copy link
Contributor Author

mbgower commented Aug 30, 2023

It's also worth mentioning that my poorly worded issue on a requirement for a visible text label (or at least accessible name) for every toggle component was IMO incorrectly closed by in 2019.
carbon-design-system/carbon#3613

I think maybe Carbon now has a requirement for a label for a toggle (except in a table cell). If that's the case, 3613 can be considered closed. But I wanted to capture that as with any input, a toggle requires a label (and "on"/"off" is not the label; that's a text representation of the state).

@sstrubberg sstrubberg modified the milestones: 2023 Q3, 2023 Q4 Sep 19, 2023
@laurenmrice laurenmrice moved this from ⏱ Backlog to ♿ Needs a11y review in Design System Dec 12, 2023
@sstrubberg sstrubberg modified the milestones: 2023 Q4, 2024 Q1 Jan 10, 2024
@laurenmrice laurenmrice modified the milestones: 2024 Q1, 2024 Q2 Mar 27, 2024
@thyhmdo thyhmdo moved this from ♿ Needs a11y review to ⏱ Backlog in Design System Apr 9, 2024
@thyhmdo thyhmdo self-assigned this Apr 24, 2024
@thyhmdo thyhmdo removed the status in Design System May 6, 2024
@thyhmdo thyhmdo removed their assignment Jun 18, 2024
@laurenmrice laurenmrice moved this to ⏱ Backlog in Design System Jun 19, 2024
@github-project-automation github-project-automation bot moved this to Triage in Roadmap Jun 20, 2024
@laurenmrice laurenmrice moved this from Triage to Now in Roadmap Jun 20, 2024
@alina-jacob alina-jacob moved this from ⏱ Backlog to 🏗 In Progress in Design System Jun 21, 2024
@sstrubberg sstrubberg modified the milestones: 2024 Q2, 2024 Q3 Jul 1, 2024
@alina-jacob alina-jacob moved this from 🏗 In Progress to 🚦 In Review in Design System Aug 23, 2024
@kodiakhq kodiakhq bot closed this as completed in #4214 Aug 29, 2024
@github-project-automation github-project-automation bot moved this from Now to Completed in Roadmap Aug 29, 2024
@github-project-automation github-project-automation bot moved this from 🚦 In Review to ✅ Done in Design System Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants