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
Inconsistent marking behavior with custom search #2200
Comments
I think that is a quite significant inconsistence in behavior, so I will be just bumping this, so that it doesn't go stale |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions 🎆 |
Bump |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions 🎆 |
Bump |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions 🎆 |
Describe the bug
According to the example page, the behavior of marking a result item is different between default search and custom search.
In case of [searchable]="true" the first Element of the results is marked, does not depend on if the results starts with the character or only contains it.
In Case of [searchable]="false" and searching on an input element, the element with the starting character is marked
(and in case of many results the container scrolls to this element), not the first one of the list. This behavior gets a bit weird in case of [searchable]="false" and typing a word slowly. Result marking and scrolling are jumping all the time.
Reproducible example
NgSelect example page
To Reproduce
Steps to reproduce the behavior (if example is not provided):
Expected behavior
The marking algorithm should be consistent, independent from the type of search
Screenshots
Using the default searching:
Using custom search control:
Desktop (please complete the following information):
It's a platform independent issue
Smartphone (please complete the following information):
N/A
The text was updated successfully, but these errors were encountered: