label:not([for]) is valid when has nested input field about holmes

[danielgwood]

It's a good point, that is valid, and currently marked not valid by Holmes. However, it can't be as simple as just adding :empty to the selector, because this makes a lot of invalid cases appear valid. Example:

<div>
    <h3>1. Valid</h3>
    <label for="foo">Label for foo</label>
    <input type="text" id="foo" name="foo" />
</div>

<div>
    <h3>2. Also valid</h3>
    <label>
        Label for bar
        <input type="text" name="bar" />
    </label>
</div>

<div>
    <h3>3. Not valid</h3>
    <label>Label "for" baz</label>
    <input type="text" name="baz" />
</div>

<div>
    <h3>4. Not valid</h3>
    <label for="">Label "for" baz</label>
    <input type="text" name="baz" />
</div>

According to the spec, cases 1 and 2 should be valid, 3 and 4 should not. Holmes currently only shows case 1 as valid. If you make the change you suggest, cases 1, 2 and 3 will all appear valid.

[danielgwood]

Here's the basic test case I worked up for this: http://danielgwood.com/lab/holmes/issue-20.html

[justmarkup]

The problem is that case 2 can not be detected via css, because there is no selector for "label that contains an input as children", at least as far as I know.

[danielgwood]

Indeed - I was unable to think of a CSS solution. I believe you're right, without a "has" or "contains" selector, you can't test case 2 without also hitting case 3.