Skip to content

Commit 5841a0f

Browse files
Updating the disposition of comments from the LC1 period. All comments have
been addressed. Added section on commentor's response to the editor's response.
1 parent 1cafa68 commit 5841a0f

File tree

1 file changed

+21
-6
lines changed

1 file changed

+21
-6
lines changed

LC1_comments.htm

Lines changed: 21 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,8 @@ <h2>DOM Parsing and Serialization</h2>
2424
<tr>
2525
<th>Comment Number</th>
2626
<th>Comment</th>
27-
<th>Response</th>
27+
<th>Editor's Response</th>
28+
<th>Commentor's Response</th>
2829

2930
<tr>
3031
<td>1
@@ -39,8 +40,11 @@ <h2>DOM Parsing and Serialization</h2>
3940
DOM4</a> the CDATASection interface and internalSubset property has been removed.
4041
<p><a href="http://lists.w3.org/Archives/Public/public-webapps/2013OctDec/0710.html">
4142
Rebuttal against making CDATASection and internalSubset optional</a>.
43+
<td data-accepted>
44+
Accepted. Change was made to remove CDATASection and internalSubset serialization steps from the algorithm.
45+
[<a href="https://dvcs.w3.org/hg/innerhtml/rev/1920f7b853a1">Diff</a>]
4246
<td>
43-
<p>No response yet.
47+
TBD
4448
<tr>
4549
<td>2
4650
<td>
@@ -52,14 +56,20 @@ <h2>DOM Parsing and Serialization</h2>
5256
<td data-accepted>
5357
Accepted. Change was reverted: <a href="http://lists.w3.org/Archives/Public/public-webapps/2013OctDec/0826.html">
5458
http://lists.w3.org/Archives/Public/public-webapps/2013OctDec/0826.html</a>
59+
[<a href="https://dvcs.w3.org/hg/innerhtml/rev/9425d4d28248">Diff</a>]
60+
<td>
61+
TBD
5562
<tr>
5663
<td>3
5764
<td>
5865
<p>Reported by Victor Costan
5966
<p><a href="https://www.w3.org/Bugs/Public/show_bug.cgi?id=24174">https://www.w3.org/Bugs/Public/show_bug.cgi?id=24174</a>
6067
<p>The &lt; (LESS THAN) character should be escaped in the XML attribute serialization algorithm.
68+
<td data-accepted>
69+
Accepted. Change was made to add the appropriate escaping.
70+
[<a href="https://dvcs.w3.org/hg/innerhtml/rev/bfa43ce953aa">Diff</a>]
6171
<td>
62-
<p>No response yet.
72+
<a href="https://www.w3.org/Bugs/Public/show_bug.cgi?id=24174#c5">Approved</a>
6373
<tr>
6474
<td>4
6575
<td>
@@ -68,10 +78,15 @@ <h2>DOM Parsing and Serialization</h2>
6878
<p>The setAttributeNS allows the creation of attributes that have a non-null namespace URI, but
6979
no prefix. This case does not seem to be handled by the current algorithm in the spec, but should
7080
be.
81+
<td data-accepted>
82+
Accepted. Fixing this required a substantial update to the algorithm, but the update now includes proper
83+
handling of attributes in non-null namespaces including automatic prefix generation if necessary (which
84+
also applies to elements in certain situations).
85+
[<a href="https://dvcs.w3.org/hg/innerhtml/rev/fb9edcfb8f5f">First draft diff</a>,
86+
<a href="https://dvcs.w3.org/hg/innerhtml/rev/f9b5a818ef99">Followup draft diff</a>,
87+
<a href="https://dvcs.w3.org/hg/innerhtml/rev/fa768c710fba">Polish pass diff</a>]
7188
<td>
72-
<p>No response yet.
73-
74-
89+
TBD
7590
</table>
7691

7792
</body>

0 commit comments

Comments
 (0)