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

feat(react): upgrade Downshift dependency to v5 #5357

Closed
wants to merge 7 commits into from

Conversation

dakahn
Copy link
Contributor

@dakahn dakahn commented Feb 14, 2020

We were on v1 Downshift and there are quite very meaningful accessibility fixes/improvements made. This PR upgrades downshift and updates tests/code for any breaking changes.

included changes

  • Downshift updated from v1 to v5
  • combobox/tests updated
  • dropdown/tests updated
  • filterable multiselect/tests updated
  • multiselect/tests updated
  • test helpers updated with new downshift props

to test

Open Multiselect, Dropdown, and Combobox and makes sure all functionality is working as expected 🏄

@aledavila @abbeyhrt and anyone on core working on our listy-dropdowny components. We should do a quick check on any bugs in our backlog and see if we don't get free fixes via this update 👍

@dakahn dakahn requested a review from a team as a code owner February 14, 2020 17:52
@ghost ghost requested review from aledavila and tw15egan February 14, 2020 17:52
@netlify
Copy link

netlify bot commented Feb 14, 2020

Deploy preview for carbon-components-react ready!

Built with commit ae94219

https://deploy-preview-5357--carbon-components-react.netlify.com

@vpicone
Copy link
Contributor

vpicone commented Feb 14, 2020

Are we not going to man-in-the-middle the downShiftProps or whatever that we allow folks to pass in? It would guard against folks that might have been using the deprecated function names

@netlify
Copy link

netlify bot commented Feb 14, 2020

Deploy preview for carbon-elements ready!

Built with commit 7d09374

https://deploy-preview-5357--carbon-elements.netlify.com

Copy link
Contributor

@vpicone vpicone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For safety, we should man-in-the-middle the downShiftProps or whatever that we allow folks to pass in. It would guard against folks that might have been using the deprecated function names

@vpicone
Copy link
Contributor

vpicone commented Feb 14, 2020

Do we have a list of follow up issues to look into after this? It unlocks some really nice things: #4950

@dakahn dakahn changed the title feat(react): upgrade downshift dependency to v5 feat(react): upgrade Downshift dependency to v5 Feb 14, 2020
@dakahn
Copy link
Contributor Author

dakahn commented Feb 15, 2020

@vpicone not that i'm aware of 🤔

@dakahn
Copy link
Contributor Author

dakahn commented Feb 17, 2020

closing and reopening as #5373

@dakahn dakahn closed this Feb 17, 2020
@joshblack joshblack deleted the upgrade-downshift-dependency branch March 18, 2020 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants