Skip to content

Commit

Permalink
fix: change value type to string to avoid leading zero
Browse files Browse the repository at this point in the history
  • Loading branch information
MikitaLisavets committed Nov 9, 2020
1 parent 90ecd6e commit 705dc21
Show file tree
Hide file tree
Showing 3 changed files with 25 additions and 16 deletions.
4 changes: 2 additions & 2 deletions packages/components/src/numberRange/NumberRange.tsx
Expand Up @@ -58,7 +58,7 @@ const NumberRange: React.FC<Props> = ({
<TextInput
type="number"
label={gteLabel}
value={value.gte}
value={String(value.gte)}
disabled={gteDisabled}
onChange={e =>
onChange({
Expand All @@ -71,7 +71,7 @@ const NumberRange: React.FC<Props> = ({
<TextInput
type="number"
label={lteLabel}
value={value.lte}
value={String(value.lte)}
disabled={lteDisabled}
onChange={e =>
onChange({
Expand Down
33 changes: 21 additions & 12 deletions packages/components/src/numberRange/README.mdx
Expand Up @@ -14,18 +14,27 @@ Filter for numerical "more than" and "less than"
## Usage

```jsx react-live
<NumberRange
text="Enter a minimum, maximum or range limit"
lteLabel="Max"
gteLabel="Min"
value={{ gte: 16, lte: 20 }}
onChange={value => console.log(value)}
lteDisabled={false}
lteError={false}
gteDisabled={false}
gteError={false}
maxWidth={246}
/>
() => {
const [value, setValue] = React.useState({ gte: 16, lte: 20 });

return (
<NumberRange
text="Enter a minimum, maximum or range limit"
lteLabel="Max"
gteLabel="Min"
value={value}
onChange={newValue => {
console.log(newValue);
setValue(newValue);
}}
lteDisabled={false}
lteError={false}
gteDisabled={false}
gteError={false}
maxWidth={246}
/>
);
};
```

## State
Expand Down
Expand Up @@ -142,7 +142,7 @@ exports[`Number Range renders correctly 1`] = `
onChange={[Function]}
placeholder=" "
type="number"
value={16}
value="16"
/>
</label>
<label
Expand All @@ -160,7 +160,7 @@ exports[`Number Range renders correctly 1`] = `
onChange={[Function]}
placeholder=" "
type="number"
value={20}
value="20"
/>
</label>
</div>
Expand Down

0 comments on commit 705dc21

Please sign in to comment.