Amy Leadem

Results 201 comments of Amy Leadem

@amycole501 Great, I'll mark this as an enhancement instead of a bug. How does this impact the accessibility checklist for tooltip? Do we rewrite or remove the related tests?

@amycole501 @alex-hull If we mark these as conditional with the tests as written, I worry that users will go looking for the word "tooltip" in their project readouts but won't...

Confirming that this issue happens when the system builds out `html-templates` with `npm run build:html`. The issue does not appear in the component's Storybook preview. Tagging this issue for 3.8.1,...

Update: I have separated out the accidental "An" piece of this issue into its own issue: #5893

Adding a few more recommendations for future tasks: - Standardize the structure for form element markup (For example, some components include a `usa-form` wrapper, others don't) - Use includes to...

@mejiaj and @mahoneycm, this still needs JSDoc comments and probably a unit test, but opening this review now so that we can start discussing if there are ways to write...

@amycole501 and @alex-hull can you confirm that the tooltip now conforms with the "hoverable" criteria in [WCAG 1.4.13](https://www.w3.org/WAI/WCAG21/Understanding/content-on-hover-or-focus.html)?

Sorry @amycole501, I accidentally re-requested your review, which cleared the green approval check 🤦

@amycole501 and @alex-hull could you test the three new race and ethnicity pattern examples for accessibility? There are also a couple of possible enhancements that we could use your input...

Putting this effort on hold until the fixes uswds/uswds#5919 and uswds/uswds#5909 are merged.