Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
feat(pages): add support for missing SEO front matter + improve SEO docs #9071
feat(pages): add support for missing SEO front matter + improve SEO docs #9071
Changes from 1 commit
9132698
9453d83
460ab57
ccbb29d
bc068e1
4b4dd95
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Since this is generally about improving SEO docs. And
front matter
is a pretty big deal for this, we should also in this same PR try to clean up some missing links aboutfront matter
.One thing I noticed above on line 61 is the mention of
front matter
. That should probably be turned into a link to the page on front matter. In general, I've noticed we are pretty bad at just typing names, when they really should be links to their respective pages, especially anything like plugin, api, etc.I think the general link for the feature would be appropriate here I think:
https://docusaurus.io/docs/markdown-features#front-matter
But I also notice we don't have a link to the API when we describe front matter on that feature section! Which should have a link to the API for the plugin, right? https://docusaurus.io/docs/api/plugins/@docusaurus/plugin-content-docs#markdown-front-matter
But we DO have an API link on the Create a Doc section about Doc front matter
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 not sure how this comment is related to the
twitter:card
meta? 🤔Will add links to front matter
Note: the SEO page is generic so it can only link to the generic front matter page.
It is normal that a docs page links to docs-specific front matter. Each plugin can have its own front matter.
To not bias toward a single plugin I'll add links to all the API ref supporting front matter: