-
-
Notifications
You must be signed in to change notification settings - Fork 8.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
arrow up causing crashed when using @docusaurus/theme-search-algolia #5031
Comments
@vagusX I'm sorry but I can't reproduce on that page https://docusaurus.io/docs/api/themes/@docusaurus/theme-search-algolia As this issue has been reported first on the Ant design website (apparently), why are you reporting it to Docusaurus? Ant Design website is not using Docusaurus. I can reproduce it on the Ant design website, not on Docusaurus website. @shortcuts is this issue known? |
Hi, I'm not able to reproduce this issue in an up-to-date DocSearch v3 experience (with ant design index https://2ppfv.csb.app/) @slorber do you have a link where I could see the issue on the Ant design website? @vagusX if you can still reproduce on your side, could you please let me know which User Agents are sent with the query? They should contain the version of the DocSearch and Autocomplete library, which might gives a hint. |
I can reproduce it on the url reported here ant-design/ant-design#31040: https://ant.design/ |
@slorber would you like to clean all recent items and try it again? |
Same issue @shortcuts The component version is here https://github.com/facebook/docusaurus/blob/master/packages/docusaurus-theme-search-algolia/package.json#L16 I just found the code is archived https://github.com/francoischalifour/autocomplete.js/blob/ds-react/src/DocSearchModal.tsx (the latest code is different with @docsearch/[email protected]) I just wonder where is |
Yep the repo of the underlying library is now https://github.com/algolia/autocomplete, thanks for pointing that out! The PR I've opened (algolia/autocomplete#599) will also fix this issue. |
Wow, just looking forward for this fix pr release. @shortcuts would you like to tell that when your pr ship out. |
To reproduce the issue, you need to open the search popup and make sure that no entries are displayed. If there are previous search entries, you can remove them. |
Update: this PR has been merged but didn't solved the DocSearch issue, I've found an other related issue which should solve it: algolia/autocomplete#623 The DocSearch update will be released by the end of the week! |
🐛 Bug Report
Prerequisites
npm run clear
oryarn clear
command.rm -rf node_modules yarn.lock package-lock.json
and re-installing packages.Description
(A clear and concise description of what the bug is.)
Have you read the Contributing Guidelines on issues?
(Write your answer here.)
Steps to reproduce
Use https://new.docusaurus.io to create a CodeSandbox reproducible demo of the bug.
(Write your steps here:)
Cmd + K
↑
(arrow up)Expected behavior
No crashing
(Write what you thought would happen.)
Actual behavior
(Write what happened. Add full console log messages and screenshots, if applicable.)
Your environment
Reproducible demo
https://docusaurus.io/docs/api/themes/@docusaurus/theme-search-algolia
The text was updated successfully, but these errors were encountered: