-
Notifications
You must be signed in to change notification settings - Fork 3.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[html] Remove a testcase to check case-insensitivity of radio button … #4374
Conversation
…group name The testcase doesn't match to the current specification, and radio-groupname-case.html covers this feature.
Notifying @Ms2ger, @ayg, @gsnedders, @jdm, @jgraham, @sideshowbarker, @zcorpan, and @zqzhang. (Learn how reviewing works.) |
@domenic @TakayoshiKochi , would you review this please? |
FirefoxTesting revision 9181e70 All results/html/semantics/forms/the-input-element/radio.html
|
ChromeTesting revision 9181e70 All results/html/semantics/forms/the-input-element/radio.html
|
Can you change it so radio3 and radio4 both have the name radio-groupname-case.html doesn't test that two controls with the same name with non-ASCII characters are in the same group, as far as I can tell. |
@zcorpan thank you for the comment. New commit addressed it. |
Thanks! |
…group name
The testcase doesn't match to the current specification, and
radio-groupname-case.html covers this feature.