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(dropdown): fetch js error on non existing letter selections #3028

Merged
merged 2 commits into from
Apr 9, 2024

Conversation

lubber-de
Copy link
Member

Description

When a dropdown is open, if you type a letter to match a label in the dropdown, an error occurs:

image

Note that you have to type a letter to match a label in the dropdown. If there isn't an entry starting with the letter given, no error will occur.

Testcase

  • Open the dropdown
  • Type "d" on your keyboard

Broken

Console Error
https://jsfiddle.net/lubber/x25f369y/

Fixed

No error, all fine
https://jsfiddle.net/lubber/x25f369y/1/

Closes

#3027

@auto-assign auto-assign bot requested review from ColinFrick, ko2in, prudho and y0hami March 28, 2024 13:32
@lubber-de lubber-de added type/bug Any issue which is a bug or PR which fixes a bug state/awaiting-reviews Pull requests which are waiting for reviews javascript Pull requests that update Javascript code labels Mar 28, 2024
@lubber-de lubber-de added this to the 2.9.4 milestone Mar 28, 2024
@lubber-de lubber-de merged commit f0d7450 into fomantic:develop Apr 9, 2024
9 checks passed
@lubber-de lubber-de deleted the dropdownFetchConsoleError branch April 9, 2024 07:19
@lubber-de lubber-de removed the state/awaiting-reviews Pull requests which are waiting for reviews label Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
javascript Pull requests that update Javascript code type/bug Any issue which is a bug or PR which fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant