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

fix(head): update open-graph width and height protocol per ogp #1512

Open
wants to merge 1 commit into
base: v4
Choose a base branch
from

Conversation

ssmendon
Copy link
Contributor

I agree to license my contribution under the MIT License.

The https://ogp.me standard doesn't have an a property called "og:width" and "og:height".

The closest thing to that are "Structured Properties" for the image, which are actually "og:image:width" and "og:image:height".

This commit uses the correct property names.

For more information, read this excerpt from https://ogp.me/#structured:

Some properties can have extra metadata attached to them. These are specified in the same way as other metadata with property and content, but the property will have extra :.

The og:image property has some optional structured properties:

  • og:image:url - Identical to og:image.
  • og:image:secure_url - An alternate url to use if the webpage requires HTTPS.
  • og:image:type - A MIME type for this image.
  • og:image:width - The number of pixels wide.
  • og:image:height - The number of pixels high.
  • og:image:alt - A description of what is in the image (not a caption). If the page specifies an og:image it should specify og:image:alt.

The <https://ogp.me> standard doesn't have an a property called "og:width" and "og:height".

The closest thing to that are "Structured Properties" for the image, which are actually "og:image:width" and "og:image:height".

This commit uses the correct property names.
@aarnphm aarnphm changed the title fix(head): use ogp structured properties fix(head): update open-graph width and height protocol per ogp Oct 14, 2024
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