Skip to content

Enable wrapping around of hue values given in degrees #108

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.

Sign up for GitHub

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

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

Enable wrapping around of hue values given in degrees #108

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
473a671
Add function boundInfinity
timothyausten Oct 30, 2015
a04d9d9
Fix function boundInfinity
timothyausten Oct 30, 2015
b82f921
Normalize values for degrees that are not [0,360]
timothyausten Oct 30, 2015
9307a22
Add functionality to function bound01 to handle values given in degre…
timothyausten Oct 31, 2015
b619bc3
Remove test folder
timothyausten Nov 1, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 15 additions & 4 deletions tinycolor.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -977,7 +977,8 @@ function bound01(n, max) {
if (isOnePointZero(n)) { n = "100%"; }

var processPercent = isPercentage(n);
n = mathMin(max, mathMax(0, parseFloat(n)));
// If range [0,360] then do nothing.
n = max === 360 ? n : mathMin(max, mathMax(0, parseFloat(n)));

// Automatically convert percentage into number
if (processPercent) {
Expand All @@ -989,11 +990,21 @@ function bound01(n, max) {
return 1;
}

// Convert into [0, 1] range if it isn't already
return (n % max) / parseFloat(max);
if (max === 360) {
// If n is a hue given in degrees,
// wrap around out-of-range values into [0, 360] range
// then convert into [0, 1].
n = (n < 0 ? n % max + max : n % max) / parseFloat(max);
} else {
// If n not a hue given in degrees
// Convert into [0, 1] range if it isn't already.
n = (n % max) / parseFloat(max);
}

return n;
}

// Force a number between 0 and 1
// Force a number between 0 and 1
function clamp01(val) {
return mathMin(1, mathMax(0, val));
}
Expand Down