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

remove search string in url #18

Closed
wants to merge 1 commit into from
Closed

Conversation

szcf-weiya
Copy link

有时在其他网站上分享链接,会自动加上搜索字符串。举个例子,在其他网站上点击http://example.com/index.html时会变成http://example.com/index.html?from=timeline,disqus默认会忽略搜索字符串,即?from=timeline;但iDisqus将其视为两个url,则第二个链接不会加载评论。如

ex

考虑到实际需求,建议去掉搜索字符串。

@fooleap
Copy link
Owner

fooleap commented Feb 22, 2018

此前考虑到有些非静态网站,如 WP,其默认 URL 带搜索字符串,故页面链接默认是以 location.pathname + location.search 为准,可以在创建实例时 配置 url 指定页面唯一链接。

@szcf-weiya
Copy link
Author

原来如此!没接触过WP,不太了解。

@fooleap fooleap force-pushed the master branch 7 times, most recently from 491b704 to 747b3f7 Compare March 10, 2018 09:52
@fooleap fooleap closed this Mar 11, 2018
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

Successfully merging this pull request may close these issues.

2 participants