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

Fixes two TypeError exceptions when up/down arrow pressed #2585

Merged
merged 2 commits into from
Mar 27, 2021

Conversation

rjregenold
Copy link
Contributor

@rjregenold rjregenold commented Mar 12, 2021

It looks like issue #2035 is happening again in the latest v1.13 release. When using live search, if a user enters a query that has no matches and then presses the up or down arrows, two different TypeError exceptions can occur. This PR fixes both exceptions.

For reference, here are the steps to reproduce the issue, copied from the original issue:

Browser: All
OS environment: All

Steps to reproduce:

  1. Open selctbox and type "xxx" to search
  2. No results matched "xxx" will appear
  3. Press DOWN key
  4. Error is thrown

RJ Regenold added 2 commits March 12, 2021 10:30
when using live search, if a user enters a value that does not return
any results and then presses the up/down arrows, this error occurs. this
commit fixes that error.
when using live search, if a user enters a value that does not return
any results and then presses the up/down arrows, this error occurs. this
commit fixes that error.
@caseyjhol
Copy link
Member

caseyjhol commented Mar 27, 2021

I can confirm that this issue exists, but this will require a different fix for v1.14.0, as it's been refactored.

This fix should work in v1.14.0 as well.

@caseyjhol caseyjhol merged commit 75d6d8a into snapappointments:v1.13.0-dev Mar 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants