#gRegorLovePerhaps because the simplest case is to cite just the text, sans markup. The example given there is "short text notes"
#tantekI suppose the question there is whether a repost should be using h-cite at all
#tanteksince a repost is intended as *post* of the original content
#tantekkylewm: that is, asking for e-content is solving the wrong problem IMO - it may be expedient, but it's not actually helping the markup be correct
#kylewmI've gone through a couple iterations of my own markup with reposts actually
#kylewmright now i have an u-repost-of h-cite INSIDE the e-content of the post itself
#kylewmthat way you can include the permalink of the original post as well as the url of your repost
#tantekthat's e-content *inside* h-cite, not e-content *on* h-cite
KevinMarks joined the channel
#glennjonesKevinMarks: Could you have a quick look at the 4 issues aginst your test changes https://github.com/microformats/tests/issues - they are all to do with when to trim whitespaces. I am happy to make corrections just want known if you think I am right or not
#Loqiglennjones: KevinMarks left you a message 6 days ago: said text was OK as a single string, but kyle's issue of pointing to the same URL twice may suggest we need an array. Are there in the wild exampels fo linking to the same URL wiht multiple types? I think that is contrived, but linking to the same URL with multiple texts is quite plausible
#KevinMarksreading now - I thought trimming whitespace was expected in general, but I may have over-interpreted
#glennjonesAs far as I can see its only mention one on the parsing page in the “implied name rules” ie “drop leading & trailing white-space from…”
#tantekwould be preferable since I don't even know of any h-review-aggregate posts yet
#glennjonesI have add it as an option to my parser just in case we move to all property text at some stage, but at moment I am trying to get the test to match current rules
#glennjonesThe test are a real mixture of old real world examples, a few taken from wiki examples and a hand full made up to help parser authors
#glennjonesHappy to see them replaced over time with indieweb patterns of use, but I need to get the current set in working order first
KartikPrabhu joined the channel
#tantekglennjones - get the current set in working order first makes sense
#glennjonesKevinMarks: you OK with me updating your changes? I known you must of put a lot of time into the pull request.
#KevinMarksIs there any pushback from anyone on leading/trailing stripping for p- properties?
#KevinMarks'cos I'd rather make that 1 line change to the parsing spec at least
#tantekKevinMarks: proposal in the wiki that you and I +1 is " keep as is but have mf2 parser trim leading/trailing whitespace" which does not limit to p-* properties
#kylewmis there ever any such thing like "first name: <span class="p-name">Kyle </span> last name: <span class="p-name">Mahan</span>"
#glennjonesI think I did across all properties just to safe guard against me adding leading/trailing single spaces by mistake. Not sure I work out all the possible impacts, but it was how my parser has work for last two years with any noticalbe problem
#tantekall the same arguments apply - in terms of having to put markup on separate lines from the thing being marked up
#KevinMarksright, the <pre> case in e-content would be inside it
#kylewm<pre> -- that's what i meant rather than e-content
#KevinMarksI mean I get missing unicode with lxml and not with html.parser or html5lib
#KevinMarksso i wonder if I have some bad lxml version
#KevinMarksno, lxml is doing it on purpose: "In Python 2, lxml's API returns byte strings for plain ASCII text values, be it for tag names or text in Element content."
#tantekin general if there are problems with using html5lib we really should work on resolving them rather than avoiding using html5lib
#tantekKevinMarks: interesting that they're looking at such structured information - that's perhaps a good chance to get them to look at adding microformats2 support
#tantekdo we know anyone there that would be good to ask about that?