Allow for meta property tags, along with other types of meta tag.

Comments

damienmckenna’s picture

Status: Active » Needs review
StatusFileSize
new1.39 KB

This patch was extracted from #803512: Support Open Graph Protocol meta tags.

Status: Needs review » Needs work

The last submitted patch, nodewords-n1359988.patch, failed testing.

damienmckenna’s picture

This makes all tags change to "properties" rather than "name".

damienmckenna’s picture

Status: Needs work » Needs review
StatusFileSize
new1.39 KB

It's amazing what happens when you forget to include an equals sign when doing a comparison. This works better.

damienmckenna’s picture

Status: Needs review » Fixed

Committed, and proper attribution given to sammys for the patch, given it came from his work in #803512: Support Open Graph Protocol meta tags.

Status: Fixed » Closed (fixed)

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

damienmckenna’s picture

Status: Closed (fixed) » Needs work

Re-opening this as this really marks an API change.

damienmckenna’s picture

Status: Needs work » Needs review
StatusFileSize
new4.19 KB

I think this should do it - the minimum supported API version stays at 1.12 but the API itself has been bumped to 1.13 to support this new attribute. I've also documented in several places that this creates invalid XHTML and that the theme must be customized.

damienmckenna’s picture

Status: Needs review » Fixed

I committed this.

Status: Fixed » Closed (fixed)

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