| 14:01 | <Ms2ger> | annevk, https://github.com/heycam/webidl/pull/652 is ready now, I think |
| 14:03 | <annevk> | Ms2ger: r+ |
| 14:03 | <nox> | annevk: Still no answer from anyone from Chrome or Edge :( |
| 14:06 | <annevk> | nox: I don't care much for Edge |
| 14:06 | <nox> | annevk: I pinged tkent again. |
| 14:06 | <annevk> | nox: perhaps there should be a summary comment or some such? What do we expect someone new coming into https://github.com/whatwg/html/pull/4354 to look at? |
| 14:06 | <nox> | annevk: Everything. |
| 14:07 | <nox> | I mean, I pinged them in https://github.com/whatwg/html/pull/4354#issuecomment-462792225, they can start from that. |
| 14:08 | <annevk> | nox: I suspect that if I reread that thread there'll be lots that's no longer relevant |
| 14:09 | <nox> | I'll link to important comments, I guess. Too bad no one chimed in 13 days ago when the thread was fresh. |
| 14:10 | <annevk> | nox: given all the developments that might have been better for them 😃 |
| 14:10 | <nox> | annevk: What do you mean? |
| 14:11 | <annevk> | nox: it's less to work to catch up with something after the exploring has been done |
| 14:12 | <nox> | I'm just pissed that a Google-sized company can't reply in time on a ticket when other people from smaller companies did. |
| 15:00 | <hsivonen> | annevk: fwiw, it looks like at least a third of Firefox 66 beta (beta population not necessarily reprsentative of release users, very short time window, etc.) Text Encoding menu invocations are on labeled pages, i.e. ones that Chrome and EdgeHTML neither use heuristics for nor offer menu override |
| 15:00 | <hsivonen> | (not sure this means anything in particular) |
| 15:51 | <annevk> | Domenic: any chance you can ping someone internally for https://github.com/whatwg/html/pull/4354 to help nox out? |
| 15:51 | <nox> | <3 |
| 15:52 | <Domenic> | Sure |
| 15:52 | <annevk> | hsivonen: that means that 2/3 are for the heuristics-requiring pages? |
| 15:52 | <Domenic> | nox: annevk: is there any summary, not necessarily of everything that's gone on, but of what problem or interop issue this is even solving? |
| 15:53 | <hsivonen> | annevk: some of it is also re-overriding already overridden, but yes |
| 15:53 | <nox> | Domenic: Basically every Live DOM Viewer link from that thread. |
| 15:54 | <nox> | I can't even write docs for my own code, let alone summary comments. |
| 15:54 | <Domenic> | nox: If I'm going to get folks involved, I'll need a bit more than that :-/ |
| 15:54 | <nox> | Domenic: https://github.com/whatwg/html/pull/4354#issuecomment-462813946 |
| 15:54 | <nox> | Domenic: https://github.com/whatwg/html/pull/4354#issuecomment-463693419 |
| 15:54 | <annevk> | Domenic: it's attempting to properly define when we execute scripts upon insertion and various related matters |
| 15:54 | <annevk> | Domenic: in particular the spec assumes a single queue, but implementations have two |
| 15:54 | <Domenic> | OK, maybe annevk could write the summary comment? :) |
| 15:55 | <nox> | Do you just ignore PRs without summary comments? That seems pretty bad to me. |
| 15:57 | <Domenic> | Yes, pretty much, if someone doesn't say what problem they're solving and just proposes a bunch of changes, it's not really a quality contribution. |
| 15:57 | <annevk> | Domenic: that's the summary you could pass along when requesting input, the PR is analysis of impls |
| 15:57 | <Domenic> | Like, I could change a ton of things in Servo with no commit message, and I'm sure that would not be welcomed. |
| 15:57 | <nox> | Domenic: We would definitely chime in the PR thread. |
| 15:58 | <nox> | I don't think Googlers are any more busy than people at Mozilla who did chime in, but ok. |
| 16:10 | <Ms2ger> | Domenic, is there any spec that says to treat WindowProxy objects as Windows? |
| 16:11 | <Domenic> | I don't remember reading any spec, but I remember a long argument where Hixie said "look, my spec just says to treat them the same, it's fine"... Let me check, but my current guess is that we used to have something hand-wavey, and in the course of making WindowProxy rigorous we lost that particular piece of hand-waving. |
| 16:14 | <Domenic> | https://www.w3.org/Bugs/Public/show_bug.cgi?id=27128 is what I'm remembering |
| 16:14 | <Domenic> | > I don't understand what there is to define. It's just an object that forwards all operations to the underlying object. Or, as the spec puts it, "all operations that would be performed on it must be performed on the Window object of the browsing context's active document instead". |
| 16:15 | <Domenic> | BTW we should probably mass-move from Bugzilla soon (for both Web IDL and HTML) since it's shutting down |
| 16:29 | <Ms2ger> | Oh right, I thought I mentioned that to MikeSmith, but I'm not sure if he was planning to do it |
| 16:42 | <MikeSmith> | I am not planning to do it |
| 16:42 | <MikeSmith> | I’m happy to if someone tells me that’s what we actually want |
| 16:43 | <MikeSmith> | but the bugs we still have open there are ones we so far have intentionally not moved |
| 16:44 | <MikeSmith> | we moved many others but not those |
| 16:44 | <gsnedders> | MikeSmith: do we have any way to at least save them somehow if we want to move later? |
| 16:44 | <Domenic> | It does seem like a hand-curated pass would work better. E.g. closing all the HTML feature requests that haven't bubbled up in the years since GitHub. |
| 16:44 | <MikeSmith> | the bugs will remain there as far as I understand |
| 16:44 | <MikeSmith> | the URLs will continue to work |
| 16:45 | <MikeSmith> | for the individual bugs |
| 16:45 | <MikeSmith> | though the search feature and other features will quit working |
| 16:48 | <Domenic> | Yeah, the search which lets us filter by component is kind of important, so ideally we should move before then. |
| 16:48 | <Domenic> | Although I guess we could re-build a filtered list from a .csv export or something |
| 16:50 | <MikeSmith> | yeah but if we’re gonna do something it would be best to do it before they flip the switch |
| 17:23 | <annevk> | Oooh, platform brands are defined now |
| 19:27 | <TimothyGu> | Domenic: is the plan to not use Web IDL for built-in modules like kv-storage? |
| 19:27 | <Domenic> | TimothyGu: it was originally but I changed my mind. https://github.com/WICG/kv-storage/issues/46 |
| 19:30 | <TimothyGu> | Domenic: Ah got it. |
| 19:34 | <TabAtkins> | Domenic: nice |
| 19:35 | <TabAtkins> | I knew that "by-hand crap" would convince you eventually ^_^ |
| 20:42 | <domfarolino> | Domenic: ahh, thanks, feels good to finally get that merged. (oh and thanks for your commit too!) |
| 20:42 | <Domenic> | :) |