User tests: Successful: Unsuccessful:
This is PR #1841 updated with the current staging branch to assist with testing. I make no assurances on the code's functionality; all I've done is merged up the current code base and resolved any merge conflicts that came up.
Labels |
Added:
?
?
|
IMO we should avoid to create more useless configuration params. Microdata should be on by default. If you don't like it you can override the templates.
Same for everything related to enable microdata (com_contact config parameter and so).
Tried the patch and it works well on article. Get excited and tried on contact page, but it is not working. Was it thought just for articles?
@chiaraaliotta according to the files changed then yes it should apply to contacts as well
I think I found a bug: the feature is not obeying the configuration on the contact item level.
Steps to reproduce the bug:
1) Set the global configuration to enable microdata
2) Set the microdata option on the contact component level to global
3) Set the microdata option on the specific contact item to global as well
In this case the microdata is not showing. However, if you explicitly enable the option on the item level, it works.
So this is where the global configuration is not obeying on the item level.
@chiaraaliotta Could you please test this PR #3330 , it's easier to add one piece at the time, thanks.
Closing this PR as it's against an old branch and we merged a different approach for now.
Alex will do a new PR during his GSoC project.
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-06-04 21:08:13 |
See JoomlaCode Tracker: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=31841&start=0