Merge pull request #432 from Budibase/colorpicker/palette-drag

Palette Drag, Debounce and Tidyup

packages/builder/src/components/userInterface/Colorpicker/actions/drag.js

@@ -2,13 +2,13 @@ export default function(node) {
   function handleMouseDown() {
     window.addEventListener("mousemove", handleMouseMove)
     window.addEventListener("mouseup", handleMouseUp)
+    node.dispatchEvent(new CustomEvent("dragstart"))
   }
 
   function handleMouseMove(event) {
-    let mouseX = event.clientX
     node.dispatchEvent(
       new CustomEvent("drag", {
-        detail: mouseX,
+        detail: { mouseX: event.clientX, mouseY: event.clientY },
       })
     )
   }

packages/builder/src/components/userInterface/Colorpicker/components/ButtonGroup.svelte

@@ -1,8 +1,10 @@
 <script>
   import FlatButton from "./FlatButton.svelte"
+  import { createEventDispatcher } from "svelte"
+
+  const dispatch = createEventDispatcher()
 
   export let format = "hex"
-  export let onclick = format => {}
 
   let colorFormats = ["hex", "rgb", "hsl"]
 </script>
@@ -12,7 +14,7 @@
     <FlatButton
       selected={format === text}
       {text}
-      on:click={() => onclick(text)} />
+      on:click={() => dispatch('click', text)} />
   {/each}
 </div>
 

packages/builder/src/components/userInterface/Colorpicker/components/Colorpicker.svelte

@@ -120,7 +120,7 @@
   }
 
   function changeFormatAndConvert(f) {
-    format = f
+    format = f.detail
     value = convertHsvaToFormat([h, s, v, a], format)
   }
 
@@ -251,7 +251,7 @@
       {/if}
 
       <div class="format-input-panel">
-        <ButtonGroup {format} onclick={changeFormatAndConvert} />
+        <ButtonGroup {format} on:click={changeFormatAndConvert} />
         <Input
           {value}
           on:input={event => handleColorInput(event.target.value)}

packages/builder/src/components/userInterface/Colorpicker/components/Colorpreview.svelte

@@ -28,6 +28,8 @@
   let errorMsg = null
 
   const dispatch = createEventDispatcher()
+  const dispatchValue = value =>
+    debounce(() => dispatch("change", value), 300, true)
 
   beforeUpdate(() => {
     format = getColorFormat(value)
@@ -47,7 +49,7 @@
 
   function onColorChange(color) {
     value = color.detail
-    dispatch("change", color.detail)
+    dispatchValue(value)
   }
 
   function calculateDimensions() {

packages/builder/src/components/userInterface/Colorpicker/components/Palette.svelte

@@ -1,5 +1,6 @@
 <script>
   import { onMount, createEventDispatcher } from "svelte"
+  import { drag } from "../actions"
   import CheckedBackground from "./CheckedBackground.svelte"
 
   const dispatch = createEventDispatcher()
@@ -14,18 +15,13 @@
   let paletteHeight,
     paletteWidth = 0
 
-  function handleClick(event) {
+  function handePaletteChange({ mouseX, mouseY }) {
     const { left, top } = palette.getBoundingClientRect()
-    let clickX = event.clientX - left
-    let clickY = event.clientY - top
-    if (
-      clickX > 0 &&
-      clickY > 0 &&
-      clickX < paletteWidth &&
-      clickY < paletteHeight
-    ) {
-      let s = (clickX / paletteWidth) * 100
-      let v = 100 - (clickY / paletteHeight) * 100
+    let x = mouseX - left
+    let y = mouseY - top
+    if (x > 0 && y > 0 && x < paletteWidth && y < paletteHeight) {
+      let s = (x / paletteWidth) * 100
+      let v = 100 - (y / paletteHeight) * 100
       dispatch("change", { s, v })
     }
   }
@@ -46,10 +42,17 @@
     bind:this={palette}
     bind:clientHeight={paletteHeight}
     bind:clientWidth={paletteWidth}
-    on:click={handleClick}
+    on:click={event => handePaletteChange({
+        mouseX: event.clientX,
+        mouseY: event.clientY,
+      })}
     class="palette"
     {style}>
-    <div class="picker" style={pickerStyle} />
+    <div
+      use:drag
+      on:drag={event => handePaletteChange(event.detail)}
+      class="picker"
+      style={pickerStyle} />
   </div>
 </CheckedBackground>
 
@@ -64,10 +67,15 @@
 
   .picker {
     position: absolute;
+    cursor: grab;
     width: 10px;
     height: 10px;
     background: transparent;
     border: 2px solid white;
     border-radius: 50%;
   }
+
+  .picker:active {
+    cursor: grabbing;
+  }
 </style>

packages/builder/src/components/userInterface/Colorpicker/components/Slider.svelte

@@ -14,8 +14,9 @@
 
   const isWithinLimit = value => value >= 0 && value <= upperLimit
 
-  function onSliderChange(mouseX, isDrag = false) {
+  function onSliderChange({ mouseX }, isDrag = false) {
     const { left, width } = slider.getBoundingClientRect()
+
     let clickPosition = mouseX - left
 
     let percentageClick = (clickPosition / sliderWidth).toFixed(2)
@@ -42,6 +43,10 @@
     }
   }
 
+  function handleDragEnd() {
+    dispatch("dragend")
+  }
+
   $: thumbPosition =
     type === "hue" ? sliderWidth * (value / 360) : sliderWidth * value
 
@@ -53,14 +58,14 @@
   bind:this={slider}
   use:keyevents={{ 37: handleLeftKey, 39: handleRightKey }}
   bind:clientWidth={sliderWidth}
-  on:click={event => onSliderChange(event.clientX)}
+  on:click={event => onSliderChange({ mouseX: event.clientX })}
   class="color-format-slider"
   class:hue={type === 'hue'}
   class:alpha={type === 'alpha'}>
   <div
     use:drag
     on:drag={e => onSliderChange(e.detail, true)}
-    on:dragend
+    on:dragend={handleDragEnd}
     class="slider-thumb"
     {style} />
 </div>
@@ -106,4 +111,8 @@
     background-color: #ffffff;
     cursor: grab;
   }
+
+  .slider-thumb:active {
+    cursor: grabbing;
+  }
 </style>

packages/builder/src/components/userInterface/Colorpicker/helpers.js

@@ -13,8 +13,12 @@ export const convertCamel = str => {
   return str.replace(/[A-Z]/g, match => `-${match.toLowerCase()}`)
 }
 
-export const debounce = (fn, milliseconds) => {
-  return () => {
+export const debounce = (fn, milliseconds, callImmediately) => {
+  const debouncedFn = () => {
     setTimeout(fn, milliseconds)
   }
+  if (callImmediately) {
+    debouncedFn()
+  }
+  return debouncedFn
 }