Re: [whatwg/dom] Add declarative Shadow DOM features (#892)

@rniwa commented on this pull request.



> @@ -4185,8 +4185,8 @@ elements. SVG ought to do the same for its <{script}> elements, but does not cal
 at the moment.
 
 <p>To <dfn export id=concept-node-clone lt="clone a node" local-lt="clone">clone</dfn> a
-<var>node</var>, with an optional <var>document</var> and <i>clone children flag</i>, run these
-steps:
+<var>node</var>, with an optional <var>document</var>, <i>clone children flag</i>, and
+<i>clone shadows flag</i>, run these steps:

I don't think we should allow cloning of shadow trees like this.
Instead, we should keep nested declarative shadow DOM as templates until the algorithm runs to turn it into a shadow tree. This algorithm can then recursively process inner declarative shadow DOMs.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/dom/pull/892#pullrequestreview-1122990822
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/dom/pull/892/review/1122990822@github.com>

Received on Wednesday, 28 September 2022 04:18:18 UTC