#microformats 2017-01-25

2017-01-25 UTC
#
tantek
that thread!
KartikPrabhu, nitot, KevinMarks and KevinMarks_ joined the channel
KevinMarks, tantek, nitot, KevinMarks_, adactio, TallTed, gRegorLove, KartikPrabhu and hober2 joined the channel
#
aaronpk
is it just me or is php-mf2's backcompat for hReview totally busted
nitot joined the channel
#
aaronpk
"// doesn’t work properly, see spec"
#
KevinMarks
We did talk about generalising the mapping that the python one uses, as it's a json structure
#
gRegorLove
aaronpk: A lot of that should be fixed in what I'm working on.
#
gRegorLove
I'll try to get it wrapped up and PR sent tonight
#
gRegorLove
Oh, though I hadn't caught that comment in the mf1 => mf2 mapping. I'll check against the spec :)
#
Loqi
h-review
#
gRegorLove
I suspect that comment goes back to previous attempts to fix backcompat parsing in general
KevinMarks, KartikPrabhu and nitot joined the channel
#
gRegorLove
Still has issues. Hm
#
gRegorLove
Updated it to parse https://aaronparecki.com/2016/12/15/16/dropvox, which seems better
KevinMarks_ and nitot joined the channel
#
gRegorLove
Man. The more I dig into backcompat, the harder it gets.
#
tantek
gRegorLove: I'd like to understand why that is
#
tantek
I tried to spec backcompat in a very straightforward / simple way
#
gRegorLove
Spec isn't confusing. Translating it into code is.
#
gRegorLove
In large part because there's existing code I didn't write, which needs to be improved to follow the spec.
#
gRegorLove
e.g. it's not correct to just replace 'fn' with 'p-item h-item p-name' in an hreview, because then it messes up the reviewer.fn property.
#
gRegorLove
I've made some good progress on following the backcompat spec, but this example just exposes another flaw I hadn't noticed yet.
#
aaronpk
does a pretty bad job on mf1 hReview until I get a new mf2 parser in there
#
KevinMarks_
The python code has a set of mappings for each mf1 format, rather than general ones
#
KevinMarks_
It warns that it doesn't fully work too
nitot and davidmead joined the channel