Conversation
From https://rocallahan.github.io/innerText-spec/ with no normative changes except adding [CEReactions] to the IDL. Fixes #465.
zcorpan
added
addition/proposal
| <code>select</code>, and <code>video</code> — but not <code>button</code>) are not rendered | ||
| by CSS, strictly speaking, and therefore have no CSS boxes for the purposes of this algorithm.</p> | ||
|
|
||
| <p class="big-issue">This algorithm is amenable to being generalized to work on a <span |
There was a problem hiding this comment.
"a ranges" -> "a range" | "ranges"
| these steps:</p> | ||
|
|
||
| <ol> | ||
| <li><p>If the element is not <span>being rendered</span>, return the same value as the |
There was a problem hiding this comment.
@domenic and I started using "this element" (or equivalent) in other algorithms. I think that would be a clearer way to refer to the object on which the IDL attribute is defined.
| steps:</p> | ||
|
|
||
| <ol> | ||
| <li><p>Let <var>document</var> be the given element's <span>node document</span>.</p></li> |
|
This looks okay. The only thing I'd consider changing further is making the recursion less declarative. Have that be some algorithm that is invoked for each child and also put the result in a variable of some kind the rest of the algorithm uses. Will let @domenic make the call on that. |
|
Yeah, it could use some more cleanup. Possibly also switch to iterative traversal instead of recursive? But I think the current state is OK to merge and I can fix more later. |
| </div> | ||
|
|
||
|
|
||
| <h4>The <code data-x="dom-innerText">innerText</code> IDL attribute</h4> |
There was a problem hiding this comment.
I wonder if a better title might be something like Manipulating an element's "as rendered" text? It seems to me like other section titles usually don't talk about a specific IDL attribute. I don't feel strongly though.
There was a problem hiding this comment.
It's a bit of a mix currently. Having "innerText" in the heading makes it easier to find. But if we also add outerText we can maybe change the heading at that point.
4 participants
From https://rocallahan.github.io/innerText-spec/ with no normative
changes except adding [CEReactions] to the IDL.
Fixes #465.
@Ms2ger @rocallahan