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

docs: combobox HTML correct structure in documentation #867

Merged
merged 7 commits into from
Dec 26, 2019

Conversation

silviuaavram
Copy link
Collaborator

@silviuaavram silviuaavram commented Dec 23, 2019

What:

Fixes #779.

Why:

The main Downshift use example is not accessible with screen readers. The HTML structure for a combobox is not achievable without using getRootProps.

How:

Change main Downshift usage example. Added more information on why is it important to use that instead of the previous one. Also added a docs page for both these usages.

Checklist:

  • Documentation
  • Tests
  • TypeScript Types
  • Flow Types
  • Ready to be merged

@silviuaavram silviuaavram changed the title docs: encourage HTML correct structure docs: combobox HTML correct structure in documentation Dec 23, 2019
@codecov-io
Copy link

codecov-io commented Dec 23, 2019

Codecov Report

Merging #867 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #867   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          12     12           
  Lines         886    886           
  Branches      184    184           
=====================================
  Hits          886    886

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e7e0ed6...742997c. Read the comment docs.

@silviuaavram silviuaavram merged commit e2404b7 into master Dec 26, 2019
@silviuaavram silviuaavram deleted the docs/change-downshift-example branch December 26, 2019 14:54
@silviuaavram
Copy link
Collaborator Author

🎉 This PR is included in version 4.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

VoiceOver cannot find listbox contents using swipe gesture
2 participants