[Bug] Fixed color picker closure when selecting first custom palette …

…value) (#1347) Signed-off-by: Giuseppe Macri <macri.giuseppe@gmail.com>
keplergl · Dec 7, 2020 · 027985a · 027985a
1 parent 7f3be27
commit 027985a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/components/side-panel/layer-panel/color-selector.js b/src/components/side-panel/layer-panel/color-selector.js
@@ -91,7 +91,7 @@ class ColorSelector extends Component {
   node = createRef();
 
   handleClickOutside = e => {
-    if (this.props.colorUI && this.props.colorUI.showSketcher) {
+    if (this.props.colorUI && Number.isInteger(this.props.colorUI.showSketcher)) {
       // if sketcher is open, let sketch to close itself first
       return;
     }

diff --git a/src/components/side-panel/layer-panel/custom-palette.js b/src/components/side-panel/layer-panel/custom-palette.js
@@ -248,7 +248,7 @@ class CustomPalette extends Component {
                     e.stopPropagation();
                   }}
                   onChange={e => this._inputColorHex(index, e)}
-                  id="input-layer-label"
+                  id={`input-layer-label-${index}`}
                 />
               </StyledInlineInput>
               <StyledTrash onClick={() => this._onColorDelete(index)}>