Skip to content

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