-
Notifications
You must be signed in to change notification settings - Fork 3
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
color could be HEX format instead #52
Comments
RGB triplets + A alpha transparency is used internally:
...yeah, we could add a rgb field to the IColor interface, which would take precedence over the triplet decimal values (if they exist). This rgb field would accept the old-school #AABBCC syntax, as a convenience to navigator API users .... just need to decide how to combine the variable opacity in the highlight code :) |
Not sure it can help inform the discussion that much, but it’s probably worth mentioning as mere “anecdata” that there’s been a major lobbying + move to I won’t list everything but a few recent examples I can find in 30 seconds:
There are significant drawbacks though:
|
HSL is sooo yesterday though! ;o) LCH is the future :) |
Note: a conversion function is available to transform a r2-navigator-js/src/electron/common/highlight.ts Lines 34 to 45 in 7da454e
|
color could be HEX format instead
avoids an another structure and weak impacts on the user dev.
it's an idea :)
r2-navigator-js/src/electron/common/highlight.ts
Line 10 in 29582f1
The text was updated successfully, but these errors were encountered: