-
Notifications
You must be signed in to change notification settings - Fork 709
Oklab captialization #7880
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
Comments
You are right. |
Now we need a consistent capitalization for OKLCH, where Björn Ottosson |
In the absence of other opinions, going for Oklch. |
"Oklch" looks really wrong. Could we please reconsider, especially since this was something we only went with in the absence of other opinions?
No opinion about OKLCH vs OKLCh vs OkLCH, but strong -1 on Oklch. |
Björn uses OKLCh in passing, while defining Okhsv and Okhsl. Note that his OKLCh uses Lr rather than L, too. |
I’m fine with OKLCh. @astearns I wonder if we could get an async resolution on this? |
@LeaVerou I am assuming this is just for the specification. The functions and keywords are case-insensitive in code, correct? |
Yes it is just a spec consistency issue with no impact on implementations |
OK, then I think there is no resolution required. This can be just at the editor’s discretion. |
So, now it’s “OKLCh” but still “Oklab”, not “OKLab”!? |
Correct. Both using the original author preferred spelling. |
In tremendous nit news: In the spec, the "Oklab" color space is capitalized as "OKLab"
https://csswg.sesse.net/css-color-4/#specifying-oklab-oklch
In Ottosson blog that the spec references, the color space is always capitalized as "Oklab":
https://bottosson.github.io/posts/oklab/
Where should we be consistent about capitalization if not in spec documents?
The text was updated successfully, but these errors were encountered: