Some new values of min, max, value do not correctly propagate about react-slider HOT 4 CLOSED

Hmmm, noticed that _trimAlignValue is using this.props.min and this.props.max, which in componentWillRecieveProps have not been set to their new values. Is that a bug? Or is the intention that the slider ranges can't change?

from react-slider.

That is definitely a bug.

I put it in there because I observed strange behaviour when suppling values that weren't multiples of step, so I forced them to be via _trimAlignValue. However, I think it should be on the user to provide correct values in the first place (also it says so in the documentation), so I reverted the changes.

Im not sure if it solves your problem however. Could you check out the current master?

from react-slider.

Thanks, this is clearer, but doesn't quite solve it. I think the problem is that the re-trimming of value is happening in componentWillReceiveProps, which happens before this.props.min and this.props.max get their new values.

Should _trimAlignValue take optional parameters which override props.min and props.max?

from react-slider.

See #21, let me know what you think.

from react-slider.