Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
See #1278146: [META] Ensure all meta tags are supported (no patches, just planning) for missing or incomplete tags required for a 1.0 release.
This ticket is for patches containing code for OpenGraph tags
Comment | File | Size | Author |
---|---|---|---|
#11 | metatag-n1900434-11.patch | 7.21 KB | DamienMcKenna |
#7 | metatag-7.x.1.x-opengraph-1900434-7.patch | 4.8 KB | j0rd |
#1 | metatag-7.x.1.x-opengraph-1900434-1.patch | 2.31 KB | Dustin Currie |
Comments
Comment #1
Dustin Currie CreditAttribution: Dustin Currie commentedPatch containing support for:
og:video
og:video:secure_url
og:video:height
og:video:width
og:video:type
Comment #2
DamienMcKennaClarified the title.
Comment #4
bleen CreditAttribution: bleen commented#1: metatag-7.x.1.x-opengraph-1900434-1.patch queued for re-testing.
Comment #5
bleen CreditAttribution: bleen commentedI just played with the patch in #1 and everything looks good. This is certainly a needed addition...
Comment #6
DamienMcKennaTag.
Comment #7
j0rd CreditAttribution: j0rd commentedI've added OG Location & Contact information to the original patch.
Since there's no descriptions on Facebook about these fields (and they're pretty self explanatory) I haven't added them in the module.
Comment #9
obleser CreditAttribution: obleser commentedOpen Graph != Facebook Graph support!
Open Graph definitions could be found here http://ogp.me/
Comment #10
j0rd CreditAttribution: j0rd commented@obleser. Thanks for pointing that out.
I would assume 90% of the people who're using open graph support with this module are using it because they want to support facebook open graph. I hate not following standards as much as the next person, but we're going to need to support the "big players" in this module for it to be useful for site creators.
With that said, I should probably change my facebook specific stuff into another metatag group instead of open graph group, which should remain standard compliant. I'll look into that when I have some time and make sure my patch applies properly for test bot.
Comment #11
DamienMcKennaRerolled and tidied up a little. I replaced the video type text field with a selector to make life easier.
Comment #12
DamienMcKennaLets just go with this for now.
Comment #13
DamienMcKennaNow that Metatag v7.x-1.0-beta5 is out am closing this to keep the issue queue clean.
Comment #13.0
DamienMcKennaLink ticket