layercake icon indicating copy to clipboard operation
layercake copied to clipboard

Add types to range calculation functions

Open rgieseke opened this issue 1 year ago • 2 comments

Maybe the s parameter should only be y or r?

rgieseke avatar Oct 04 '24 14:10 rgieseke

What's your thinking in having it just be those two? createScale gets called with all of those parameters, no?

mhkeller avatar Oct 07 '24 01:10 mhkeller

What's your thinking in having it just be those two? createScale gets called with all of those parameters, no?

Uh, yeah, I got confused in the ternaries handling min/max - 'x' and 'z' are the final case.

rgieseke avatar Oct 07 '24 21:10 rgieseke

Handled in https://github.com/mhkeller/layercake/pull/231

mhkeller avatar Jul 12 '25 21:07 mhkeller