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

Failed to load the popup window of local search [solved] #190

Closed
sundongxu opened this issue Mar 19, 2018 · 28 comments
Closed

Failed to load the popup window of local search [solved] #190

sundongxu opened this issue Mar 19, 2018 · 28 comments

Comments

@sundongxu
Copy link

sundongxu commented Mar 19, 2018

@iissnan
Hello, my problem is about the Local Search function on the version 5.x of Next. After I finished all the configuration to the _config.yml in the next repo according to here: https://theme-next.iissnan.com/third-party-services.html#search-system, and of course enabled the local search function, too, it still failed to load the popup search window just as the underlying image showed, keeping loading....... please help me to solve this problem.
Thanks.

1

@sli1989
Copy link
Collaborator

sli1989 commented Mar 19, 2018

the edge explorer? if it is, you can try to update to V6+ (i have the same problem in v5) or fixed like this.
IDK why isn't work.

@sundongxu
Copy link
Author

@sli1989 chrome... It did work well before, but recently...just go crashed...

@sli1989
Copy link
Collaborator

sli1989 commented Mar 19, 2018

maybe slow network in china..

@ivan-nginx
Copy link
Member

ivan-nginx commented Mar 19, 2018

maybe slow network in china..

Slow network can't be in v5 but not in v6. MB AdBlock or something, need to see live demo at least.

@sundongxu and of course recommended to update from 5 to 6 to bypass all possible bugs and get new features. Docs in CN and EN also aviable.

@sundongxu
Copy link
Author

@ivan-nginx I read the docs you offered about updating and was going to try to update v5 to v6, but there were some customizations that I didn't remember clearly... I don't want any customization for my site to be missing... Any tool recommendation for comparing between the new repo and the old one?

Thanks a lot~

@ivan-nginx
Copy link
Member

«Beyond Compare» to compare files in localhost.

@sundongxu
Copy link
Author

it doesn't work well even after I have already updated the Next to v6...Who can help me debug this annoying bug???
selection_002

@ivan-nginx
Copy link
Member

This config not from v6. Here is current config:

hexo-theme-next/_config.yml

Lines 677 to 687 in f2a939f

# Local search
# Dependencies: https://github.com/theme-next/hexo-generator-searchdb
local_search:
enable: false
# if auto, trigger search by changing input
# if manual, trigger search by pressing enter key or search button
trigger: auto
# show top n results per article, show all results by setting to -1
top_n_per_article: 1
# unescape html strings to the readable one
unescape: false

@sundongxu
Copy link
Author

@iissnan @ivan-nginx
When I click the search button in the top menu, the window keeps loading...and never show up...
selection_003
selection_004

@ivan-nginx
Copy link
Member

Give your reproduce steps and all possible links (repo, live demo, etc.)

@sundongxu
Copy link
Author

@ivan-nginx what material am i supposed to provide to reproduce the bug???Aren't those pictures enough???

@sundongxu
Copy link
Author

@ivan-nginx
My site: dongdongdong.me.

@ivan-nginx
Copy link
Member

what material am i supposed to provide to reproduce the bug???Aren't those pictures enough???

All your steps during upgrading from 5 to 6.

My site: dongdongdong.me.

I see. And i see what this is 5.1.3 version.

@sundongxu
Copy link
Author

@ivan-nginx
Yes, my site is still using v5.1.3 but I did update and run locally while still got failed.

@sundongxu
Copy link
Author

@ivan-nginx
I just modify some css files and the _config.yml after I clone the v6 Next.

@sundongxu
Copy link
Author

@ivan-nginx
They can't be the reasons why it failed...I think.

@ivan-nginx
Copy link
Member

ivan-nginx commented Mar 19, 2018

Ok, let's go with another variant: can u try to install new latest version 6 release with search, search is works?

It must work, because in other peoples search works fine.

  1. Let's look on stargazers
  2. Then look on last man — https://github.com/21sta
  3. Then look at he's repo — 21sta.github.io — go to he's site and as we can see current NexT version is 6.0.6 (latest) and local search works fine.

@sundongxu
Copy link
Author

Still following these steps???
selection_005

@ivan-nginx
Copy link
Member

ivan-nginx commented Mar 19, 2018

Ok, let's go with another variant: can u try to install new latest version 6 release with search, search is works?

How to install NexT v6 see Installation section in readme.

@sundongxu
Copy link
Author

@ivan-nginx
Hey! I just can't believe! You know what... I uninstall the hexo-generator-searchdb module using cmd below:
$ npm uninstall hexo-generator-searchdb --save

And the search window works!!!
Then I quickly switch to the v5, it still works!
Oh yeah...
MB it's because I installed hexo-generator-searchdb while I already got the hexo-generator-search... There may exist some confliction between these two module.

@ivan-nginx ivan-nginx added Solved and removed Invalid labels Mar 19, 2018
@ivan-nginx ivan-nginx changed the title Failed to load the popup window of local search. Failed to load the popup window of local search. [no need to install both hexo-generator-searchdb and hexo-generator-search] Mar 19, 2018
@sundongxu
Copy link
Author

Thank you for being so nice~

@Raincal
Copy link
Member

Raincal commented Mar 19, 2018

@ivan-nginx
hexo-generator-searchdb was forked from hexo-generator-search
This is a bug in early version of the hexo-generator-search and it has been fixed recently
wzpan/hexo-generator-search#30 (comment)

@ivan-nginx
Copy link
Member

@Raincal so, u suggest to update hexo-generator-searchdb?

@Raincal
Copy link
Member

Raincal commented Mar 19, 2018

@ivan-nginx Yes

@ivan-nginx
Copy link
Member

@flashlab can u support this?

@ivan-nginx ivan-nginx mentioned this issue Mar 19, 2018
72 tasks
@ivan-nginx ivan-nginx changed the title Failed to load the popup window of local search. [no need to install both hexo-generator-searchdb and hexo-generator-search] Failed to load the popup window of local search. [solved] Mar 4, 2019
@stevenjoezhang
Copy link
Contributor

One year has passed, is this bug still there?

@ivan-nginx
Copy link
Member

Need to try reproduce this. I think bug still exists, but I can't catch him.

@ivan-nginx ivan-nginx changed the title Failed to load the popup window of local search. [solved] Failed to load the popup window of local search [solved] Apr 1, 2019
@theme-next theme-next locked as resolved and limited conversation to collaborators Apr 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants