"GWicke" posted a comment on MediaWiki.r111439. URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/111439#c31198
Commit summary for MediaWiki.r111439: Introduce property: and itemprop: support for addMeta to add RDFa <meta property="..." content="..."> and Microdata <meta itemprop="..." content="..."> to the <head>. This is done in a way that can also be feature tested, and technically could be expanded by extensions. GWicke's comment: For WebPage, that definitely makes more sense. The only trouble I see with the global scope is that stray itemprops in the content would end up in the outermost itemscope. But that should be fixable in the parser. I am still not that happy with the need to use an extra hook to avoid producing invalid microdata. Some of the WebPage schema data would be naturally added in the skin (mainContentOfPage for example), so it might make sense to add this itemtype and itemscope on the html element in the skin as well. Header itemprops from other itemtypes could still be added using absolute URLs in itemprop names. If we allow users to replace the itemtype on the html element arbitrarily, then we could not use something like WebPage to mark up parts of the page in the skin- at least not without using absolute URLs throughout. Afaik right now search engine crawlers don't yet use itemprops with absolute URLs, but I guess that might change (hopefully also in the HTML spec) once mixing multiple itemtypes becomes more common. Until then, we have to decide if something like WebPage or an arbitrary user itemtype should take precedence. I would vote for a static itemtype like WebPage, as it covers most of the metadata already published with pages. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
