Update to use null and rework Okhsl saturation handling

src/spaces/okhsl.js

@@ -376,8 +376,8 @@ function okhslToOklab (hsl, lmsToRgb,  okCoeff) {
 
 	let [h, s, l] = hsl;
 	let L = toeInv(l);
-	let a = 0.0;
-	let b = 0.0;
+	let a = null;
+	let b = null;
 	h = constrain(h) / 360.0;
 
 	if (L !== 0.0 && L !== 1.0 && s !== 0) {
@@ -462,12 +462,22 @@ function oklabToOkhsl (lab, lmsToRgb, okCoeff) {
 		}
 	}
 
-	if (Math.abs(s) < εS || l === 0.0 || Math.abs(1 - l) < εL) {
-		h = NaN;
-		s = 0.0;
+	const achromatic = Math.abs(s) < εS;
+	if (achromatic || l === 0.0 || Math.abs(1 - l) < εL) {
+		h = null;
+		// Due to floating point imprecision near lightness of 1, we can end up
+		// with really high around white, this is to provide consistency as
+		// saturation can be really high for white due this imprecision.
+		if (!achromatic) {
+			s = 0.0;
+		}
+	}
+
+	else {
+		h = constrain(h * 360);
 	}
 
-	return [constrain(h * 360), s, l];
+	return [h, s, l];
 }
 
 

src/spaces/okhsv.js

@@ -47,8 +47,8 @@ function okhsvToOklab (hsv, lmsToRgb, okCoeff) {
 	h = constrain(h) / 360.0;
 
 	let l = toeInv(v);
-	let a = 0.0;
-	let b = 0.0;
+	let a = null;
+	let b = null;
 
 	// Avoid processing gray or colors with undefined hues
 	if (l !== 0.0 && s !== 0.0) {
@@ -137,11 +137,14 @@ function oklabToOkhsv (lab, lmsToRgb, okCoeff) {
 	}
 
 	if (Math.abs(s) < ε || v === 0.0) {
-		h = NaN;
-		s = 0.0;
+		h = null;
 	}
 
-	return [constrain(h * 360), s, v];
+	else {
+		h = constrain(h * 360);
+	}
+
+	return [h, s, v];
 }
 
 

test/conversions.js

@@ -965,7 +965,7 @@ const tests = {
 				{
 					name: "sRGB white to Okhsl",
 					args: "white",
-					expect: [NaN, 0.0, 1.0000000000000002],
+					expect: [ null, 0.0, 1.0000000000000002 ],
 				},
 				{
 					name: "sRGB red to Okhsl",
@@ -1000,7 +1000,7 @@ const tests = {
 				{
 					name: "sRGB black to Okhsl",
 					args: "black",
-					expect: [NaN, 0.0, 0.0],
+					expect: [ null, 0.0, 0.0 ],
 				},
 			],
 		},
@@ -1013,7 +1013,7 @@ const tests = {
 				{
 					name: "sRGB white to Okhsv",
 					args: "white",
-					expect: [ NaN, 0.0, 1.0000000000000007 ],
+					expect: [ null, 1.3189507366749435e-15, 1.0000000000000007 ],
 				},
 				{
 					name: "sRGB red to Okhsv",
@@ -1048,7 +1048,7 @@ const tests = {
 				{
 					name: "sRGB black to Okhsv",
 					args: "black",
-					expect: [NaN, 0.0, 0.0],
+					expect: [ null, 0.0, 0.0],
 				},
 			],
 		},