From ae0a12dcd8435d74709d41ed5a14001cd1c2b445 Mon Sep 17 00:00:00 2001
From: lukasmartinelli <me@lukasmartinelli.ch>
Date: Mon, 12 Sep 2016 20:29:53 +0200
Subject: [PATCH] Handle on change in field

---
 src/fields/color.jsx  | 6 +++++-
 src/fields/enum.jsx   | 6 +++++-
 src/fields/number.jsx | 9 +++++++--
 src/fields/spec.jsx   | 4 ++--
 src/fields/string.jsx | 8 ++++++--
 5 files changed, 25 insertions(+), 8 deletions(-)

diff --git a/src/fields/color.jsx b/src/fields/color.jsx
index 4c1895cf..26ad8fba 100644
--- a/src/fields/color.jsx
+++ b/src/fields/color.jsx
@@ -11,9 +11,13 @@ static propTypes = {
     doc: React.PropTypes.string,
   }
 
+	onChange(e) {
+		return this.props.onChange(e.target.value)
+	}
+
 	render() {
 		return <Input
-			onChange={this.props.onChange}
+			onChange={this.onChange.bind(this)}
 			label={this.props.name}
 			name={this.props.name}
 			value={this.props.value}
diff --git a/src/fields/enum.jsx b/src/fields/enum.jsx
index dc740207..3795de10 100644
--- a/src/fields/enum.jsx
+++ b/src/fields/enum.jsx
@@ -10,12 +10,16 @@ class EnumField extends React.Component {
     doc: React.PropTypes.string,
   }
 
+	onChange(e) {
+		return this.props.onChange(e.target.value)
+	}
+
 	render() {
 		const options = this.props.allowedValues.map(val => {
 			return {children: val, value: val}
 		})
 		return <Select
-			onChange={this.props.onChange}
+			onChange={this.onChange.bind(this)}
 			name={this.props.name}
 			options={options}
 			label={this.props.name}
diff --git a/src/fields/number.jsx b/src/fields/number.jsx
index 4b764a73..996325c7 100644
--- a/src/fields/number.jsx
+++ b/src/fields/number.jsx
@@ -14,9 +14,14 @@ class NumberField extends React.Component {
     doc: React.PropTypes.string,
   }
 
+	onChange(e) {
+		return this.props.onChange(parseFloat(e.target.value))
+	}
+
 	render() {
-		return <Input type="number"
-			onChange={this.props.onChange}
+		return <Input
+			type="number"
+			onChange={this.onChange.bind(this)}
 			label={this.props.name}
 			name={this.props.name}
 			message={this.props.doc}
diff --git a/src/fields/spec.jsx b/src/fields/spec.jsx
index b20ff5d7..f6871a16 100644
--- a/src/fields/spec.jsx
+++ b/src/fields/spec.jsx
@@ -18,8 +18,8 @@ class SpecField extends React.Component {
     ]).isRequired,
   }
 
-	onValueChanged(property, e) {
-		return this.props.onChange(property, e.target.value)
+	onValueChanged(property, value) {
+		return this.props.onChange(property, value)
 	}
 
 	render() {
diff --git a/src/fields/string.jsx b/src/fields/string.jsx
index b1b1f2dc..191e2b32 100644
--- a/src/fields/string.jsx
+++ b/src/fields/string.jsx
@@ -6,14 +6,18 @@ class StringField extends React.Component {
 static propTypes = {
     onChange: React.PropTypes.func.isRequired,
 		name: React.PropTypes.string.isRequired,
-    value: React.PropTypes.number,
+    value: React.PropTypes.string,
     default: React.PropTypes.number,
     doc: React.PropTypes.string,
   }
 
+	onChange(e) {
+		return this.props.onChange(e.target.value)
+	}
+
 	render() {
 		return <Input
-			onChange={this.props.onChange}
+			onChange={this.onChange.bind(this)}
 			label={this.props.name}
 			name={this.props.name}
 			value={this.props.value}