- From: Mason Freed <notifications@github.com>
- Date: Wed, 18 Jan 2023 16:28:44 -0800
- To: whatwg/dom <dom@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Thursday, 19 January 2023 00:28:56 UTC
> I put your build fixes in #1143 and rebased this. I also pushed a fixup commit to correct a number of references to shadow root, adopt some modern practices and address a nit. Improving "allow declarative shadow roots" is left still. Thanks. As for making "allow declarative shadow roots" non-tri-state, the problem I wanted to avoid was finding all of the places where a `create-the-document-object` is used, and instead defer the decision to the parser. That's how I implemented it in code, and it felt quite a bit simpler than always initializing the document correctly. Perhaps @rniwa can weigh in on how he implemented this in WebKit? > > Adding `<div algorithm>` also uncovered an error. "attach a shadow root" didn't account for the addition of "slot assignment". It's probably best if you address that? To make sure it's good for all callers. Thanks - fixed. -- Reply to this email directly or view it on GitHub: https://github.com/whatwg/dom/pull/892#issuecomment-1396281006 You are receiving this because you are subscribed to this thread. Message ID: <whatwg/dom/pull/892/c1396281006@github.com>
Received on Thursday, 19 January 2023 00:28:56 UTC