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

Toggletip: v11 audit #10438

Closed
17 of 29 tasks
Tracked by #10551
aagonzales opened this issue Jan 13, 2022 · 3 comments
Closed
17 of 29 tasks
Tracked by #10551

Toggletip: v11 audit #10438

aagonzales opened this issue Jan 13, 2022 · 3 comments
Assignees
Milestone

Comments

@aagonzales
Copy link
Member

aagonzales commented Jan 13, 2022

Audit

A designer and developer will pair on this audit, some tasks may be discipline focused. Follow the checklist below for the features and items to review. If there are no issues with the item then check the item as complete. If any problems or bugs come up when auditing add a comment to this issue with the problem and how to correct it (if you know how). Bugs do not need to be fixed while auditing.

Resources

Checklist

Visual (in React code)
Design and dev check

  • Component is using the correct design tokens (color and type), see design spec and scss code.
  • Component is rendering correctly across themes and layers, see design spec and storybook.
    • White theme
    • Gray 10 theme
    • Gray 90 theme
    • Gray 100 theme
  • Component is rendering correctly across browsers (check themes across browsers as well), see storybook.
    • Firefox
    • Safari
    • Chrome
  • Component sizes are rendering and named correctly (if applicable) NA

Website (v11)
Design checks

  • Style tab has correct design tokens listed
  • Style and usage tabs are using the correct size props names (if applicable) NA
  • Usage tab is up-to-date for with any v11 behavioral changes (if any)
  • Live Demo has applied v11 changes
    • Themes are rendering correctly
    • Size props are named correctly NA
    • No light props included

Design Kits (Sketch only)
Design checks

  • Correct design tokens (type and color) are used
  • Component using the correct size prop names NA
  • Any additional v11 behaviors have been added

Storybook
Dev checks

  • Confirm that prop table is populating
  • Take note of missing examples/stories (if any)

React package
Dev checks

Accessibility
Dev checks

  • Confirm that there are no violations in Accessibility checker
  • Confirm that component works as expected with VoiceOver

Migration docs
Design and dev check

  • Any breaking changes to this component are present in the v11 migration guide.
    • Design
    • Develop
@laurenmrice
Copy link
Member

laurenmrice commented Feb 8, 2022

Visual bugs

*Link in toggletip: Known issue to make color tokens for the hover and active state for inverse-links. Just need to make the issue and will post it here.

*Button in toggletip: Known issue to about the inset color during the focus state. Just need to make the issue and will post it here.

scss

  • Text inside of the toggletip should be using type token $body-01. As per our discussion, we are going to keep the type token at body-01 in cases where text wraps to multiple lines.

@laurenmrice
Copy link
Member

laurenmrice commented Feb 8, 2022

Website

Usage tab

  • Change main image of page.
  • Add a sentence explaining what we offer out of the box for Toggletip versus what people can make with it on their own.

Code tab

A11y tab

Live demo

@tay1orjones tay1orjones moved this to ⏱ Backlog in Design System Feb 25, 2022
@tay1orjones tay1orjones added this to the v11 GA milestone Feb 25, 2022
@joshblack joshblack moved this from ⏱ Backlog to 🏗 In Progress in Design System Mar 10, 2022
@tay1orjones
Copy link
Member

Closing since we've got everything remaining tracked in #2624, #2623, #2622, #10973

Repository owner moved this from 🏗 In Progress to ✅ Done in Design System Apr 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

4 participants