Conversation
Part of our test suite, but not actually part of the spec yet; see whatwg/html#7320
wooorm
left a comment
There was a problem hiding this comment.
LGTM but weird to merge something that isn’t in HTML.
Is there a way for the tests to only test what is merged?
If not, should such a “feature-flag” be proposed there?
|
It is a bit odd that this was already added to |
Hello @fb55, I'm the software architect who implemented the Nevertheless, as I see those details won't affect your PR. |
|
@Kaleidea Thanks a lot for the context! |
|
The search element is now part of the HTML spec, so let's get this merged! |
Not sure if this PR should be merged at this time.
The
<search>element is tested inhtml5lib-tests, but the PR for the spec hasn't been merged yet (whatwg/html#7320). It seems like there is consensus around this spec change (see whatwg/html#7581), but we might hold off on this until the spec has actually been updated.