Skip to content

Commit

Permalink
Editorial: a removed node's root is itself
Browse files Browse the repository at this point in the history 
Also address some issues Bikeshed found (incorrect linking and the use of "optional" in a non-normative context).

Fixes #691.
  • Loading branch information
@annevk
annevk authored Sep 11, 2018
1 parent 28cdb47 commit 42d2485
Showing 1 changed file with 19 additions and 18 deletions.
37 changes: 19 additions & 18 deletions dom.bs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -495,7 +495,7 @@ the empty list.</p>
<dt><code><var>event</var> = new <a constructor lt="Event()">Event</a>(<var>type</var> [, <var>eventInitDict</var>])</code>
<dd>Returns a new <var>event</var> whose
{{Event/type}} attribute value is set to
<var>type</var>. The optional <var>eventInitDict</var> argument
<var>type</var>. The <var>eventInitDict</var> argument
allows for setting the {{Event/bubbles}} and
{{Event/cancelable}} attributes via object
members of the same name.
Expand Down Expand Up @@ -793,7 +793,7 @@ dictionary CustomEventInit : EventInit {
<dl class=domintro>
<dt><code><var>event</var> = new <a constructor lt="CustomEvent()">CustomEvent</a>(<var>type</var> [, <var>eventInitDict</var>])</code>
<dd>Works analogously to the constructor for {{Event}} except
that the optional <var>eventInitDict</var> argument now
that the <var>eventInitDict</var> argument now
allows for setting the {{CustomEvent/detail}} attribute
too.

Expand Down Expand Up @@ -2646,7 +2646,7 @@ with an optional <i>suppress observers flag</i>, run these steps:
<ol>
<li><p>Run <a>assign slotables for a tree</a> with <var>parent</var>'s <a for=tree>root</a>.

<li><p>Run <a>assign slotables for a tree</a> with <var>node</var>'s <a for=tree>root</a>.
<li><p>Run <a>assign slotables for a tree</a> with <var>node</var>.
</ol>

<li><p>Run the <a>removing steps</a> with <var>node</var> and <var>parent</var>.
Expand Down Expand Up @@ -9071,7 +9071,7 @@ must return the <a>context object</a>'s <a for=traversal>filter</a>.
<p>If <var>beforeNode</var> is false, then set it to true.
</dl>

<li><p>Let <var>result</var> be the result of <a for=Node>filtering</a> <var>node</var> within
<li><p>Let <var>result</var> be the result of <a for=/>filtering</a> <var>node</var> within
<var>iterator</var>.

<li><p>If <var>result</var> is {{NodeFilter/FILTER_ACCEPT}}, then <a for=iteration>break</a>.
Expand Down Expand Up @@ -9154,7 +9154,7 @@ these steps:
<ol>
<li><p>Set <var>node</var> to <var>node</var>'s <a for=tree>parent</a>.

<li><p>If <var>node</var> is non-null and <a for=Node>filtering</a> <var>node</var> within the
<li><p>If <var>node</var> is non-null and <a for=/>filtering</a> <var>node</var> within the
<a>context object</a> returns {{NodeFilter/FILTER_ACCEPT}}, then set the <a>context object</a>'s
<a for=TreeWalker>current</a> to <var>node</var> and return <var>node</var>.
</ol>
Expand All @@ -9175,7 +9175,7 @@ these steps:
<p>While <var>node</var> is non-null:

<ol>
<li><p>Let <var>result</var> be the result of <a for=Node>filtering</a> <var>node</var> within
<li><p>Let <var>result</var> be the result of <a for=/>filtering</a> <var>node</var> within
<var>walker</var>.

<li><p>If <var>result</var> is {{NodeFilter/FILTER_ACCEPT}}, then set <var>walker</var>'s
Expand Down Expand Up @@ -9243,7 +9243,7 @@ and <var>type</var>, run these steps:
<ol>
<li><p>Set <var>node</var> to <var>sibling</var>.

<li><p>Let <var>result</var> be the result of <a for=Node>filtering</a> <var>node</var> within
<li><p>Let <var>result</var> be the result of <a for=/>filtering</a> <var>node</var> within
<var>walker</var>.

<li><p>If <var>result</var> is {{NodeFilter/FILTER_ACCEPT}}, then set <var>walker</var>'s
Expand All @@ -9263,8 +9263,8 @@ and <var>type</var>, run these steps:
<li><p>If <var>node</var> is null or <var>walker</var>'s <a for=traversal>root</a>, then return
null.

<li><p>If the return value of <a for=Node>filtering</a> <var>node</var> within <var>walker</var>
is {{NodeFilter/FILTER_ACCEPT}}, then return null.
<li><p>If the return value of <a for=/>filtering</a> <var>node</var> within <var>walker</var> is
{{NodeFilter/FILTER_ACCEPT}}, then return null.
</ol>
</ol>

Expand Down Expand Up @@ -9292,8 +9292,8 @@ these steps:
<ol>
<li><p>Set <var>node</var> to <var>sibling</var>.

<li><p>Let <var>result</var> be the result of <a for=Node>filtering</a> <var>node</var> within
the <a>context object</a>.
<li><p>Let <var>result</var> be the result of <a for=/>filtering</a> <var>node</var> within the
<a>context object</a>.

<li>
<p>While <var>result</var> is not {{NodeFilter/FILTER_REJECT}} and <var>node</var> has a
Expand All @@ -9302,8 +9302,8 @@ these steps:
<ol>
<li><p>Set <var>node</var> to <var>node</var>'s <a for=tree>last child</a>.

<li><p>Set <var>result</var> to the result of <a for=Node>filtering</a> <var>node</var>
within the <a>context object</a>.
<li><p>Set <var>result</var> to the result of <a for=/>filtering</a> <var>node</var> within
the <a>context object</a>.
</ol>

<li><p>If <var>result</var> is {{NodeFilter/FILTER_ACCEPT}}, then set the
Expand All @@ -9318,7 +9318,7 @@ these steps:

<li><p>Set <var>node</var> to <var>node</var>'s <a for=tree>parent</a>.

<li><p>If the return value of <a for=Node>filtering</a> <var>node</var> within the
<li><p>If the return value of <a for=/>filtering</a> <var>node</var> within the
<a>context object</a> is {{NodeFilter/FILTER_ACCEPT}}, then set the <a>context object</a>'s
<a for=TreeWalker>current</a> to <var>node</var> and return <var>node</var>.
</ol>
Expand All @@ -9345,8 +9345,8 @@ steps:
<ol>
<li><p>Set <var>node</var> to its <a for=tree>first child</a>.

<li><p>Set <var>result</var> to the result of <a for=Node>filtering</a> <var>node</var> within
the <a>context object</a>.
<li><p>Set <var>result</var> to the result of <a for=/>filtering</a> <var>node</var> within the
<a>context object</a>.

<li><p>If <var>result</var> is {{NodeFilter/FILTER_ACCEPT}}, then set the
<a>context object</a>'s <a for=TreeWalker>current</a> to <var>node</var> and return
Expand All @@ -9371,8 +9371,8 @@ steps:
<li><p>Set <var>temporary</var> to <var>temporary</var>'s <a for=tree>parent</a>.
</ol>

<li><p>Set <var>result</var> to the result of <a for=Node>filtering</a> <var>node</var> within
the <a>context object</a>.
<li><p>Set <var>result</var> to the result of <a for=/>filtering</a> <var>node</var> within the
<a>context object</a>.

<li><p>If <var>result</var> is {{NodeFilter/FILTER_ACCEPT}}, then set the <a>context object</a>'s
<a for=TreeWalker>current</a> to <var>node</var> and return <var>node</var>.
Expand Down Expand Up @@ -9993,6 +9993,7 @@ Ondřej Žára,
Peter Sharpe,
Philip Jägenstedt,
Philippe Le Hégaret,
Pierre-Marie Dartus,
Ra'Shaun Stovall (Snuggs),
Rafael Weinstein,
Richard Bradshaw,
Expand Down

0 comments on commit 42d2485

Please sign in to comment.