From d9ceee49a1d212e8217869a7aa9e3bd22976fb1f Mon Sep 17 00:00:00 2001
From: Michael Shanks <mike@budibase.com>
Date: Wed, 19 Feb 2020 10:17:31 +0000
Subject: [PATCH] reording and duplication of components

---
 packages/builder/src/builderStore/store.js    | 82 +++++++++++++++++--
 .../src/common/Icons/ChevronDown.svelte       |  8 ++
 .../builder/src/common/Icons/ChevronUp.svelte |  8 ++
 packages/builder/src/common/Icons/Copy.svelte |  8 ++
 packages/builder/src/common/Icons/index.js    |  3 +
 .../userInterface/ComponentsHierarchy.svelte  |  5 +-
 .../ComponentsHierarchyChildren.svelte        | 73 ++++++++++++++---
 .../userInterface/UserInterfaceRoot.svelte    |  5 +-
 8 files changed, 170 insertions(+), 22 deletions(-)
 create mode 100644 packages/builder/src/common/Icons/ChevronDown.svelte
 create mode 100644 packages/builder/src/common/Icons/ChevronUp.svelte
 create mode 100644 packages/builder/src/common/Icons/Copy.svelte

diff --git a/packages/builder/src/builderStore/store.js b/packages/builder/src/builderStore/store.js
index 8d2456abb0..70207314ac 100644
--- a/packages/builder/src/builderStore/store.js
+++ b/packages/builder/src/builderStore/store.js
@@ -33,6 +33,7 @@ import { loadLibs, loadLibUrls } from "./loadComponentLibraries"
 import { buildCodeForScreens } from "./buildCodeForScreens"
 import { generate_screen_css } from "./generate_css"
 import { insertCodeMetadata } from "./insertCodeMetadata"
+import { uuid } from "./uuid"
 
 let appname = ""
 
@@ -107,6 +108,9 @@ export const getStore = () => {
   store.setComponentCode = setComponentCode(store)
   store.setScreenType = setScreenType(store)
   store.deleteComponent = deleteComponent(store)
+  store.moveUpComponent = moveUpComponent(store)
+  store.moveDownComponent = moveDownComponent(store)
+  store.copyComponent = copyComponent(store)
   return store
 }
 
@@ -815,13 +819,7 @@ const setScreenType = store => type => {
 
 const deleteComponent = store => component => {
   store.update(s => {
-    let parent
-    walkProps(s.currentPreviewItem.props, (p, breakWalk) => {
-      if (p._children.includes(component)) {
-        parent = p
-        breakWalk()
-      }
-    })
+    const parent = getParent(s.currentPreviewItem.props, component)
 
     if (parent) {
       parent._children = parent._children.filter(c => c !== component)
@@ -835,6 +833,76 @@ const deleteComponent = store => component => {
   })
 }
 
+const moveUpComponent = store => component => {
+  store.update(s => {
+    const parent = getParent(s.currentPreviewItem.props, component)
+
+    if (parent) {
+      const currentIndex = parent._children.indexOf(component)
+      if (currentIndex === 0) return s
+
+      const newChildren = parent._children.filter(c => c !== component)
+      newChildren.splice(currentIndex - 1, 0, component)
+      parent._children = newChildren
+    }
+    s.currentComponentInfo = component
+    s.currentFrontEndType === "page"
+      ? _savePage(s)
+      : _saveScreenApi(s.currentPreviewItem, s)
+
+    return s
+  })
+}
+
+const moveDownComponent = store => component => {
+  store.update(s => {
+    const parent = getParent(s.currentPreviewItem.props, component)
+
+    if (parent) {
+      const currentIndex = parent._children.indexOf(component)
+      if (currentIndex === parent._children.length - 1) return s
+
+      const newChildren = parent._children.filter(c => c !== component)
+      newChildren.splice(currentIndex + 1, 0, component)
+      parent._children = newChildren
+    }
+    s.currentComponentInfo = component
+    s.currentFrontEndType === "page"
+      ? _savePage(s)
+      : _saveScreenApi(s.currentPreviewItem, s)
+
+    return s
+  })
+}
+
+const copyComponent = store => component => {
+  store.update(s => {
+    const parent = getParent(s.currentPreviewItem.props, component)
+    const copiedComponent = cloneDeep(component)
+    walkProps(copiedComponent, p => {
+      p._id = uuid()
+    })
+    parent._children = [...parent._children, copiedComponent]
+    s.curren
+    s.currentFrontEndType === "page"
+      ? _savePage(s)
+      : _saveScreenApi(s.currentPreviewItem, s)
+    s.currentComponentInfo = copiedComponent
+    return s
+  })
+}
+
+const getParent = (rootProps, child) => {
+  let parent
+  walkProps(rootProps, (p, breakWalk) => {
+    if (p._children.includes(child)) {
+      parent = p
+      breakWalk()
+    }
+  })
+  return parent
+}
+
 const walkProps = (props, action, cancelToken = null) => {
   cancelToken = cancelToken || { cancelled: false }
   action(props, () => {
diff --git a/packages/builder/src/common/Icons/ChevronDown.svelte b/packages/builder/src/common/Icons/ChevronDown.svelte
new file mode 100644
index 0000000000..4e678136ac
--- /dev/null
+++ b/packages/builder/src/common/Icons/ChevronDown.svelte
@@ -0,0 +1,8 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-chevron-down"><polyline points="6 9 12 15 18 9"></polyline></svg>
+
+<style>
+svg {
+    height: 100%;
+    width: 100%;
+}
+</style>
\ No newline at end of file
diff --git a/packages/builder/src/common/Icons/ChevronUp.svelte b/packages/builder/src/common/Icons/ChevronUp.svelte
new file mode 100644
index 0000000000..8a0e3246a5
--- /dev/null
+++ b/packages/builder/src/common/Icons/ChevronUp.svelte
@@ -0,0 +1,8 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-chevron-up"><polyline points="18 15 12 9 6 15"></polyline></svg>
+
+<style>
+svg {
+    height: 100%;
+    width: 100%;
+}
+</style>
\ No newline at end of file
diff --git a/packages/builder/src/common/Icons/Copy.svelte b/packages/builder/src/common/Icons/Copy.svelte
new file mode 100644
index 0000000000..f8fdc25663
--- /dev/null
+++ b/packages/builder/src/common/Icons/Copy.svelte
@@ -0,0 +1,8 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" class="feather feather-copy"><rect x="9" y="9" width="13" height="13" rx="2" ry="2"></rect><path d="M5 15H4a2 2 0 0 1-2-2V4a2 2 0 0 1 2-2h9a2 2 0 0 1 2 2v1"></path></svg>
+
+<style>
+svg {
+    height: 100%;
+    width: 100%;
+}
+</style>
\ No newline at end of file
diff --git a/packages/builder/src/common/Icons/index.js b/packages/builder/src/common/Icons/index.js
index 7258d4c90c..353985a59e 100644
--- a/packages/builder/src/common/Icons/index.js
+++ b/packages/builder/src/common/Icons/index.js
@@ -8,3 +8,6 @@ export { default as CircleIndicator } from "./CircleIndicator.svelte"
 export { default as PencilIcon } from "./Pencil.svelte"
 export { default as EventsIcon } from "./Events.svelte"
 export { default as XCircleIcon } from "./XCircle.svelte"
+export { default as ChevronUpIcon } from "./ChevronUp.svelte"
+export { default as ChevronDownIcon } from "./ChevronDown.svelte"
+export { default as CopyIcon } from "./Copy.svelte"
diff --git a/packages/builder/src/userInterface/ComponentsHierarchy.svelte b/packages/builder/src/userInterface/ComponentsHierarchy.svelte
index 274615c9cc..4df058e455 100644
--- a/packages/builder/src/userInterface/ComponentsHierarchy.svelte
+++ b/packages/builder/src/userInterface/ComponentsHierarchy.svelte
@@ -74,7 +74,10 @@
         components={screen.component.props._children}
         currentComponent={$store.currentComponentInfo}
         onSelect={store.selectComponent} 
-        onDeleteComponent={confirmDeleteComponent}/>
+        onDeleteComponent={confirmDeleteComponent}
+        onMoveUpComponent={store.moveUpComponent}
+        onMoveDownComponent={store.moveDownComponent}
+        onCopyComponent={store.copyComponent}/>
     {/if}
   {/each}
 
diff --git a/packages/builder/src/userInterface/ComponentsHierarchyChildren.svelte b/packages/builder/src/userInterface/ComponentsHierarchyChildren.svelte
index 7b0b326a94..6e1de3db6f 100644
--- a/packages/builder/src/userInterface/ComponentsHierarchyChildren.svelte
+++ b/packages/builder/src/userInterface/ComponentsHierarchyChildren.svelte
@@ -1,13 +1,16 @@
 <script>
   import { last } from "lodash/fp"
   import { pipe } from "../common/core"
-  import { XCircleIcon } from "../common/Icons"
+  import { XCircleIcon, ChevronUpIcon, ChevronDownIcon, CopyIcon } from "../common/Icons"
 
   export let components = []
   export let currentComponent
   export let onSelect = () => {}
   export let level = 0
   export let onDeleteComponent
+  export let onMoveUpComponent
+  export let onMoveDownComponent
+  export let onCopyComponent
 
   
   const capitalise = s => s.substring(0, 1).toUpperCase() + s.substring(1)
@@ -19,22 +22,43 @@
       [get_name, capitalise]
     )
 
+  const moveDownComponent = component  => {
+    const c = component
+    return () => {
+      return onMoveDownComponent(c)
+    }
+  }
+
 </script>
 
 <ul>
-  {#each components as component}
+  {#each components as component, index (component._id)}
     <li on:click|stopPropagation={() => onSelect(component)}>
-      <span
+      <div
         class="item"
         class:selected={currentComponent === component}
         style="padding-left: {level * 20 + 67}px">
         <span class="item-name">{get_capitalised_name(component._component)}</span>
+        <div class="reorder-buttons">
+          {#if index > 0}
+          <button on:click|stopPropagation={() => onMoveUpComponent(component)}>
+            <ChevronUpIcon />
+          </button>
+          {/if}
+          {#if index < (components.length - 1)}
+          <button on:click|stopPropagation={moveDownComponent(component)}>
+            <ChevronDownIcon />
+          </button>
+          {/if}
+        </div>
+        <button on:click|stopPropagation={() => onCopyComponent(component)}>
+            <CopyIcon />
+        </button>
         <button 
-          class="delete-component"
-          on:click={() => onDeleteComponent(component)}>
+          on:click|stopPropagation={() => onDeleteComponent(component)}>
           <XCircleIcon />
         </button>
-      </span>
+      </div>
 
       {#if component._children}
         <svelte:self
@@ -42,7 +66,10 @@
           {currentComponent}
           {onSelect}
           level={level + 1}
-          {onDeleteComponent} />
+          {onDeleteComponent}
+          {onMoveUpComponent}
+          {onMoveDownComponent}
+          {onCopyComponent} />
       {/if}
     </li>
   {/each}
@@ -54,11 +81,19 @@
     padding-left: 0;
     margin: 0;
   }
+
+  li {
+    background: #fafafa;
+  }
+
   .item {
     display: flex;
     flex-direction: row;
-    padding: 11px 5px 11px 67px;
+    padding: 0px 5px 0px 67px;
+    margin: auto 0px;
     border-radius: 3px;
+    height: 43px;
+    align-items: center;
   }
 
   .item > span {
@@ -66,25 +101,25 @@
     flex: 1 1 auto;
   }
 
-  .item > button {
+  .item button {
     display: none;
     height: 20px;
-    color: var(--slate)
+    color: var(--slate);
+    padding: 0px 5px;
   }
 
   .item:hover {
-
     background: #fafafa;
     cursor: pointer;
   }
-  .item:hover > button {
+  .item:hover button {
     border-style: none;
     background: rgba(0,0,0,0);
     display: block;
     cursor: pointer;
   }
 
-  .item:hover > button:hover {
+  .item:hover button:hover {
     color: var(--button-text);
   }
 
@@ -92,4 +127,16 @@
     color: var(--button-text);
     background: var(--background-button) !important;
   }
+
+  .reorder-buttons {
+    display: flex;
+    flex-direction: column;
+    height: 100%;
+  }
+
+  .reorder-buttons > button {
+    flex: 1 1 auto;
+    height: 17px
+  }
+
 </style>
diff --git a/packages/builder/src/userInterface/UserInterfaceRoot.svelte b/packages/builder/src/userInterface/UserInterfaceRoot.svelte
index 82a99c249f..84b1e2728a 100644
--- a/packages/builder/src/userInterface/UserInterfaceRoot.svelte
+++ b/packages/builder/src/userInterface/UserInterfaceRoot.svelte
@@ -31,7 +31,7 @@
     confirmDeleteDialog.show()
   }
   
-  const lastPartOfName = c => c ? c.split("/") : ""
+  const lastPartOfName = c => c ? last(c.split("/")) : ""
   
 </script>
 
@@ -67,6 +67,9 @@
             currentComponent={$store.currentComponentInfo}
             onDeleteComponent={confirmDeleteComponent}
             onSelect={store.selectComponent}
+            onMoveUpComponent={store.moveUpComponent}
+            onMoveDownComponent={store.moveDownComponent}
+            onCopyComponent={store.copyComponent}
             level={-2} />
         {/if}