#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)