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

Search: does not work anymore #839

Closed
jaykayone opened this issue Sep 1, 2015 · 6 comments
Closed

Search: does not work anymore #839

jaykayone opened this issue Sep 1, 2015 · 6 comments
Assignees

Comments

@jaykayone
Copy link
Contributor

(Firefox on Mac): You cannot choose a result from the search anymore by hitting enter or clicking the result

Might be a build problem ... we'll see

@jaykayone
Copy link
Contributor Author

@elemoine might this be a regression coming from ngeo?
As @petzlux has not changed anything this week, I wonder where this regression might come from

@elemoine
Copy link
Contributor

elemoine commented Sep 2, 2015

This is probably a regression introduced by #817.

@jaykayone
Copy link
Contributor Author

I get an Error: Pattern length is too long in debug mode (console).

@elemoine
Copy link
Contributor

elemoine commented Sep 2, 2015

I don't know what this means…

@elemoine
Copy link
Contributor

elemoine commented Sep 2, 2015

Ok, we're fixing a regression introduced in typeahead 0.11.1. The regression is tracked by twitter/typeahead.js#1313. And there is an (unmerged) PR here: twitter/typeahead.js#1207.

One workaround would involve storing the dataset in the suggestion object, but this would require some work, as we handle different suggestion types.

Another option is to try to revert #817. This may just require adding a custom extern type for TypeaheadDataset. Not so bad I think.

It took me 1h to investigate the issue. I'll take me another hour or so to revert #817 and introduce the extern type override.

@elemoine
Copy link
Contributor

elemoine commented Sep 2, 2015

I finally decided to make the revert PR. See #849. And it actually took me less than 1h.

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

No branches or pull requests

3 participants