docs(helpers/open_graph): Fix incorrect default value and unintended rendering #2156
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.
Check List
en
Englishru
Russianzh-cn
simplified Chinesezh-tw
traditional Chineseja
JapaneseIncorrect Default Value
The value
blog
specified as the default value forog:type
is incorrect. As per the following implementation:https://github.com/hexojs/hexo/blob/v7.1.1/lib/plugins/helper/open_graph.ts#L82
The default value is
article
orwebsite
depending on whether it is a post page or not.Unintended Rendering
The default value of
og:locale
is chosen from several settings, as implemented below:Although this was written in the documentation, the
|
in this code was treated as a table separator, resulting in the display being cut off in the middle. I fixed it by escaping|
.Before fix:
After fix: