spec for toggleAttribute could be a bit clearer

[bzbarsky]

The current spec does this:

4. If attribute is null, then:

   1. If force is not given or is true, create an attribute whose local name is qualifiedName, value is the empty string, and node document is the context object’s node document, then append this attribute to the context object, and then return true.
   2. Return false.

Note the need for a bunch of "then" in the first nested case. This might be easier to read as:

4. If attribute is null, then:

   1. If force is given and is false, return false.

   2. Create an attribute whose local name is qualifiedName, value is the empty string, and node document is the context object’s node document.

   3. Append this attribute to the context object.

   4. Return true.

or so. Similar for the case when attribute is not null.

[jonathanKingston]

I can likely tackle this at the same time of the following bug: whatwg/dom#657.

Thanks for the feedback!

[jonathanKingston]

@bzbarsky does whatwg/dom#663 solve this?

[bzbarsky]

It's a step in the right direction, but it still has the run-on "then" clauses instead of just breaking out its steps into a numbered list.

[jonathanKingston]

Ah sorry I missed that thanks, fixing now.

[jonathanKingston]

Updated and also moved out this attribute creation into it's own concept.