clamp instead of mod

oven-sh · Sep 25, 2024 · 5f8872b · 5f8872b
1 parent 5b5b680
commit 5f8872b
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/css/values/color_js.zig b/src/css/values/color_js.zig
@@ -63,7 +63,8 @@ fn colorIntFromJS(globalThis: *JSC.JSGlobalObject, input: JSC.JSValue, comptime
         return null;
     }
 
-    return @mod(input.coerce(i32, globalThis), 256);
+    // CSS spec says to clamp values to their valid range so we'll respect that here
+    return std.math.clamp(input.coerce(i32, globalThis), 0, 255);
 }
 
 // https://github.com/tmux/tmux/blob/dae2868d1227b95fd076fb4a5efa6256c7245943/colour.c#L44-L55