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

Typeahead "limit" if equal to results returned, shows "no-data". #1312

Closed
rash805115 opened this issue Jul 6, 2015 · 6 comments
Closed

Typeahead "limit" if equal to results returned, shows "no-data". #1312

rash805115 opened this issue Jul 6, 2015 · 6 comments

Comments

@rash805115
Copy link

Bloodhound returns correct number of results. But if the number of results is same as the limit option in the datasets, then this results in showing "no-results" instead of showing all of them.

I have created this JSFIDDLE that demonstrates this issue.
http://jsfiddle.net/rash805115/kosocz84/
Note: Searching for 1947 returns exactly 5 rows.

_When limit is set to 4:_
Searching for 1947 returns 4 results.

_When limit is set to 5:_
Searching for 1947 returns nothing.

_When limit is set to 6:_
Searching for 1947 returns one 1 result - the first result.

PS: My first time reporting a bug, so maybe I am not clear enough. Please teach me if I am wrong or something is missing.

@philippeboyd
Copy link

I had the same bug today, here's what I found, hope it helps (it did for me)
http://stackoverflow.com/questions/31007825/bootstrap-typeahead-not-showing-hints-as-expected

@rash805115
Copy link
Author

Awesome. I will also update the issue I opened and mark it as duplicate of
yours. Can you tell me the issue number that you opened.

Regards.

On Tue, Jul 7, 2015 at 6:05 PM, Philippe Boyd notifications@github.com
wrote:

I had the same bug today, here's what I found, hope it helps (it did for
me)

http://stackoverflow.com/questions/31007825/bootstrap-typeahead-not-showing-hints-as-expected


Reply to this email directly or view it on GitHub
#1312 (comment)
.

Regards,
Rahul Chaudhary
Ph - 703-568-4718

@philippeboyd
Copy link

I'm not the one who asked the question on stackoverflow sorry...
but from the looks of it, i think it's this one : #1299

@rsutphin
Copy link

This appears to be the issue fixed by #1212.

@rash805115
Copy link
Author

@rsutphin thanks for the update...any ideas when the new build will be released. Closing this issue.

@nancy-grow
Copy link

you can set limit Infinity,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

4 participants