New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Editorial: cleanup activeElement #3513
Conversation
Correct usage of let/set and stop referring to it as a method.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good but there's more opportunities for cleanup if you want.
source
Outdated
@@ -73500,36 +73500,32 @@ END:VCARD</pre> | |||
<code>Document</code> objects must return the value returned by the following steps:</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe uniformize the prelude too? I think these days we're using "must, on getting, run these steps:". (Well, OK, searching for "on getting", we're not very uniform at all. But something like that...)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought it was "The activeElement attribute's getter, when invoked, must run these steps:"...
source
Outdated
|
||
<li><p>If <var>candidate</var> is a <span>focusable area</span>, let <var>candidate</var> be <var>candidate</var>'s <span>DOM anchor</span>.</p></li> | ||
<li><p>If <var>candidate</var> is a <span>focusable area</span>, then set <var>candidate</var> to | ||
<var>candidate</var>'s <span>DOM anchor</span>.</p></li> | ||
|
||
<li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This may be cleaner as a set of three nested steps with "If candidate is a Document object:" as the parent step.
Markup error causing build failure at the moment. |
Correct usage of let/set and stop referring to it as a method.
/browsers.html ( diff )
/interaction.html ( diff )