Add tests for element/attribute name selectors in HTML #13738
Conversation
wpt-pr-bot
requested review from
annevk,
domenic,
foolip,
jdm,
jgraham,
lilles,
zcorpan and
zqzhang
html/semantics/selectors/case-sensitivity-of-selectors/name-selectors.html
Outdated
Show resolved
Hide resolved
gsnedders
force-pushed
the
html-selector-tests
branch
from
455f020
to
28cb054
Compare
| <!doctype html> | ||
| <title>Test whether namespace prefixes are lowercased</title> | ||
| <script src=/resources/testharness.js></script> | ||
| <script src=/resources/testharnessreport.js></script> |
There was a problem hiding this comment.
Doesn't this test really belong in css/css-namespaces[1], and isn't this already covered by css/css-namespaces/prefix-001.xml ?
There was a problem hiding this comment.
This is testing:
When comparing a CSS element type selector to the names of HTML elements in HTML documents, the CSS element type selector must first be converted to ASCII lowercase. The same selector when compared to other elements must be compared according to its original case. In both cases, the comparison is case-sensitive.
Specifically because of whatwg/html#4155, and whether the whole CSS Qualified Name is lowercased or whether it's just the local part.
Therefore it's different to css/css-namespaces/prefix-001.xml because it's an HTML element in an HTML document.
Let me add a comment to clarify this.
| <meta charset=utf8> | ||
| <title>CSS element selector case-sensitivity</title> | ||
| <script src=/resources/testharness.js></script> | ||
| <script src=/resources/testharnessreport.js></script> |
There was a problem hiding this comment.
Could you add a <link rel=help> with the relevant spec-link?
There was a problem hiding this comment.
if you don't know, for HTML you can always take the last path segment as a fragment (i.e., #case-sensitivity-of-selectors).
| <meta charset=utf8> | ||
| <title>CSS attribute name selector case-sensitivity</title> | ||
| <script src=/resources/testharness.js></script> | ||
| <script src=/resources/testharnessreport.js></script> |
There was a problem hiding this comment.
Could you add a <link rel=help> with the relevant spec-link?
Inspired by whatwg/html#3372. This tests the current spec.