-
Notifications
You must be signed in to change notification settings - Fork 34
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
Link canonical cascade #101
Comments
Interesting! What would a cascading a Link canonical look like? Would we just append them down the tree? 🤔 I guess we could support something like this by tweaking the cascade list here: react-head/src/HeadProvider.js Line 4 in 47393b4
Although it would need to be slightly more sophisticated than that if we only want to cascade |
First off, love the lib. Would there be some merit to cascading a
<link rel="canonical" ... />
tag? (link tags aren't cascaded and for good reason)But this particular
link
tag has a uniqueness requirement akin totitle
and othermeta
tags,Multiple
canonicals
are ignored (this is old, but looks to be still relevant)To bring this to a question: right now
canonical
is required to be set on every page, but should this be handled as an edge case and cascaded, or maybe there something a little more interesting here where we can leverage the platform and give anylink
tag the ability to cascade given a unique identifier via anid
orclass
(or other) attribute?One last thing, the
canonical
tag in the readme is using acontent
attribute but I think you want anhref
attribute:<Link rel="canonical" content="http://jeremygayed.com/" />
to<Link rel="canonical" href="http://jeremygayed.com/" />
Thank you for you time
The text was updated successfully, but these errors were encountered: