From 897a0a875b950509b61a0f8d07a160dae94747d7 Mon Sep 17 00:00:00 2001
From: Conor_Mack <cmckeown101@gmail.com>
Date: Mon, 22 Jun 2020 16:32:57 +0100
Subject: [PATCH] Swatch Panel, Logic and UI Updates

---
 .../Colorpicker/CheckedBackground.svelte      |   5 +-
 .../Colorpicker/Colorpicker.svelte            | 104 ++++++++++++------
 .../Colorpicker/Colorpreview.svelte           |   4 +-
 .../userInterface/Colorpicker/Swatch.svelte   |  53 +++++++++
 4 files changed, 129 insertions(+), 37 deletions(-)
 create mode 100644 packages/builder/src/components/userInterface/Colorpicker/Swatch.svelte

diff --git a/packages/builder/src/components/userInterface/Colorpicker/CheckedBackground.svelte b/packages/builder/src/components/userInterface/Colorpicker/CheckedBackground.svelte
index 9ed2899203..19d54523b2 100644
--- a/packages/builder/src/components/userInterface/Colorpicker/CheckedBackground.svelte
+++ b/packages/builder/src/components/userInterface/Colorpicker/CheckedBackground.svelte
@@ -6,8 +6,9 @@
     export let borderRadius = ""
     export let height = ""
     export let width = ""
+    export let margin = ""
 
-    $: style = buildStyle({backgroundSize, borderRadius, height, width})
+    $: style = buildStyle({backgroundSize, borderRadius, height, width, margin})
 
 </script>
 
@@ -19,6 +20,6 @@
   }
 </style>
 
-<div transition:fade {style}>
+<div in:fade {style}>
     <slot />
 </div>
\ No newline at end of file
diff --git a/packages/builder/src/components/userInterface/Colorpicker/Colorpicker.svelte b/packages/builder/src/components/userInterface/Colorpicker/Colorpicker.svelte
index 2e80f826bd..5989e6b3ed 100644
--- a/packages/builder/src/components/userInterface/Colorpicker/Colorpicker.svelte
+++ b/packages/builder/src/components/userInterface/Colorpicker/Colorpicker.svelte
@@ -1,6 +1,7 @@
 <script>
   import { onMount, createEventDispatcher } from "svelte";
   import { fade } from 'svelte/transition';
+  import Swatch from "./Swatch.svelte";
   import CheckedBackground from "./CheckedBackground.svelte"
   import {buildStyle} from "./helpers.js"
   import {
@@ -14,13 +15,16 @@
   import Input from "./Input.svelte";
 
   export let value = "#3ec1d3ff";
+  export let swatches = [] //TODO: Safe swatches - limit to 12. warn in console
+  export let disableSwatches = false
   export let format = "hexa";
+  export let open = false;
   
-  let recentColors = []
-
   export let pickerHeight = 0;
   export let pickerWidth = 0;
 
+  let adder = null;
+
   let h = null;
   let s = null;
   let v = null;
@@ -29,7 +33,10 @@
   const dispatch = createEventDispatcher();
 
   onMount(() => {
-    getRecentColors()
+    if(!swatches.length > 0) {
+      //Don't use locally stored recent colors if swatches have been passed as props
+      getRecentColors()
+    }
 
     if (format) {
       convertAndSetHSVA()
@@ -39,17 +46,17 @@
   function getRecentColors() {
     let colorStore = localStorage.getItem("cp:recent-colors")
     if (colorStore) {
-      recentColors = JSON.parse(colorStore)
+      swatches = JSON.parse(colorStore)      
     }
   }
 
   function setRecentColor(color) {
-    if (recentColors.length === 6) {
-      recentColors.splice(0, 1)
+    if (swatches.length === 12) {
+      swatches.splice(0, 1)
     }
-    if (!recentColors.includes(color)) {
-      recentColors = [...recentColors, color]
-      localStorage.setItem("cp:recent-colors", JSON.stringify(recentColors))
+    if (!swatches.includes(color)) {
+      swatches = [...swatches, color]
+      localStorage.setItem("cp:recent-colors", JSON.stringify(swatches))
     }
   }
 
@@ -70,7 +77,6 @@
     s = detail.s;
     v = detail.v;
     value = convertHsvaToFormat([h, s, v, a], format);
-    setRecentColor(value)
     dispatchValue()
   }
 
@@ -78,7 +84,6 @@
     h = color;
     value = convertHsvaToFormat([h, s, v, a], format);   
     if(!isDrag) {
-      setRecentColor(value)
       dispatchValue()
     } 
   }
@@ -87,7 +92,6 @@
     a = color === "1.00" ? "1" : color;
     value = convertHsvaToFormat([h, s, v, a], format);    
     if(!isDrag) {
-        setRecentColor(value)
         dispatchValue()
       } 
   }
@@ -111,13 +115,26 @@
 
   function dispatchInputChange() {
     if(format) {
-      setRecentColor(value)
       dispatchValue()
     }
   }
+
+  function addSwatch() {
+    if(format) {
+      dispatch("addswatch", value)
+      setRecentColor(value)
+    }
+  }
+
+  function removeSwatch(idx) {
+    let removedSwatch = swatches.splice(idx, 1);
+    swatches = swatches
+    dispatch("removeswatch", removedSwatch)
+    localStorage.setItem("cp:recent-colors", JSON.stringify(swatches))
+  }
   
 
-  function applyRecentColor(color) {
+  function applySwatch(color) {
     if(value !== color) {
       format = getColorFormat(color)
       if(format) {
@@ -127,8 +144,10 @@
       }
     }
   }
+
     $: border = (v > 90 && s < 5) ? "1px dashed #dedada" : ""
     $: style = buildStyle({background: value, border})
+    $: shrink = swatches.length > 0
 </script>
 
 <style>
@@ -173,28 +192,40 @@
     border-radius: 50%;
   }
 
-  .recent-color-panel {
+   .swatch-panel {
     flex: 0 0 15px;
-    display: grid;
-    grid-gap: 10px;
+    display: flex;
+    flex-flow: row wrap;
     justify-content: flex-start;
-    grid-auto-flow: column;
-    padding: 5px;
-    margin-left: 6px;
+    padding: 0 5px;
+    max-height: 56px;
   }
 
-  .recent-color {
+  .adder {
+    flex: 1;
+    height: 20px;		
+    display: flex;
+    transition: flex 0.5s;
+    justify-content: center;
+    align-items: center;
+    background: #f1f3f4;
     cursor: pointer;
-    border-radius: 6px;
-    border: 1px solid #dedada;  
-    height: 20px; 
-    width: 20px;   
+    border: 1px solid #d4d4d4;
+    border-radius: 8px;
+    margin-left: 5px;
+    margin-top: 3px;
+    font-weight: 500;
+  }
+
+  .shrink { 
+    flex: 0 0 20px;
   }
 
   .format-input-panel {
     display: flex;
     flex-direction: column;
     justify-content: center;
+    padding-top: 3px;
   }
 </style>
 
@@ -226,15 +257,20 @@
       </div>
     </div>
 
-    {#if recentColors.length > 0}
-      <div transition:fade class="recent-color-panel">
-        {#each recentColors as color}
-          <CheckedBackground borderRadius="6px"> 
-            <div transition:fade class="recent-color" style={`background: ${color};`} on:click={() => applyRecentColor(color)} />
-          </CheckedBackground>
-        {/each}
-      </div>
-    {/if}
+      {#if !disableSwatches}
+        <div transition:fade class="swatch-panel">
+          {#if swatches.length > 0}
+            {#each swatches as color, idx}
+              <Swatch {color} on:click={() => applySwatch(color)} on:removeswatch={() => removeSwatch(idx)} />
+            {/each}
+          {/if}
+          {#if swatches.length !== 12}
+            <div bind:this={adder} transition:fade class="adder" on:click={addSwatch} class:shrink>
+              <span>&plus;</span>
+            </div>
+          {/if}
+        </div>
+      {/if}
 
     <div class="format-input-panel">
       <ButtonGroup {format} onclick={changeFormatAndConvert} />
diff --git a/packages/builder/src/components/userInterface/Colorpicker/Colorpreview.svelte b/packages/builder/src/components/userInterface/Colorpicker/Colorpreview.svelte
index 0534666e1c..32a5e63ff3 100644
--- a/packages/builder/src/components/userInterface/Colorpicker/Colorpreview.svelte
+++ b/packages/builder/src/components/userInterface/Colorpicker/Colorpreview.svelte
@@ -8,6 +8,8 @@
     import {getColorFormat} from "./utils.js"
 
     export let value = "#3ec1d3ff"
+    export let swatches = []
+    export let disableSwatches = false
     export let open = false;
     export let width = "25px"
     export let height = "25px"
@@ -99,7 +101,7 @@
 
         {#if open}
         <div transition:fade class="picker-container" style={pickerStyle}>
-            <Colorpicker on:change={onColorChange} bind:format bind:value bind:pickerHeight bind:pickerWidth />
+            <Colorpicker on:change={onColorChange} on:addswatch on:removeswatch bind:format bind:value bind:pickerHeight bind:pickerWidth {swatches} {disableSwatches} {open} />
         </div>
         <div on:click|self={() => open = false} class="overlay"></div>
         {/if}
diff --git a/packages/builder/src/components/userInterface/Colorpicker/Swatch.svelte b/packages/builder/src/components/userInterface/Colorpicker/Swatch.svelte
new file mode 100644
index 0000000000..e602b72d33
--- /dev/null
+++ b/packages/builder/src/components/userInterface/Colorpicker/Swatch.svelte
@@ -0,0 +1,53 @@
+<script>
+    import {createEventDispatcher} from "svelte"
+    import { fade } from 'svelte/transition';
+    import CheckedBackground from "./CheckedBackground.svelte"
+
+    export let hovered = false
+    export let color = "#fff"
+
+    const dispatch = createEventDispatcher()
+
+</script>
+
+<style>
+  .swatch {
+    position: relative;
+    cursor: pointer;
+    border-radius: 6px;
+    border: 1px solid #dedada;  
+    height: 20px; 
+    width: 20px;
+  }
+
+  .space {
+      padding: 3px 5px;
+  }
+
+  .remove-icon {
+    position: absolute;
+    right: 0;
+    top: -5px;
+    right: -4px;
+    width:10px;
+    height: 10px;
+    border-radius: 50%;
+    background-color: #800000;
+    color: #fff;
+    display: flex;
+    justify-content: center;
+    align-items: center;
+  }
+</style>
+
+<div class="space">
+  <CheckedBackground borderRadius="6px"> 
+    <div in:fade class="swatch" style={`background: ${color};`} on:click|self on:mouseover={() => hovered = true} on:mouseleave={() => hovered = false}>
+        {#if hovered}
+        <div in:fade class="remove-icon" on:click|self={()=> dispatch("removeswatch")}>
+            <span on:click|self={()=> dispatch("removeswatch")}>&times;</span>
+        </div>
+        {/if}
+    </div>            
+</CheckedBackground>
+</div>
\ No newline at end of file