Skip to content

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

Closed
mysteryDate opened this issue Oct 13, 2022 · 11 comments
Closed

Oklab captialization #7880

mysteryDate opened this issue Oct 13, 2022 · 11 comments

Comments

@mysteryDate
Copy link

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?

@svgeesus
Copy link
Contributor

You are right.

@svgeesus
Copy link
Contributor

Now we need a consistent capitalization for OKLCH, where Björn Ottosson
does not give any guidance. I guess Oklch?

@svgeesus
Copy link
Contributor

In the absence of other opinions, going for Oklch.

@LeaVerou
Copy link
Member

"Oklch" looks really wrong. Could we please reconsider, especially since this was something we only went with in the absence of other opinions?

  • There is no precedent for referring to LCH as lch. LCH is typically capitalized, or only the h is lowercase: https://en.wikipedia.org/wiki/HCL_color_space
  • LCH being an initialism would point towards uppercase, while it's ok to lowercase the "ab" in "Lab" since that’s what they are actually called.
  • "OKLCH" seems far more widespread

No opinion about OKLCH vs OKLCh vs OkLCH, but strong -1 on Oklch.

@svgeesus
Copy link
Contributor

svgeesus commented Nov 30, 2024

Björn uses OKLCh in passing, while defining Okhsv and Okhsl. Note that his OKLCh uses Lr rather than L, too.

@LeaVerou
Copy link
Member

I’m fine with OKLCh. @astearns I wonder if we could get an async resolution on this?

@LeaVerou LeaVerou reopened this Nov 30, 2024
@astearns
Copy link
Member

astearns commented Dec 3, 2024

@LeaVerou I am assuming this is just for the specification. The functions and keywords are case-insensitive in code, correct?

@svgeesus
Copy link
Contributor

svgeesus commented Dec 4, 2024

Yes it is just a spec consistency issue with no impact on implementations

@astearns
Copy link
Member

astearns commented Dec 4, 2024

OK, then I think there is no resolution required. This can be just at the editor’s discretion.

@Crissov
Copy link
Contributor

Crissov commented Dec 5, 2024

So, now it’s “OKLCh” but still “Oklab”, not “OKLab”!?

@svgeesus
Copy link
Contributor

svgeesus commented Dec 5, 2024

So, now it’s “OKLCh” but still “Oklab”, not “OKLab”!?

Correct. Both using the original author preferred spelling.

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

No branches or pull requests

5 participants