-
-
Notifications
You must be signed in to change notification settings - Fork 563
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
Improve SEO features #1218
Comments
TWITTER cards, canonical url |
Alt tag on image blocks |
@jbrozek The image block just refers an image in the media library which has a field for alt text (added in 8.1) Regards |
If this hasn’t been done, I’ll be happy to make the change. |
Hi @danielshue! No this hasn't been implemented yet! It's basically about adding some fields to Pages & Posts and updating the |
@danielshue We're always happy for contributions, so feel free to join in! Much appreciated |
Great! I'll give it a go! |
The Open Graph Protocol - https://ogp.me has a number of supported attributes. The minimum values appear to be:
Do we want to support the additional properties available for image such as image:type, image:width, image:alt, and image:height? If an image isn't provided, it seems like using the Piranha logo seems like a natural choice. |
@danielshue Well these were my initial thoughts.
As for Regarding meta information about the selected image this can probably be collected from the media object as this will contain all of these fields. What do you think? |
@danielshue And when I wrote that it should default to other fields, I think a good solution is to just show the default values as |
@danielshue Added first version of helper for rendering meta-tags. See the commit above |
@danielshue Just so I know, are you looking at this issue. I'm thinking about starting with #1000 which also requires Db migrations and sometimes it can get a bit messy when trying to merge migrations from several branches. |
The following fields should be provided out of the box
Basic
Open Graph
Should there be more fields? If so please comment below
The text was updated successfully, but these errors were encountered: