Merge pull request #33 from commitd/sh/step

fix: improve number handling

Author: stuarthendren

src/components/widgets/TextWidget/TextWidget.stories.tsx

@@ -13,3 +13,67 @@ Default.args = {
     name: Type.String({ title: 'Name', description: 'description' }),
   }),
 }
+
+export const Number = DefaultStory.bind({})
+Number.args = {
+  schema: Type.Object({
+    name: Type.Number({ title: 'Number', description: 'description' }),
+  }),
+}
+
+export const Integer = DefaultStory.bind({})
+Integer.args = {
+  schema: Type.Object({
+    name: Type.Integer({ title: 'Integer', description: 'description' }),
+  }),
+}
+
+export const Limited = DefaultStory.bind({})
+Limited.args = {
+  schema: Type.Object({
+    name: Type.Integer({
+      title: 'Limited',
+      minimum: 0,
+      maximum: 10,
+      description: 'min 0 - max 10',
+    }),
+  }),
+}
+
+export const Multiple = DefaultStory.bind({})
+Multiple.args = {
+  schema: Type.Object({
+    name: Type.Integer({
+      title: 'Multiple',
+      minimum: 0,
+      maximum: 10,
+      multipleOf: 2,
+      description: '0-2 multiple of 2',
+    }),
+  }),
+}
+
+export const LimitedFloat = DefaultStory.bind({})
+LimitedFloat.args = {
+  schema: Type.Object({
+    name: Type.Number({
+      title: 'Float',
+      minimum: 0,
+      maximum: 1,
+      description: 'between 0 and 1',
+    }),
+  }),
+}
+
+export const MultipleFloat = DefaultStory.bind({})
+MultipleFloat.args = {
+  schema: Type.Object({
+    name: Type.Number({
+      title: 'Multiple Float',
+      minimum: 0,
+      maximum: 1,
+      multipleOf: 0.01,
+      description: 'between 0 and 1, multiple of 0.01',
+    }),
+  }),
+}

src/components/widgets/TextWidget/TextWidget.tsx

@@ -60,7 +60,10 @@ export const TextWidget: FC<TextWidgetProps> = ({
 
   const inputType = getInputType(type ?? schema.type ?? 'string')
 
-  const rangeProps = inputType === 'number' ? { ...rangeSpec(schema) } : {}
+  const rangeProps =
+    inputType === 'number'
+      ? { step: schema.type === 'integer' ? '1' : 'any', ...rangeSpec(schema) }
+      : {}
 
   return (
     <Input