fix(color-contrast): allow disabled label children

Closes #596
dequelabs · Dec 12, 2017 · 6fcaf93 · 6fcaf93
1 parent af0050f
commit 6fcaf93
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/lib/rules/color-contrast-matches.js b/lib/rules/color-contrast-matches.js
@@ -45,7 +45,7 @@ if (nodeName === 'LABEL' || nodeParentLabel) {
 		return false;
 	}
 
-	var candidate = node.querySelector('input:not([type="hidden"]):not([type="image"])' +
+	var candidate = relevantNode.querySelector('input:not([type="hidden"]):not([type="image"])' +
 		':not([type="button"]):not([type="submit"]):not([type="reset"]), select, textarea');
 	if (candidate && candidate.disabled) {
 		return false;

diff --git a/test/rule-matches/color-contrast-matches.js b/test/rule-matches/color-contrast-matches.js
@@ -130,7 +130,17 @@ describe('color-contrast-matches', function () {
 		fixture.innerHTML = '<input type="text" disabled>';
 		var target = fixture.querySelector('input');
 		assert.isFalse(rule.matches(target));
+	});
 
+	it('should not match a disabled implicit label child', function () {
+		fixture.innerHTML = '<label>' +
+			'<input type="checkbox" style="position: absolute;display: inline-block;width: 1.5rem;height: 1.5rem;opacity: 0;" disabled checked>' +
+			'<span style="background-color:rgba(0, 0, 0, 0.26);display:inline-block;width: 1.5rem;height: 1.5rem;" aria-hidden="true"></span>' +
+			'<span style="color:rgba(0, 0, 0, 0.38);" id="target">Baseball</span>' +
+			'</label>';
+		var target = fixture.querySelector('#target');
+		var result = rule.matches(target);
+		assert.isFalse(result);
 	});
 
 	it('should not match <textarea disabled>', function () {