There's some confusion between the two Facebook tags and the OpenGraph tags - the former are not needed to use the latter, and vice-versa. In order to simplify the UX the Facebook tags should be in a separate group.

CommentFileSizeAuthor
#6 metatag-n2276361-6.patch7.24 KBdamienmckenna
#2 metatag-n2276361-2.patch8.33 KBdamienmckenna

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Component: Open Graph » User interface
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new metatag-n2276361-2.patch8.33 KB

This is almost done, it just needs some improvements to hook_update.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

FYI I'm splitting the two tags into their own module because they were rarely needed and aren't directly related to the Open Graph tags.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

2: metatag-n2276361-2.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 2: metatag-n2276361-2.patch, failed testing.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Needs review
StatusFileSize
new metatag-n2276361-6.patch7.24 KB

Rerolled after #1809356: Meta tag output sorting / order was committed.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Committed.

  • Commit 7f8e2e4 on 7.x-1.x by DamienMcKenna: 
    Issue #2276361 by DamienMcKenna: Move Facebook meta tags into a separate...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.