-
Notifications
You must be signed in to change notification settings - Fork 516
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
Url sync examples implementation should not update at every character #1856
Labels
Comments
vvo
pushed a commit
that referenced
this issue
Jan 26, 2017
Before this commit, we were advocating for updating the url at every keystrokes. We had multiple reports since releases of instansearch.js that this led to slowdowns of the browser UI. fixes #1856
vvo
pushed a commit
that referenced
this issue
Jan 26, 2017
Before this commit, our default implementation tried to update the url at every character. This led to many slowdown reports from users of the library. Now we only update the url after the threshold fixes #1856
vvo
pushed a commit
that referenced
this issue
Jan 27, 2017
Before this commit, we were advocating for updating the url at every keystrokes. We had multiple reports since releases of instansearch.js that this led to slowdowns of the browser UI. fixes #1856
vvo
pushed a commit
that referenced
this issue
Jan 27, 2017
Before this commit, we were advocating for updating the url at every keystrokes. We had multiple reports since releases of instansearch.js that this led to slowdowns of the browser UI. fixes #1856
vvo
added a commit
that referenced
this issue
Jan 27, 2017
) Before this commit, we were advocating for updating the url at every keystrokes. We had multiple reports since releases of instansearch.js that this led to slowdowns of the browser UI. fixes #1856
vvo
added a commit
that referenced
this issue
Jan 27, 2017
Before this commit, our default implementation tried to update the url at every character. This led to many slowdown reports from users of the library. Now we only update the url after the threshold fixes #1856
vvo
pushed a commit
that referenced
this issue
Feb 6, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No description provided.
The text was updated successfully, but these errors were encountered: