- From: Anne van Kesteren <notifications@github.com>
- Date: Mon, 24 Apr 2023 07:32:38 -0700
- To: whatwg/dom <dom@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Monday, 24 April 2023 14:32:43 UTC
@annevk commented on this pull request. > <li><a for=list>Remove</a> <var>attribute</var> from <var>attribute</var>'s <a for=Attr>element</a>'s <a for=Element>attribute list</a>. <li><p>Set <var>attribute</var>'s <a for=Attr>element</a> to null. + + <li><p><a>Handle attribute changes</a> for <var>attribute</var> with <var>attribute</var>'s + <a for=Attr>element</a>, <var>attribute</var>'s <a for=Attr>value</a>, and null. Presumably we need to cache attribute's element beforehand? Also, it's surprising that for element removal it runs before, but for attribute removal it's after. Did you double check what implementations do here? -- Reply to this email directly or view it on GitHub: https://github.com/whatwg/dom/pull/1176#pullrequestreview-1398152380 You are receiving this because you are subscribed to this thread. Message ID: <whatwg/dom/pull/1176/review/1398152380@github.com>
Received on Monday, 24 April 2023 14:32:43 UTC