| 00:06 | <rniwa> | MikeSmith: yt? |
| 00:34 | <MikeSmith> | botie, inform rniwa here now |
| 00:34 | <botie> | will do |
| 07:59 | <hsivonen> | github is supposed to be great for specs, yet https://w3c.github.io/webappsec-referrer-policy/ is a month out of date |
| 08:08 | <hsivonen> | annevk_: you wanted the DOM attribute to be referrerPolicy but the spec says referrerpolicy. are you OK with that or is there a spec bug? |
| 08:47 | <annevk> | hsivonen: that seems like a bug |
| 08:49 | <annevk> | hsivonen: does the specification even define IDL? |
| 08:53 | <annevk> | hsivonen: I filed https://github.com/w3c/webappsec-referrer-policy/issues/3 to keep track |
| 09:18 | <annevk> | Ms2ger: https://github.com/whatwg/html/pull/304 |
| 09:19 | <Ms2ger> | What's wrong with my commit messages? |
| 09:19 | <Ms2ger> | Also, no point in squashing, they're separate changes |
| 09:38 | <yoav> | Domenic/annevk: When talking about running a validation step in the value setter on https://github.com/whatwg/dom/pull/103#issuecomment-153641224, where you talking about https://dom.spec.whatwg.org/#dom-attr-value ? |
| 09:39 | <annevk> | yoav: no, I think he meant DOMSettableTokenList, if anything |
| 09:39 | <annevk> | yoav: though I'm not sure that's needed |
| 09:39 | <yoav> | OK, that makes more sense |
| 09:40 | <yoav> | annevk: leaving it out for now then |
| 09:44 | <Domenic> | I think it's needed... |
| 09:44 | <annevk> | Ms2ger: you can just force push to your Ms2ger:hyperlink branch |
| 09:44 | <Ms2ger> | Not worth my time |
| 09:44 | <annevk> | Domenic: why? that just forwards to setAttribute() today |
| 09:45 | <annevk> | Ms2ger: wow |
| 09:45 | <Domenic> | annevk: hmm, fair... |
| 09:45 | <Domenic> | wait, that was a real ragequit? |
| 09:45 | <annevk> | I guess so |
| 09:45 | <Domenic> | wow |
| 09:47 | <Domenic> | Sigh, Attr. Just when I reimplemented them in jsdom as being simple non-Nodes, too... |
| 09:47 | <annevk> | Ms2ger: does that mean you object to any of us merging the changes? |
| 09:49 | <jgraham> | FWIW I think being asine about things like the length of commit messages is a net negative |
| 09:50 | <jgraham> | It wastes the time of people who have put in the effort to make a change, but the benefits provided are super-marginal |
| 09:50 | <yoav> | Domenic: Were you referring to the DOMSettableTokenList's setter? |
| 09:51 | <Domenic> | yoav: yeah, but I guess annevk's point that it currently just forwards to setAttribute is convincing enough to leave as-is... |
| 09:51 | <annevk> | jgraham: yeah, don't disagree, I was happy to do it for Ms2ger |
| 09:51 | <annevk> | jgraham: not sure what to do now though |
| 09:51 | <yoav> | Domenic: OK |
| 09:51 | <Ms2ger> | The commits are still there, you can make whatever change you like when merging |
| 09:52 | <jgraham> | annevk: Pull his branch, create your own PR and merge that |
| 09:52 | <Ms2ger> | I just have no interest in spending any more time on it |
| 09:53 | <Domenic> | yes, in the future please just say "OK, can you do that for me please" instead of closing the PR... |
| 09:53 | jgraham | wonders what actual word he was trying to use |
| 09:56 | <Ms2ger> | That would sound like I think it's a good idea |
| 10:24 | <gsnedders> | jgraham: OTOH, I think commit messages like "update webplatform-tests" are a net negative |
| 10:24 | <gsnedders> | jgraham: there's some middle ground to be found here |
| 10:28 | <annevk> | Somewhat sad that https://github.com/whatwg/html/graphs/contributors is still broken |
| 10:38 | <annevk> | Domenic: btw, happy to have you here, but I'd rather you enjoy your vacation a bit more |
| 10:39 | <Domenic> | annevk: yeah, just took a couple hours downtime at the new hotel to catch up on things :). Going back out into Tokyo momentarily! |
| 10:39 | <jgraham> | gsnedders: I agree that sucky commit messages suck. I disagree that good commit messages that are 60 characters long are unacceptable |
| 10:40 | <jgraham> | In the case of wpt the fact that we really want direct contributions from browsers ofc affects the amount of leeway we are prepared to have in the style |
| 11:44 | <annevk> | wanderview: https://github.com/whatwg/fetch/pull/146 |
| 11:44 | <annevk> | mkwst: you might want to have a look too |
| 11:45 | <annevk> | mkwst: it's pretty easy to make a refactor that would enable code sharing between service worker responses and network responses |
| 12:24 | <hsivonen> | annevk: thanks |
| 13:01 | <annevk> | hsivonen: would you be okay with going from "Encoding Standard" to "Text Encoding Standard"? |
| 13:02 | <annevk> | I kinda like single-word titles, but I don't feel super strongly about it either |
| 13:13 | <hsivonen> | annevk: I think the current title is cooler, but I wouldn't oppose to the longer title. Where is the longer title coming from? Didn't even the W3C go to CR with the current title? |
| 13:18 | <philipj> | Domenic: a quick experiment to see if making Attr not a Node in Blink would actually allow any crufty code to be removed would be interesting |
| 13:18 | <philipj> | I kind of suspect it won't, it's like these objects inherit a lot of useles stuff that just doesn't matter, and ignoring those extra bits is ~0 lines of code |
| 13:55 | <jgraham> | hsivonen: dsinger (I think) was complaining that lots of/all data other than text also has an encoding (e.g. media files) |
| 15:03 | <annevk> | hsivonen: https://github.com/whatwg/encoding/issues/11 |
| 15:03 | <annevk> | hsivonen: I agree about the current title being cooler |
| 15:04 | <annevk> | philipj: yeah, I was thinking about the changes required to the DOM Standard and I don't think it amounts to much |
| 15:05 | <annevk> | philipj: there's a few extra places that have to consider Attr, but not many |
| 15:20 | <annevk> | wanderview: yeah, heard that before about HTML diff :-/ |
| 15:20 | <annevk> | wanderview: the other problem is spotting markup errors |
| 15:31 | <wanderview> | annevk: its really just a way of saying "don't blame me if I missed something" :-) |
| 17:26 | <annevk> | MikeSmith: https://www.w3.org/Bugs/Public/show_activity.cgi?id=26734 last change is spam |
| 17:26 | <annevk> | MikeSmith: do we have to undo that manually? |
| 18:44 | <jsx> | What does a "document environment" mean? A web page? |
| 19:23 | <wanderview> | jsx: this suggests it means a window or iframe: https://html.spec.whatwg.org/#document-environment |
| 19:24 | <jsx> | Thanks wanderview. |