Inaccurate readme about better-color-tools HOT 3 CLOSED

It should accept an object; these tests even sure that’s still working right now.

I believe maybe there’s just confusion around the types—those are output types, not input types. In your example, if you called better.from(…).oklabVal you would indeed get a [number, number, number, number] type ([0.4856949, -0.03971, -0.14459, 1]) which matches (note: worth noting that this library will “snap” values to the closest sRGB color, so it’s intentionally lossy, in case you are ever returned a different value it just calculated the nearest-displayable color since Oklab’s range is boundless).

However, if you then tried and take that 4-length array and pass it back into better.from(), that’s where it would err because it’s just an array—it has no clue what colorspace that is. And so better.from() only accepts sRGB arrays as input, hence the object format.

Unless I’m misunderstanding I don’t think there’s an issue with the README? Perhaps just the exported types aren’t what you expected?

from better-color-tools.

It should accept an object;
better.from() only accepts sRGB arrays as input

There is definitely a misunderstanding somewhere because i see these sentences as contradictory.

I believe maybe there’s just confusion around the types—those are output types, not input types

I am pretty sure those are input types and ColorOutput is the output type.

// parse.ts
export function from(rawColor: Color): ColorOutput {
  // ...
}

I don't know if this actually works at runtime but type checker is not happy with me providing it an object.

from better-color-tools.

Thanks for clarifying. I was able to push a better update to the types, and also converted the tests to TypeScript to prevent this from happening again. Please try the latest version and let me know if you have any additional problems 🙏

from better-color-tools.