#microformats 2018-07-06
2018-07-06 UTC
[cleverdevil], [manton], tantek__, [miklb], [snarfed], [aaronpk], cheim and barpthewire joined the channel
#
@Sebsel ↩️ Oeps, ik ben dus overgestapt van de Twitter-reader naar een Indie-reader, en nu krijg ik dus dit soort notificaties niet meer. Tijd om de e-mails weer aan te zetten. (twitter.com/_/status/1015137420981960704)
#
@Sebsel ↩️ Maar inhoudelijk: het lijkt me inderdaad wat Martijn al zei: de Microformats van Ton's blog zijn wel aanwezig, maar niet goed aangebracht, waardoor Frank's weergave van slag is. Dat vind ik zelf het zwakste punt van de hele flow: dat Microformats zo kwetsbaar zijn in CMS'en. (twitter.com/_/status/1015138444941590528)
#
@frankmeeuwsen ↩️ Kunnen webmentions opnieuw worden verzonden als microformats eenmaal op orde zijn? Zoals Ton naderhand heeft gedaan. Zie ook mijn vragen op http://diggingthedigital.com//Waar-te-beginnen-met-Webmentions/ (twitter.com/_/status/1015140467510513664)
#
@ton_zylstra ↩️ als ik het goed begrijp wel (via handmatig form iig), ook voor verwijdering stuur je het nog een keer. De ontvanger checkt dan de gegevens en past aan. (twitter.com/_/status/1015150592916967424)
#
@ton_zylstra ↩️ De vraag is wat ik daar zelf aan kan sleutelen. Die microformats worden door Webmention en Semantic Backlinks plugins in WP aangebracht. (twitter.com/_/status/1015151050175799301)
#
@frankmeeuwsen ↩️ WP plugins zijn veelal open source dus je zou zelf wat in de PHP kunnen wroeten (succes...). Wellicht is via hooks in functions.php nog wat te tweaken? (twitter.com/_/status/1015153778285268992)
#
@ton_zylstra ↩️ ongetwijfeld maar de eerstvolgende plugin of thema update overschrijft dat waarschijnlijk. (twitter.com/_/status/1015157622595518464)
#
@ton_zylstra ↩️ al ben ik op zich prima bereid de code in te duiken. vanochtend zitten speuren nav eem handmatige trackback van jou. blog ik zo wel even. (twitter.com/_/status/1015160546923110400)
adactio, [jgmac1106], [eddie] and [jgarber] joined the channel
#
sknebel reading through https://www.zylstra.org/blog/2018/07/wrapping-my-head-around-webmentions/ and looking at the example where pages are shown as "u-url" news.indieweb.org: IMPLIED U-URL parsing...
#
[jgarber] Today I learned! https://en.wikipedia.org/wiki/Wikipedia:Chesterton%27s_fence

nitot and [cjwillcock] joined the channel
#
[cjwillcock] sknebel++ ^^^
tantek__, [manton], [cleverdevil], jackjamieson, [eddie] and [dgold] joined the channel
#
gRegorLove Backs up the suggestion to restrict implied p-name to any other property, not just p-*

#
gRegorLove "and no properties" appears to apply to implied name, url, and photo based on that page.

[schmarty] and [cleverdevil] joined the channel
#
gRegorLove Was this the img src alt update, or was it newer? http://microformats.org/wiki/microformats2-parsing-issues#img_fallback_in_p-

#
gRegorLove Ah, #17. nvm

sketchess joined the channel
#
gRegorLove That reasoning sounds good to me

jackjamieson joined the channel
#
sknebel which is why I'd like to know if there is a specific reasoning behind not doing so, or if people might have missed the context - since it's really new it's unlikely anyone is relying on it, and it's different from removing a long-standing rule, so I'd advocate for removing the line entirely. bit unlucky I didn't have time to call into the IWS session :/
#
gRegorLove I lean towards removing the line entirely since it's such a young change

#
gRegorLove I didn't understand the context of #17 at IWS and I don't think others did; it was just a quick, conservative update.

#
gRegorLove Ohh, yeah I forgot Kartik has this in a PR already

#
@neauoire Does Firefox still support microformats?
Has it been replaced with something else? (twitter.com/_/status/1015329222854770688)
KartikPrabhu joined the channel
KartikPrabhu and cheim joined the channel
#
tantek__ aaronpk, can you link to the ruby microformats parser bug you're referring to here? https://github.com/tootsuite/mastodon/issues/7926#issuecomment-402888936

#
KartikPrabhu everyone not in the channel raise your hands!

#
gRegorLove I think if we don't have any objections, and KartikPrabhu and tantek__ agree with the above reasoning, we'll have consensus

#
KartikPrabhu lol

#
gRegorLove Or other implementers :)

[eddie] joined the channel
#
gRegorLove In the change control process we're at "Iterate to resolve objections if any"

#
KartikPrabhu wait I thought implied name didn't have src replacement only alt replacement

#
KartikPrabhu hmm where the lastest link to this discussion?

#
gRegorLove But also see today's chat logs with sknebel and I digging into it. We believe it was a mistake to add that to implied p-name parsing in #17

#
KartikPrabhu mistake as in never substiture images at all?

#
gRegorLove No, scope of the conversation is implied p-name

#
KartikPrabhu yeah I am in the same scope

#
KartikPrabhu so no image subsitution for implied name?

#
Loqi [gRegorLove] Blame me: https://github.com/microformats/microformats2-parsing/issues/17#issuecomment-357462463 :/

#
KartikPrabhu yeah I saw that. But is the final resolution to never replace imgs with anything or replace with alt but not src (for implied name)

#
gRegorLove implied p-name final rule would become: "else return the textContent of the .h-x after: 1) dropping any nested <script> & <style> elements; 2) removing all leading/trailing whitespace."

#
gRegorLove textContent will get the alt, afaik

#
KartikPrabhu not sure. textContent rules are not defined anywhere in the wiki

#
gRegorLove Oops, I misspoke about textContent. Earlier rule will get the alt.

#
KartikPrabhu yeah already implemented in mf2py latest

#
KartikPrabhu unreleased (waiting for review from kevinmarks)

#
gRegorLove step 3, `.h-x>img:only-child[alt]:not([alt=""]):not[.h-*]`

#
KartikPrabhu eh! now I am lost

#
gRegorLove The final fallback rule would do nothing special with <img>. No replacement.

#
KartikPrabhu I thought we are talking about the final step

#
gRegorLove (if this counter-proposal is accepted)

#
gRegorLove Hah, sorry

#
KartikPrabhu I am so confusedn

#
gRegorLove KartikPrabhu, Check the implied p-name update here: what it was before and what was accepted: https://github.com/microformats/microformats2-parsing/issues/17#issuecomment-357462463

#
KartikPrabhu it isn't option 1 that is the whole proposal

#
gRegorLove sknebel and I are proposing dropping the second bullet "replacing any nested..."

#
KartikPrabhu gRegorLove: yeah, the point is I don't know what textContent is supposed to do with the image

#
gRegorLove What does it do currently?

#
KartikPrabhu you mean mf2py?

#
gRegorLove Yeah

#
KartikPrabhu I can make it do anything. Right now it replaces img with the alt but never with the src (for implied name)

#
gRegorLove That gets into #15 as well

#
gRegorLove Which I haven't followed very closely

#
KartikPrabhu can we decide a fix for one issue first :P

#
gRegorLove I don't know if it's "right" or not, I'm just saying that I think falling back to the parser's current implementation of textContent is ok.

#
gRegorLove "else use the textContent..." has been in the spec for a while, so mf2py has done something to that effect previously

#
KartikPrabhu I just want to know what to do with <img>. replace with stuff or just drop it

#
sknebel http://microformats.org/wiki/index.php?title=microformats2-parsing&diff=prev&oldid=66660 (#17 initially only proposed to add the language about script/style tags, and then "accidentially" got expanded)
#
KartikPrabhu if you say use "textContent" I have no idea what that means

#
gRegorLove Where was Zegnat's textContent web thingy...

#
gRegorLove KartikPrabhu, What was mf2py doing previously? This spec change was only in January.

#
KartikPrabhu dropping all images

#
gRegorLove Ah, interesting.

#
gRegorLove checks what php-mf2 did / does

#
KartikPrabhu just like dropping <script> and <style> is specified, I'd be ok with any explicit specification of what to do with <img>

#
KartikPrabhu but saying textContent is meaningless for me

#
gRegorLove Well, that's #15 (as much as you don't want me to mention another issue ;)

#
gRegorLove Hehe

#
KartikPrabhu yeah ok then lets do 15 or something

#
KartikPrabhu but that is more whitespace oriented

#
gRegorLove Found Zegnat's proposed spec and tool: https://wiki.zegnat.net/media/textparsing.html

#
KartikPrabhu right

#
KartikPrabhu mf2py right now does <img> subsitution differently in differnt contexts

#
gRegorLove Kind of, but the spec still says "use textcontent"

#
KartikPrabhu right. subsituting <img> with src in implied name had strange bridgy effects

#
KartikPrabhu so now it is replace by alt but not src, only in implied name everywhere else it is replace by alt or src

#
KartikPrabhu yup

#
KartikPrabhu exactly my point!

#
KartikPrabhu yeah because mf2py spec does not say use textContent as defined in the DOM

#
KartikPrabhu it just say textCOntent

#
KartikPrabhu in that sense we don't need to say drop <style> and <script> either

#
KartikPrabhu DOM textContent implicitly drops <script> no?

#
KartikPrabhu hmm weird

#
gRegorLove Latest 0.4.4-alpha uses Zegnat's algorithm

#
gRegorLove Right

#
gRegorLove I haven't dived into the code of it, but the previous plaintext functions in php-mf2 did have a context of whether it was implied parsing or not, and handled them slightly differently

#
gRegorLove I presume it still does similar

#
KartikPrabhu instead of randomly deciding this we should ask aaronpk and snarfed since Xray and Bridgy are the biggest consumers

#
gRegorLove snarfed is already on record, definitely doesn't want src

#
gRegorLove and probably doesn't want alt

[jgmac1106] joined the channel
#
KartikPrabhu aaronpk: when parsing for impled name should <img> be replaced by 1) alt or src or 2) only alt never src 3) dropped completely

#
KartikPrabhu one sentence summary ^

#
KartikPrabhu yes. agreed what about "alt"

#
gRegorLove At IWS we agreed to #2 btw

#
KartikPrabhu I know which is why I added it to mf2py

#
gRegorLove Just context for aaronpk

#
KartikPrabhu ok good. so aaronpk is with #2, snarfed is ok with #2, I already implemented #2 ;)

#
KartikPrabhu aaronpk: yes. but more in-the-wild uses for mf2 parsing

#
KartikPrabhu aaronpk: or I am being a bit thick since had long meetings :P

#
gRegorLove It's 100 here today, so my brain might be frying a bit ;)

#
KartikPrabhu to conclude my take: replace <img> with 'alt' sounds fine for now

#
KartikPrabhu if some cases come up where this is not desired then we can change the spec accordingly instead of just anticipating it now

#
gRegorLove Say bridgy three times

snarfed joined the channel
#
KartikPrabhu oh hey!

#
KartikPrabhu snarfed: when parsing for impled name from textContent should <img> be replaced by 1) alt or src or 2) only alt never src 3) dropped completely

#
KartikPrabhu I know you don't like #1 but which of #2 or #3

#
KartikPrabhu of course. But from bridgy experience which one would be more suitable

#
KartikPrabhu cool. so you would prefer #3 drop img completely

#
KartikPrabhu :thumbsup:

#
KartikPrabhu no. just asking the biggest user of mf2py :)

#
KartikPrabhu cool. so gRegorLove sknebel maybe instead of resolving #15 now, we can resolve #35 with either option #2 (as suggested by Zegnat) or #3 drop img completely

#
gRegorLove #2 was agreed to at IWS. Today's discussion is basically: should we do #3?

#
KartikPrabhu yes

#
KartikPrabhu I can implement any of those

#
gRegorLove With any decision though, you still have the question "what does textContent mean", right?

#
KartikPrabhu yes. but that is a separate issue if the <img> replacement is explicitly specified

#
gRegorLove Oh, for #3 you want it to say "drop the image" before doing textContent. Ok

#
KartikPrabhu also what snarfed asked :P

#
gRegorLove snarfed, short version: a previous spec change in January added the img src alt text to implied p-name

#
gRegorLove Before that it was just "else use the textContent of the .h-x for name"

#
KartikPrabhu no #3 was also talked about

#
KartikPrabhu maybe we shouldn't worry too much about what was/wasn't done in January and update the spec according to recent discussion

#
KartikPrabhu cool. count mf2py as implemented

#
KartikPrabhu yeah, where's my cheque ;)

#
KartikPrabhu sknebel: yeah no worries. I'll in fact add you as reviewer

#
KartikPrabhu oh no! I can't officially add you on the repo. But please leave comments

#
KartikPrabhu owners on the microformats repo

#
KartikPrabhu also only "owners" can review it seems

#
gRegorLove Can someone capture on github #35 what the new proposal is?

#
KartikPrabhu gRegorLove: already there https://github.com/microformats/microformats2-parsing/issues/35#issuecomment-393615508

#
KartikPrabhu or your latest comment

#
Loqi [gRegorLove] Oops, I should have linked the video earlier. I've transcribed some of it below for easier reference. We discussed and agreed on the second proposal.
Topic starts https://youtu.be/0OUy7jjUN5k?t=2h1m20s, then at 2:02:23:
> Tantek: so we're just go...

#
gRegorLove Hah. Which one.

#
KartikPrabhu they are the same (?!)

#
gRegorLove Zegnat's has two proposals.

#
gRegorLove "No change" is what we're landing with? I'm just confirming.

#
gRegorLove No change from my last comment, I mean.

#
KartikPrabhu yes

#
KartikPrabhu hey where's my karma Loqi

#
KartikPrabhu gives Loqi cake

#
KartikPrabhu oh well

snarfed left the channel
#
gRegorLove KartikPrabhu++

#
gRegorLove sknebel++

#
gRegorLove cake++

#
gregorlove edited /microformats2-parsing (-135) "/* parsing for implied properties */ update implied p-name <img> processing per https://github.com/microformats/microformats2-parsing/issues/35" (view diff)