Skip to content

Native HCG support implementation #2921

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
wants to merge 2 commits into from
Closed

Native HCG support implementation #2921

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jul 4, 2016

I added support of HCG color model.

Plug-in official for HCG.
https://github.com/acterhd/less-hcg

@seven-phases-max
Copy link
Member

seven-phases-max commented Jul 4, 2016

It seems you've ignored #2803, did you? So technically it's not that I specifically would argue, but I actually wonder if this is worth being in the core instead of a plugin.

@@ -132,6 +132,28 @@ Color.prototype.toHSL = function () {
}
return { h: h * 360, s: s, l: l, a: a };
};
Color.prototype.toHCG = function () {
Copy link
Member

@seven-phases-max seven-phases-max Jul 4, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This function does not seem to be used anywhere so what is it its purpose?

Copy link
Author

@ghost ghost Jul 4, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For HCG planned modifiers (for example "purity", "absorbtion"). But I thinking about names, that will not conflict with HSL/HSV/HWB. Problem of HCG: they describes material property.

@seven-phases-max
Copy link
Member

seven-phases-max commented Jul 4, 2016

So I'm adding "needs decision" flag and referencing #1853 once again. While two functions are probably not something to bother arguing for/against with, further this leads to all that sort of proposals/extensions discussed in #1853, thus it's better to decide earlier (plugin? plugin?).

@stale
Copy link

stale bot commented Nov 14, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Nov 14, 2017
@stale stale bot closed this Nov 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants