diff --git a/src/components/fields/NumberField.js b/src/components/fields/NumberField.js index 1f7f7c112d..a788ca1240 100644 --- a/src/components/fields/NumberField.js +++ b/src/components/fields/NumberField.js @@ -3,14 +3,71 @@ import React from "react"; import * as types from "../../types"; import { asNumber } from "../../utils"; -function NumberField(props) { - const { StringField } = props.registry.fields; - return ( - props.onChange(asNumber(value))} - /> - ); +/** + * The NumberField class has some special handling for dealing with trailing + * decimal points and/or zeroes. This logic is designed to allow trailing values + * to be visible in the input element, but not be represented in the + * corresponding form data. + * + * The algorithm is as follows: + * + * 1. When the input value changes the value is cached in the component state + * + * 2. The value is then normalized, removing trailing decimal points and zeros, + * then passed to the "onChange" callback + * + * 3. When the component is rendered, the formData value is checked against the + * value cached in the state. If it matches the cached value, the cached + * value is passed to the input instead of the formData value + */ +class NumberField extends React.Component { + constructor(props) { + super(props); + + this.state = { + lastValue: props.value, + }; + } + + handleChange = value => { + // Cache the original value in component state + this.setState({ lastValue: value }); + + // Check that the value is a string (this can happen if the widget used is a + //