Skip to content

/ servo

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

Jump to bottom

Attribute selector should treat values of some attributes as case-insensitive #15006

Closed
upsuper opened this issue Jan 13, 2017 · 3 comments
Closed

Attribute selector should treat values of some attributes as case-insensitive #15006

upsuper opened this issue Jan 13, 2017 · 3 comments
Labels
A-content/css
Milestone
Stylo: Style syst...

Comments

@upsuper
Copy link
Member

@upsuper upsuper commented Jan 13, 2017

See Case-sensitivity of selectors secion in the HTML spec, values of some attributes need to be treated as case-insensitive when applying attribute selector.

Servo currently doesn't match this requirement.
@Ms2ger
Copy link
Contributor

@Ms2ger Ms2ger commented Jan 13, 2017

Do we have tests?
@upsuper
Copy link
Member Author

@upsuper upsuper commented Jan 13, 2017

Gecko has a preliminary test for this: https://dxr.mozilla.org/mozilla-central/source/layout/style/test/test_bug357614.html

But I think it's worth writing a more complete one in wpt.
@upsuper upsuper mentioned this issue Feb 6, 2017
Closed
@upsuper
Copy link
Member Author

@upsuper upsuper commented May 24, 2017

It seems to have been fixed by #16915.
@upsuper upsuper closed this May 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-content/css
Projects
None yet
Milestone
Stylo: Style syst...
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@Ms2ger @upsuper
You can’t perform that action at this time.