Re: [whatwg/dom] Editorial: create a concept for event's target (#621)

miketaylr commented on this pull request.

These changes LGTM with one small typo.

Should we update the following from 2.1?:

> The event ’s target attribute value returns the object to which the event was dispatched ( obj above). 

The other mention of target below that seems easier to understand in the context of the code example as-is:

> Each time the event ’s target attribute value will be the span element .


>  
-The <dfn attribute for=Event><code>target</code></dfn> and
-<dfn attribute for=Event><code>currentTarget</code></dfn>
-attributes must return the values they were initialized to. When an
-<a>event</a> is created the attributes must be
-initialized to null.
+<p>The <dfn attribute for=Event><code>target</code></dfn> attribute's getter must return the
+<a>context object</a>'s <a for=Event>target</a>.
+
+<p>The <dfn attribute for=Event><code>currentTarget</code></dfn> attribute must return the values it

nit: s/values/value/

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/dom/pull/621#pullrequestreview-108199895

Received on Thursday, 29 March 2018 21:14:01 UTC