appdotnet / api-spec

App.net API Documentation is on the web at https://developers.app.net. Source for these docs is in the new-docs branch here. Please use the issue tracker and submit pull requests! Help us build the real-time social service where users and developers come first, not advertisers.
https://developers.app.net
952 stars 99 forks source link

Post microdata doesn't follow spec #331

Open derelk opened 11 years ago

derelk commented 11 years ago

Microdata on posts is currently declared as: <span itemscope="https://app.net/schemas/Post">

The itemscope attribute is boolean and should have no value; the schema should be specified in the itemtype attribute: <span itemscope itemtype="https://app.net/schemas/Post">

http://schema.org/docs/gs.html#microdata_itemscope_itemtype http://www.w3.org/html/wg/drafts/microdata/master/Overview.html#typed-items

voidfiles commented 11 years ago

I wouldn't mind changing this. We would need to see if anyone is depending on this stuff. I am interested though is this breaking something for you?

derelk commented 11 years ago

The HTML fails W3C validation, and since the post HTML is embedded in Climber pages, it in turn causes our pages to fail validation as well. Not a big deal obviously, but conforming to standards makes me feel warm and fuzzy inside. I'm currently fixing it on our end with a regex anyway, but I figured you'd want to know.