Skip to content
This repository has been archived by the owner on Jul 29, 2021. It is now read-only.

老铁我找到 search 的BUG了 #85

Open
css-master opened this issue Mar 19, 2020 · 1 comment
Open

老铁我找到 search 的BUG了 #85

css-master opened this issue Mar 19, 2020 · 1 comment

Comments

@css-master
Copy link

感觉作者大大好像都没处理 Issue,只能寄希望与自己了。。。花了我一个小时擦。。。写在这里给其他喜欢这个主题的老铁们看看吧。 就是在 search.js 下面,一个是

content = Util.filterHTML(entry.children[2].textContent),把这个 children[2] 改成 children[3]

另一个是 isMatch 变量,初次声明要在每个 data.forEach 函数里面,要不然之后永远都是 false,会影响下一次的 loop。

这两点改了之后就能正常使用搜索功能了。 除此之外还要注意下 hexo-generator-search 这个的版本,建议直接在 package.json 中锁死 2.2.2,具体的坑就不多谈了。。。。

本来想干脆提个 pr,但是又想作者可能不会来处理,另外自己也改了很多地方,也就作罢了。不过我想说一下这版本的搜索框样式不如之前那个版本好看,之前那个版本初始时是矮小的且有动画,我自己改回来了。

还有一点就是 tags 那一页也有 BUG,该 BUG 出现在你的某一个标签下超出一页的时候。不过这个很好改,你只要把那个分页的 ejs 模板引擎拉过来放最下面就好了。

这是我第一次去找别人的库中的 BUG 再去修,也算是真爱这个主题了。

@Lemonreds
Copy link
Owner

@cyboning 哈哈哈,感谢喜欢,因为现在工作了,没那么多时间处理了,很抱歉。当然可以提PR,我会定期看邮件的,后面有时间的话,我会统一处理下遗留的issue了。

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

No branches or pull requests

2 participants