-
-
Notifications
You must be signed in to change notification settings - Fork 31.7k
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
[Autocomplete] Fix autoHighlight for dynamic options #21090
[Autocomplete] Fix autoHighlight for dynamic options #21090
Conversation
Details of bundle changes.Comparing: 8357c68...766dd1c Details of page changes
|
@mstykow Thanks for opening a pull request! Do you or @marcosvega91 want to look at adding a regression test? :) |
Sure thing. First I need to fix the broken test though. :) @marcosvega91 any chance you could give me a hand with this? |
Hi @mstykow of course, I can help you. You can add a new test case inside You can comment the fix that you have done, check that the test fails and apply the fix again |
Thanks guys! |
Closes #21024