refactor: no reducer needed
This commit is contained in:
parent
d096c12636
commit
91b96283ad
|
@ -12,7 +12,6 @@ const Range = ({
|
|||
integer,
|
||||
onChange,
|
||||
range,
|
||||
reducer,
|
||||
}) => {
|
||||
const [isSingle, setIsSingle] = useState(!Number.isNaN(range));
|
||||
return (
|
||||
|
@ -22,10 +21,10 @@ const Range = ({
|
|||
onChange={(value) => {
|
||||
onChange?.(
|
||||
isSingle
|
||||
? reducer(value)
|
||||
? value
|
||||
: {
|
||||
min: reducer(value),
|
||||
max: reducer(range.max),
|
||||
min: value,
|
||||
max: range.max,
|
||||
},
|
||||
);
|
||||
}}
|
||||
|
@ -41,10 +40,10 @@ const Range = ({
|
|||
setIsSingle(!event.target.checked);
|
||||
onChange?.(
|
||||
!event.target.checked
|
||||
? reducer(range.min)
|
||||
? range.min
|
||||
: {
|
||||
min: reducer(range),
|
||||
max: reducer(range),
|
||||
min: range,
|
||||
max: range,
|
||||
},
|
||||
);
|
||||
}}
|
||||
|
@ -55,8 +54,8 @@ const Range = ({
|
|||
integer={integer}
|
||||
onChange={(value) => {
|
||||
onChange?.({
|
||||
min: reducer(range.min),
|
||||
max: reducer(value),
|
||||
min: range.min,
|
||||
max: value,
|
||||
});
|
||||
}}
|
||||
value={range.max}
|
||||
|
@ -91,14 +90,12 @@ rangePropType.isRequired = createRangePropType(true);
|
|||
Range.defaultProps = {
|
||||
integer: false,
|
||||
onChange: null,
|
||||
reducer: (state) => parseFloat(state),
|
||||
};
|
||||
|
||||
Range.propTypes = {
|
||||
integer: PropTypes.bool,
|
||||
onChange: PropTypes.func,
|
||||
range: rangePropType.isRequired,
|
||||
reducer: PropTypes.func,
|
||||
};
|
||||
|
||||
export default Range;
|
||||
|
|
Loading…
Reference in New Issue
Block a user