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

fix: a11y: adds css to hide search clear button that causes accessibility issues #2467

Merged
merged 2 commits into from
Jul 6, 2023

Conversation

shkeating
Copy link
Contributor

Summary

Related Issues or PRs

USWDS issue: uswds/uswds#5277

Chromium issue: https://bugs.chromium.org/p/chromium/issues/detail?id=1147314

How To Test

yarn storybook

http://localhost:9009/?path=/story/components-search-search--default-search

  1. type in the search box in Chrome
  2. make sure the little X icon does not show up

Screenshots (optional)

before:
image

after:
image

@shkeating shkeating added type: bug Something isn't working like it's supposed to a11y Relates to accessibility standards and practices labels Jun 28, 2023
@shkeating shkeating requested a review from gidjin as a code owner June 28, 2023 16:24
@shkeating shkeating self-assigned this Jun 28, 2023
@shkeating shkeating requested a review from a user June 28, 2023 16:24
@brandonlenz
Copy link
Contributor

Since the SCSS implementation of this library is always hard (to me) I'll be curious to know if you have to do anything special to make sure to use react-uswds' index.scss when setting up react uswds + uswds on consuming projects

@shkeating shkeating merged commit d19527f into main Jul 6, 2023
@shkeating shkeating deleted the shk-search-a11y-2466 branch July 6, 2023 17:21
gidjin pushed a commit that referenced this pull request Jul 14, 2023
## [5.1.0](5.0.0...5.1.0) (2023-07-14)


### Features

* added customizable text control to file input ([#2417](#2417)) ([0a387be](0a387be))
* adds unbounded functionality to pagination ([#2418](#2418)) ([f9e7957](f9e7957))


### Bug Fixes

* a11y: adds css to hide search clear button that causes accessibility issues ([#2467](#2467)) ([d19527f](d19527f))
* added error param, test, story ([#2488](#2488)) ([42da85d](42da85d))

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Relates to accessibility standards and practices type: bug Something isn't working like it's supposed to
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants