-
Notifications
You must be signed in to change notification settings - Fork 89
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
Update SEO docs for PR #1515 #93
base: source
Are you sure you want to change the base?
Conversation
source/docs/theme-settings/seo.md
Outdated
robots: noimageindex, nofollow, nosnippet | ||
noindex: true # This will be ignored if you've set the robots. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In fact, these two parameters can not be used at the same time, here may mislead the user. May be split to explain that it would be better.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm sorry for my poor English. I'm not sure how to explain it so I just provided an example with comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
其实正常的逻辑应该是,先说 robots:
,然后再说 noindex: true
是 robots: noindex
的一个 convenience 。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
其实倒也没有必要非得支持 noindex: true
,只是之前看到有别人用了 noindex: true
这种写法了,觉得保持兼容会门槛低一点,也省得普通用户去理解 robots 的那堆 noimageindex, nofollow, nosnippet 之类的
theme-next/hexo-theme-next#1515