Skip to content

Hardcoded Microdata for com_contact#3359

Merged
mbabker merged 3 commits intojoomla:3.3-devfrom
Bakual:MicroDataContact
Apr 2, 2014
Merged

Hardcoded Microdata for com_contact#3359
mbabker merged 3 commits intojoomla:3.3-devfrom
Bakual:MicroDataContact

Conversation

@Bakual
Copy link
Copy Markdown
Contributor

@Bakual Bakual commented Mar 23, 2014

This PR introduces Microdata tags to com_content in a hardcoded way.
This is based on some feedback that we don't need more parameters.

The PR is based on PR #3252, but unlike this one it doesn't use the Microdata library and instead just hardcodes the tags into the layouts.

Advantage

  • Easy to understand
  • Easy to change for endusers and designers who like to override a layout

Disadvantages

  • You can't enable/disable Microdata tags globally or per contact. You can of course use layout overrides and alternate layouts to achieve the same. It's also to note that there isn't really a reason to turn the Microdata off.
  • You can't select a different Microdata type for a contact using a parameter. It's hardcoded to be "Person". You can of course use layout overrides and alternate layouts here as well to achieve the same, but you need to make sure yourself that the properties are valid for the selected type.

Testing

Test the different views of com_contact and check the microdata output using the Google richsnippet test tool: http://www.google.com/webmasters/tools/richsnippets

@Bakual
Copy link
Copy Markdown
Contributor Author

Bakual commented Mar 23, 2014

@betweenbrain
Copy link
Copy Markdown
Contributor

👍

@Bakual Bakual deleted the MicroDataContact branch April 2, 2014 06:21
Bakual pushed a commit to Bakual/joomla-cms that referenced this pull request May 12, 2014
Hardcoded Microdata for com_contact
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants