Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nico059’s picture

I just realease a sandbox project here: http://drupal.org/sandbox/nico059/1741784
I name it metatag_twittercard (could be changed if needed).
The code is inspired from the metatag_opengraph module.

Very basic for now (only support summary card type), but I will add the 2 others type asap
I'm waiting to participate to Twitter Card to test this module, so use at your own risk ;-)

Twitter Cards support the OpenGraph meta tags, so I leverage on this metatags (og:xyz..) for the description, title, image etc ... Not sure yet if this is the good strategy or if it's better to make metatag_twittercard independant (without dependency to the metatag_opengraph module)

Give me some feedback on this module and which feature you see important,

Enjoy ;-)

Paisible’s picture

+1
I vote for this feature too.

Tanzaho’s picture

I ma also following this thread since I am really interested.

Tanzaho’s picture

I used it, and added the 'image' and 'description' tag. Waiting for twitter approval now.

nico059’s picture

Hi Tanzaho, thank you for your interest in this small module.

Could you please share the code you added for image and description? A patch, I could merge ?

Thx,
Nico.

Tanzaho’s picture

I just sent you a mail. Check it out...

ipwa’s picture

I guess the next step would be for nico059 to make a patch on from Meta tags HEAD, adding his module as a sub module of Meta tags.

kerasai’s picture

Patch rolled from latest 7.x-1.x.

DamienMcKenna’s picture

Status: Active » Needs work

Good work, but metatag_twittercard_metatag_config_default_alter() needs to be updated for the Twitter tags :)

DamienMcKenna’s picture

miechiel’s picture

I also started a sandbox project (see #10) to extend metatag with Twitter Cards metatag. Here is my work patched on nico059 sandbox.
This patch adds more twitter cards metatags and is using a proper metatag_twittercard_metatag_config_default_alter() function. I also added the dependency in the info file.

I'am also thinking in adding more functionality for "media" files such as images and audio/video. So that the "player" and "image" Twitter Card can be used.

idflood’s picture

Status: Needs work » Needs review
FileSize
5.69 KB

Here is a patch from miechiel sandbox with some little fixes (3 whitepace issue and a double break at the same spot).

@DamienMcKenna: Anything else to have this committed to metatag?

DamienMcKenna’s picture

FileSize
0 bytes

How's about this? :)

DamienMcKenna’s picture

FileSize
10.03 KB

Doh.

alexweber’s picture

FileSize
521 bytes

The patch in #14 works partially as the "twitter:card" metatag's options and human-readable name are switched so it outputs the administrative title instead of the actual value in the metatag. Also, the site's twitter account should have its context set to to global.

Patch attached.

alexweber’s picture

FileSize
10.06 KB

Doh! Correct patch attached.

Status: Needs review » Needs work

The last submitted patch, metatag-n1664322-15.patch, failed testing.

alexweber’s picture

FileSize
10.06 KB

getting my butt kicked. this one should do it.

alexweber’s picture

Status: Needs work » Needs review
DamienMcKenna’s picture

FileSize
11.52 KB

I decided to add a README.txt file for the submodule.

DamienMcKenna’s picture

Status: Needs review » Fixed

Committed! Thanks everyone!

DamienMcKenna’s picture

DamienMcKenna’s picture

Status: Fixed » Closed (fixed)

Version 7.x-1.0-beta3 has been released and includes this work. Am closing this issue in an effort to keep the issue queue clean. Thank you all for your continued help!

alexweber’s picture

Awesome :)

jbrown’s picture

Component: Code » Documentation
Status: Closed (fixed) » Active

This needs to be documented on the module front page.

Dave Reid’s picture

Component: Documentation » Code
Status: Active » Closed (fixed)

Added to homepage.

jbrown’s picture

Thanks!

jbrown’s picture

Issue summary: View changes

Added a link to the corresponding Nodewords issue.