Skip to content
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

radio-groupname-case.html still fails one caseless comparison test #7818

Open
jdm opened this issue Oct 1, 2015 · 8 comments
Open

radio-groupname-case.html still fails one caseless comparison test #7818

jdm opened this issue Oct 1, 2015 · 8 comments
Labels

Comments

@jdm
Copy link
Member

@jdm jdm commented Oct 1, 2015

https://github.com/servo/servo/pull/7777/files#diff-ed2c588bed7bbd35cf3ebc3cc09ded10R3

This points at an incomplete implementation in https://github.com/SimonSapin/rust-caseless, but we should figure out precisely what the failure case is in the testcase after #7777 merges.

cc @KiChjang in case you're interested in tackling this :)

@jdm jdm added the A-content/dom label Oct 1, 2015
@KiChjang
Copy link
Member

@KiChjang KiChjang commented Nov 21, 2015

I think I'll dive into this one once I get some of my other stuff done. In the meantime, if anyone is interested in this issue, go ahead ;)

@psdh
Copy link
Contributor

@psdh psdh commented Feb 11, 2016

This seems to fail on https://github.com/w3c/web-platform-tests/blob/master/html/semantics/forms/the-input-element/radio-groupname-case.html#L17 (on K to be specific). But at the same time, line number 34 doesn't fail.

@KiChjang
Copy link
Member

@KiChjang KiChjang commented Feb 11, 2016

For reference, K is the Kelvin sign used for measuring temperature.

@psdh
Copy link
Contributor

@psdh psdh commented Feb 12, 2016

@KiChjang Yep it is, the same kelvin sign matches correctly to 'k' (006b) in line 34. Not sure why that is happening

@psdh
Copy link
Contributor

@psdh psdh commented Feb 12, 2016

As SimonSapin pointed out, html spec says "no language-specific tailorings", and Unicode's CaseFolding.txt says about T "For non-Turkic languages, this mapping is normally not used."

@SimonSapin
Copy link
Member

@SimonSapin SimonSapin commented Feb 12, 2016

I think this is a test bug: web-platform-tests/wpt#2579

@annevk
Copy link

@annevk annevk commented Mar 14, 2017

FYI: HTML changed in whatwg/html#1941.

@nox
Copy link
Member

@nox nox commented Sep 30, 2017

This seems to still be relevant.

@jdm jdm removed the C-assigned label Sep 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
6 participants
You can’t perform that action at this time.