Fix update value in colorPicker

deskpro · Dec 11, 2018 · 81350e7 · 81350e7
1 parent 6a47acb
commit 81350e7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@deskpro/react-components",
-  "version": "1.4.5-beta.1",
+  "version": "1.4.5-beta.2",
   "description": "Deskpro React Components",
   "main": "dist/index.js",
   "homepage": "https://github.com/deskpro/react-components",

diff --git a/src/Components/Forms/Colorpicker.jsx b/src/Components/Forms/Colorpicker.jsx
@@ -44,7 +44,6 @@ class Colorpicker extends React.Component {
 
     this.state = {
       displayColorPicker: false,
-      color:              props.value,
     };
   }
 
@@ -55,12 +54,10 @@ class Colorpicker extends React.Component {
   handleClose = () => {
     const { name } = this.props;
     this.setState({ displayColorPicker: false });
-
-    this.props.onChange(this.state.color, name);
   };
 
   handleChange = (color) => {
-    this.setState({ color: color[this.props.format] });
+    this.props.onChange(this.state.color, color[this.props.format]);
   };
 
   render() {
@@ -101,7 +98,7 @@ class Colorpicker extends React.Component {
         { this.state.displayColorPicker ? (
           <div className="dp-colorpicker--popover">
             <div className="dp-colorpicker--popover--cover" onClick={this.handleClose} />
-            <SketchPicker color={this.state.color} onChange={this.handleChange} />
+            <SketchPicker color={color} onChange={this.handleChange} />
           </div>
         ) : null }
       </div>