Skip to content
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

Added Color::LAB and Color::XYZ classes #11

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

stiff
Copy link

@stiff stiff commented Jun 22, 2014

Sorry guys I missed the existing #8 pull request, but anyway think some bits maybe useful.

This should be backward-compatible: previously to_xyz/to_lab returned hash now its LAB and XYZ classes, but they support accessing individual components by square brackets, like hash: rgb.to_xyz[:x]

@halostatue
Copy link
Owner

I'd be happier if you work with @daveheitzman to figure out which bits work well together. I'm a little swamped right now, so I'm not having a lot of time to spend on the design for color 2.0, which is when these classes will be introduced.

@halostatue halostatue mentioned this pull request Jul 17, 2014
@halostatue
Copy link
Owner

I am interested in this, but would prefer if you could target this to the changes being made either in color-2 directly or preferably the CIE-Lab-class branch referenced in #12.

@halostatue halostatue added this to the 2.0 milestone Jul 17, 2014
@stiff
Copy link
Author

stiff commented Jul 19, 2014

Done. Take a look please

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.52%) when pulling f8d375c on stiff:master into 3a214a6 on halostatue:master.

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

Successfully merging this pull request may close these issues.

4 participants