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

[css-text-decor-3] text-underline-position usage in Default UA Stylesheet doesn't match its syntax #3441

Closed
upsuper opened this issue Dec 13, 2018 · 1 comment

Comments

@upsuper
Copy link
Member

upsuper commented Dec 13, 2018

The syntax of text-underline-position is auto | [ under || [ left | right ] ] which means auto should never be combined with other values.

However, in Appendix B: Default UA Stylesheet, we have

:root:lang(ja), [lang|=ja],
:root:lang(mn), [lang|=mn],
:root:lang(ko), [lang|=ko] { text-underline-position: auto right; }
:root:lang(zh), [lang|=zh] { text-underline-position: auto left;  }

which doesn't match the syntax above.

One of them should be updated.

@fantasai
Copy link
Collaborator

OK, fixed the examples to match the normative syntax, since that's what Chrome has implemented, and either way it'll be valid. Let me know if you think we should adjust the syntax definition also.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment