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: filter list checkbox properly hidden and has proper IDs #321

Merged

Conversation

jhung
Copy link
Contributor

@jhung jhung commented Apr 22, 2020

Description

Resolve an issue where invisible checkbox in filter-list was receiving keyboard focus. Also fixes an issue where filter-list checkbox IDs were not properly populated.

Steps to test

  1. Go to filter-list
  2. Use keyboard to expand and tab through list of filters
  3. Focus should move from visible element to visible element in a predictable and expected way.

Related issues

Fixes #317.

@jhung jhung requested a review from greatislander April 22, 2020 15:17
@jhung jhung self-assigned this Apr 22, 2020
@greatislander greatislander modified the milestone: 1.0.0-alpha.14 Apr 22, 2020
@netlify
Copy link

netlify bot commented Apr 22, 2020

Deploy preview for pinecone ready!

Built with commit e8b97c8

https://deploy-preview-321--pinecone.netlify.app

@greatislander greatislander added the bug Something isn't working label Apr 22, 2020
@greatislander greatislander added this to the 1.0.0-beta.1 milestone Apr 22, 2020
@jhung
Copy link
Contributor Author

jhung commented Apr 28, 2020

@greatislander I've tested this fix in the resource library and it does not affect the functionality of hidden inputs from being submitted.

@greatislander greatislander merged commit 3d2d85d into platform-coop-toolkit:dev Apr 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

aria-hidden checkbox in Filter list is receiving invisible keyboard focus
2 participants