Skip to content

fix(hooks): call a11y status message on items change#1073

Merged
silviuaavram merged 6 commits intomasterfrom
fix/update-status-on-items
Jun 12, 2020
Merged

fix(hooks): call a11y status message on items change#1073
silviuaavram merged 6 commits intomasterfrom
fix/update-status-on-items

Conversation

@silviuaavram
Copy link
Copy Markdown
Collaborator

@silviuaavram silviuaavram commented Jun 7, 2020

What:
Calling updateA11yStatus when items prop change.

Why:
To reflect, for instance, the number of results change when combobox filters the items by input value
Closes #1076.

How:
Add items to the list of dependencies that trigger the effect with the status update.

Checklist:

  • Documentation
  • Tests
  • TypeScript Types N/A
  • Flow Types N/A
  • Ready to be merged

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jun 7, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #1073   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           17        17           
  Lines         1319      1319           
  Branches       241       241           
=========================================
  Hits          1319      1319           
Impacted Files Coverage Δ
src/hooks/useCombobox/index.js 100.00% <ø> (ø)
src/hooks/useSelect/index.js 100.00% <ø> (ø)

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 45985a8...bbcba0c. Read the comment docs.

@silviuaavram silviuaavram merged commit 88559fc into master Jun 12, 2020
@silviuaavram silviuaavram deleted the fix/update-status-on-items branch June 12, 2020 06:36
@silviuaavram
Copy link
Copy Markdown
Collaborator Author

🎉 This PR is included in version 5.4.3 🎉

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.

useCombobox: a11y status message doesn't seem to work properly compared with Downshift component

2 participants