Skip to content

Commit 0b1ca07

Browse files
annevkdizhang168
authored andcommitted
Editorial: initialize clearTargets in event dispatch correctly
Also clean up "for each" wording while here. Fixes #1344.
1 parent 4f13f9b commit 0b1ca07

File tree

1 file changed

+17
-15
lines changed

1 file changed

+17
-15
lines changed

dom.bs

Lines changed: 17 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ first <a>following</a> <a for=tree>sibling</a> or null if it has no <a for=tree>
148148

149149
<li><p>Let <var>tokens</var> be a new <a>ordered set</a>.
150150

151-
<li><p><a for=list>For each</a> <var>token</var> in <var>inputTokens</var>, <a for=set>append</a>
151+
<li><p><a for=list>For each</a> <var>token</var> of <var>inputTokens</var>, <a for=set>append</a>
152152
<var>token</var> to <var>tokens</var>.
153153

154154
<li>Return <var>tokens</var>.
@@ -896,7 +896,7 @@ correct defaults.</p>
896896
<a>relative high resolution coarse time</a> given <var>time</var> and <var>event</var>'s
897897
<a>relevant global object</a>.
898898

899-
<li><p><a for=map>For each</a> <var>member</var><var>value</var> in <var>dictionary</var>, if
899+
<li><p><a for=map>For each</a> <var>member</var><var>value</var> of <var>dictionary</var>, if
900900
<var>event</var> has an attribute whose <a spec=webidl>identifier</a> is <var>member</var>, then
901901
initialize that attribute to <var>value</var>.
902902

@@ -1307,6 +1307,8 @@ property of the event being dispatched.
13071307
<li><p>Let <var>relatedTarget</var> be the result of <a>retargeting</a> <var>event</var>'s
13081308
<a for=Event>relatedTarget</a> against <var>target</var>.
13091309

1310+
<li><p>Let <var>clearTargets</var> be false.
1311+
13101312
<li>
13111313
<p>If <var>target</var> is not <var>relatedTarget</var> or <var>target</var> is <var>event</var>'s
13121314
<a for=Event>relatedTarget</a>:
@@ -1407,18 +1409,17 @@ property of the event being dispatched.
14071409
<li><p>Let <var>clearTargetsStruct</var> be the last struct in <var>event</var>'s
14081410
<a for=Event>path</a> whose <a for=Event/path>shadow-adjusted target</a> is non-null.
14091411

1410-
<li><p>Let <var>clearTargets</var> be true if <var>clearTargetsStruct</var>'s
1411-
<a for=Event/path>shadow-adjusted target</a>, <var>clearTargetsStruct</var>'s
1412-
<a for=Event/path>relatedTarget</a>, or an {{EventTarget}} object in
1413-
<var>clearTargetsStruct</var>'s <a for=Event/path>touch target list</a> is a <a for=/>node</a>
1414-
and its <a for=tree>root</a> is a <a for=/>shadow root</a>; otherwise false.
1412+
<li><p>If <var>clearTargetsStruct</var>'s <a for=Event/path>shadow-adjusted target</a>,
1413+
<var>clearTargetsStruct</var>'s <a for=Event/path>relatedTarget</a>, or an {{EventTarget}} object
1414+
in <var>clearTargetsStruct</var>'s <a for=Event/path>touch target list</a> is a <a for=/>node</a>
1415+
whose <a for=tree>root</a> is a <a for=/>shadow root</a>: set <var>clearTargets</var> to true.
14151416

14161417
<li><p>If <var>activationTarget</var> is non-null and <var>activationTarget</var> has
14171418
<a for=EventTarget>legacy-pre-activation behavior</a>, then run <var>activationTarget</var>'s
14181419
<a for=EventTarget>legacy-pre-activation behavior</a>.
14191420

14201421
<li>
1421-
<p><a for=list>For each</a> <var>struct</var> in <var>event</var>'s <a for=Event>path</a>, in
1422+
<p><a for=list>For each</a> <var>struct</var> of <var>event</var>'s <a for=Event>path</a>, in
14221423
reverse order:
14231424

14241425
<ol>
@@ -1433,7 +1434,7 @@ property of the event being dispatched.
14331434
</ol>
14341435

14351436
<li>
1436-
<p><a for=list>For each</a> <var>struct</var> in <var>event</var>'s <a for=Event>path</a>:
1437+
<p><a for=list>For each</a> <var>struct</var> of <var>event</var>'s <a for=Event>path</a>:
14371438

14381439
<ol>
14391440
<li><p>If <var>struct</var>'s <a for=Event/path>shadow-adjusted target</a> is non-null, then
@@ -1593,7 +1594,7 @@ and an optional <var>legacyOutputDidListenersThrowFlag</var>, run these steps:
15931594
<li><p>Let <var>found</var> be false.
15941595

15951596
<li>
1596-
<p><a for=list>For each</a> <var>listener</var> in <var>listeners</var>, whose
1597+
<p><a for=list>For each</a> <var>listener</var> of <var>listeners</var>, whose
15971598
<a for="event listener">removed</a> is false:
15981599

15991600
<ol>
@@ -4812,7 +4813,7 @@ steps are:
48124813

48134814
<ol>
48144815
<li>
4815-
<p><a for=list>For each</a> <var>attr</var> in <var>node2</var>'s
4816+
<p><a for=list>For each</a> <var>attr</var> of <var>node2</var>'s
48164817
<a for=Element>attribute list</a>:
48174818

48184819
<ol>
@@ -7381,7 +7382,7 @@ steps:
73817382
<li>
73827383
<p>If this {{NamedNodeMap}} object's <a for=NamedNodeMap>element</a> is in the
73837384
<a>HTML namespace</a> and its <a for=Node>node document</a> is an <a>HTML document</a>, then
7384-
<a for=list>for each</a> <var>name</var> in <var>names</var>:
7385+
<a for=list>for each</a> <var>name</var> of <var>names</var>:
73857386

73867387
<ol>
73877388
<li><p>Let <var>lowercaseName</var> be <var>name</var>, in <a>ASCII lowercase</a>.
@@ -10080,7 +10081,7 @@ method steps are:
1008010081

1008110082
<ol>
1008210083
<li>
10083-
<p><a for=list>For each</a> <var>token</var> in <var>tokens</var>:
10084+
<p><a for=list>For each</a> <var>token</var> of <var>tokens</var>:
1008410085

1008510086
<ol>
1008610087
<li><p>If <var>token</var> is the empty string, then <a>throw</a> a "{{SyntaxError!!exception}}"
@@ -10090,7 +10091,7 @@ method steps are:
1009010091
"{{InvalidCharacterError!!exception}}" {{DOMException}}.
1009110092
</ol>
1009210093

10093-
<li><p><a for=list>For each</a> <var>token</var> in <var>tokens</var>, <a for=set>append</a>
10094+
<li><p><a for=list>For each</a> <var>token</var> of <var>tokens</var>, <a for=set>append</a>
1009410095
<var>token</var> to <a>this</a>'s <a>token set</a>.
1009510096

1009610097
<li><p>Run the <a>update steps</a>.
@@ -10102,7 +10103,7 @@ method steps are:
1010210103

1010310104
<ol>
1010410105
<li>
10105-
<p><a for=list>For each</a> <var>token</var> in <var>tokens</var>:
10106+
<p><a for=list>For each</a> <var>token</var> of <var>tokens</var>:
1010610107

1010710108
<ol>
1010810109
<li><p>If <var>token</var> is the empty string, then <a>throw</a> a "{{SyntaxError!!exception}}"
@@ -10580,6 +10581,7 @@ Sergey G. Grekhov,
1058010581
Shiki Okasaka,
1058110582
Shinya Kawanaka,
1058210583
Simon Pieters,
10584+
Simon Wülker,
1058310585
Stef Busking,
1058410586
Steve Byrne,
1058510587
Stig Halvorsen,

0 commit comments

Comments
 (0)