| 11:33 | <foolip> | annevk smaug I got a debrief from Noam Rosenthal yesterday about https://github.com/whatwg/html/issues/11982#issuecomment-3614418258 but just want to confirm my understanding before writing any comments. The only difference between contentmethod=append and contentmethod=replace-children is that the latter removes children in one step. Is the hope that append isn't needed because it could be done with marker nodes instead? Is the main benefit having just one method (and thus a boolean attribute) or is append in particular troublesome? |
| 13:13 | <foolip> | I wrote https://github.com/whatwg/html/issues/11542#issuecomment-3616886398 |
| 16:09 | <annevk> | foolip: I think my hope is having one method and not needing the attribute at all (as we already have other attributes). It also feels a little weird to have "replace" and "append", but not the others. Need to think about it some more. |
| 16:16 | <smaug> | foolip: my comment about append was mostly that it is very much unclear in the explainer that what it does. |
| 16:16 | <smaug> | Not sure replace-children is very clear either |
| 16:18 | <smaug> | "replace-children is the basic one that allows replacing a placeholder with its contents". 'placeholder' there might refer to the element which has contentname, or it might refer to the content of the element which has contentname. And 'its' refers somehow to placeholder in that sentence |
| 16:19 | <smaug> | Anyhow, I should probably read some pr |
| 19:57 | <foolip> | @smaug:mozilla.org: I see, I honestly haven't looked at the explainer in a while, should give it a refresh to align with decisions made in the PR. |
| 19:58 | <foolip> | @smaug:mozilla.org: in short, the target element is the one with the contentname attribute. append appends nodes to the target element. replace-children does too, but first removes any existing nodes. |