-
Notifications
You must be signed in to change notification settings - Fork 39
Only convert classic microformats classnames if there aren’t microformats2 classnames #45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
In particular, as noted in http://microformats.org/wiki/microformats2-parsing#parse_an_element_for_properties :
In particular note that only when parsing a backcompat root, should any backcompat properties be parsed. |
Discussion today about only implying properties on mf2 roots, no backcompat roots: https://indiewebcamp.com/irc/2016-05-18#t1463596806212 I can take a stab at some of this. |
This just got bumped up in my priority list. Since I added hReview markup to my review posts to get them to show up as Google Rich Snippets, the mf2 parsed version of the posts are a total mess. Take a look at this example: It's kind of a giant mess of duplicated objects and values appearing in bizarre places. |
I am for this even if some parsers have an override. |
In theory this had already been fixed as of #32, but it appears there are some residual cases causing problems.
Example: http://notizblog.org/2014/02/20/wir-brauchen-metadaten-fuer-telefonnummern/
Somewhat reduced case:
The text was updated successfully, but these errors were encountered: