From 2db40c6d2c6178bea6ba91381dfb2e83973f8fa3 Mon Sep 17 00:00:00 2001 From: Michael Shanks Date: Wed, 12 Aug 2020 16:28:19 +0100 Subject: [PATCH] dnd component nav --- .../builder/src/builderStore/store/index.js | 53 +++++++- .../builder/src/builderStore/storeUtils.js | 6 + .../ComponentDropdownMenu.svelte | 46 +------ .../userInterface/ComponentsHierarchy.svelte | 19 +-- .../ComponentsHierarchyChildren.svelte | 113 +++++++++++++++++- 5 files changed, 185 insertions(+), 52 deletions(-) diff --git a/packages/builder/src/builderStore/store/index.js b/packages/builder/src/builderStore/store/index.js index 5b78ea0979..3894725ead 100644 --- a/packages/builder/src/builderStore/store/index.js +++ b/packages/builder/src/builderStore/store/index.js @@ -1,4 +1,4 @@ -import { values } from "lodash/fp" +import { values, cloneDeep } from "lodash/fp" import { get_capitalised_name } from "../../helpers" import { backendUiStore } from "builderStore" import * as backendStoreActions from "./backend" @@ -24,8 +24,8 @@ import { saveCurrentPreviewItem as _saveCurrentPreviewItem, saveScreenApi as _saveScreenApi, regenerateCssForCurrentScreen, + generateNewIdsForComponent, } from "../storeUtils" - export const getStore = () => { const initial = { apps: [], @@ -70,6 +70,8 @@ export const getStore = () => { store.addTemplatedComponent = addTemplatedComponent(store) store.setMetadataProp = setMetadataProp(store) store.editPageOrScreen = editPageOrScreen(store) + store.pasteComponent = pasteComponent(store) + store.storeComponentForCopy = storeComponentForCopy(store) return store } @@ -454,3 +456,50 @@ const setMetadataProp = store => (name, prop) => { return s }) } + +const storeComponentForCopy = store => (component, cut = false) => { + store.update(s => { + const copiedComponent = cloneDeep(component) + s.componentToPaste = copiedComponent + s.componentToPaste.isCut = cut + if (cut) { + const parent = getParent(s.currentPreviewItem.props, component._id) + parent._children = parent._children.filter(c => c._id !== component._id) + selectComponent(s, parent) + } + + return s + }) +} + +const pasteComponent = store => (targetComponent, mode) => { + store.update(s => { + if (!s.componentToPaste) return s + + const componentToPaste = cloneDeep(s.componentToPaste) + // retain the same ids as things may be referencing this component + if (componentToPaste.isCut) { + // in case we paste a second time + s.componentToPaste.isCut = false + } else { + generateNewIdsForComponent(componentToPaste) + } + delete componentToPaste.isCut + + if (mode === "inside") { + targetComponent._children.push(componentToPaste) + return s + } + + const parent = getParent(s.currentPreviewItem.props, targetComponent) + + const targetIndex = parent._children.indexOf(targetComponent) + const index = mode === "above" ? targetIndex : targetIndex + 1 + parent._children.splice(index, 0, cloneDeep(componentToPaste)) + regenerateCssForCurrentScreen(s) + _saveCurrentPreviewItem(s) + selectComponent(s, componentToPaste) + + return s + }) +} diff --git a/packages/builder/src/builderStore/storeUtils.js b/packages/builder/src/builderStore/storeUtils.js index 63fd7ecd00..1630d118e0 100644 --- a/packages/builder/src/builderStore/storeUtils.js +++ b/packages/builder/src/builderStore/storeUtils.js @@ -1,6 +1,7 @@ import { makePropsSafe } from "components/userInterface/pagesParsing/createProps" import api from "./api" import { generate_screen_css } from "./generate_css" +import { uuid } from "./uuid" export const selectComponent = (state, component) => { const componentDef = component._component.startsWith("##") @@ -79,3 +80,8 @@ export const regenerateCssForCurrentScreen = state => { ]) return state } + +export const generateNewIdsForComponent = c => + walkProps(c, p => { + p._id = uuid() + }) diff --git a/packages/builder/src/components/userInterface/ComponentDropdownMenu.svelte b/packages/builder/src/components/userInterface/ComponentDropdownMenu.svelte index 28dd59c16f..23e28a7b48 100644 --- a/packages/builder/src/components/userInterface/ComponentDropdownMenu.svelte +++ b/packages/builder/src/components/userInterface/ComponentDropdownMenu.svelte @@ -28,8 +28,7 @@ }) $: dropdown && UIkit.util.on(dropdown, "shown", () => (hidden = false)) $: noChildrenAllowed = - !component || - getComponentDefinition($store, component._component).children === false + !component || !getComponentDefinition($store, component._component).children $: noPaste = !$store.componentToPaste const lastPartOfName = c => (c ? last(c._component.split("/")) : "") @@ -105,49 +104,14 @@ }) } - const generateNewIdsForComponent = c => - walkProps(c, p => { - p._id = uuid() - }) - const storeComponentForCopy = (cut = false) => { - store.update(s => { - const copiedComponent = cloneDeep(component) - s.componentToPaste = copiedComponent - if (cut) { - const parent = getParent(s.currentPreviewItem.props, component._id) - parent._children = parent._children.filter(c => c._id !== component._id) - selectComponent(s, parent) - } - - return s - }) + // lives in store - also used by drag drop + store.storeComponentForCopy(component, cut) } const pasteComponent = mode => { - store.update(s => { - if (!s.componentToPaste) return s - - const componentToPaste = cloneDeep(s.componentToPaste) - generateNewIdsForComponent(componentToPaste) - delete componentToPaste._cutId - - if (mode === "inside") { - component._children.push(componentToPaste) - return s - } - - const parent = getParent(s.currentPreviewItem.props, component) - - const targetIndex = parent._children.indexOf(component) - const index = mode === "above" ? targetIndex : targetIndex + 1 - parent._children.splice(index, 0, cloneDeep(componentToPaste)) - regenerateCssForCurrentScreen(s) - saveCurrentPreviewItem(s) - selectComponent(s, componentToPaste) - - return s - }) + // lives in store - also used by drag drop + store.pasteComponent(component, mode) } diff --git a/packages/builder/src/components/userInterface/ComponentsHierarchy.svelte b/packages/builder/src/components/userInterface/ComponentsHierarchy.svelte index b25d7a4e7b..db4ea2fc27 100644 --- a/packages/builder/src/components/userInterface/ComponentsHierarchy.svelte +++ b/packages/builder/src/components/userInterface/ComponentsHierarchy.svelte @@ -7,6 +7,7 @@ import { store } from "builderStore" import { ArrowDownIcon, ShapeIcon } from "components/common/Icons/" import ScreenDropdownMenu from "./ScreenDropdownMenu.svelte" + import { writable } from "svelte/store" export let screens = [] @@ -16,12 +17,15 @@ const joinPath = join("/") const normalizedName = name => - pipe(name, [ - trimCharsStart("./"), - trimCharsStart("~/"), - trimCharsStart("../"), - trimChars(" "), - ]) + pipe( + name, + [ + trimCharsStart("./"), + trimCharsStart("~/"), + trimCharsStart("../"), + trimChars(" "), + ] + ) const changeScreen = screen => { store.setCurrentScreen(screen.props._instanceName) @@ -57,7 +61,8 @@ {#if $store.currentPreviewItem.props._instanceName && $store.currentPreviewItem.props._instanceName === screen.props._instanceName && screen.props._children} + currentComponent={$store.currentComponentInfo} + dragDropStore={writable({})} /> {/if} {/each} diff --git a/packages/builder/src/components/userInterface/ComponentsHierarchyChildren.svelte b/packages/builder/src/components/userInterface/ComponentsHierarchyChildren.svelte index 691d188b07..7d24a42a8e 100644 --- a/packages/builder/src/components/userInterface/ComponentsHierarchyChildren.svelte +++ b/packages/builder/src/components/userInterface/ComponentsHierarchyChildren.svelte @@ -15,11 +15,19 @@ export let currentComponent export let onSelect = () => {} export let level = 0 + export let dragDropStore + + let dropUnderComponent + let componentToDrop const capitalise = s => s.substring(0, 1).toUpperCase() + s.substring(1) const get_name = s => (!s ? "" : last(s.split("/"))) - const get_capitalised_name = name => pipe(name, [get_name, capitalise]) + const get_capitalised_name = name => + pipe( + name, + [get_name, capitalise] + ) const isScreenslot = name => name === "##builtin/screenslot" const selectComponent = component => { @@ -32,15 +40,92 @@ // Go to correct URL $goto(`./:page/:screen/${path}`) } + + const dragstart = component => e => { + e.dataTransfer.dropEffect = "move" + dragDropStore.update(s => { + s.componentToDrop = component + return s + }) + } + + const dragover = (component, index) => e => { + const canHaveChildrenButIsEmpty = + $store.components[component._component].children && + component._children.length === 0 + + e.dataTransfer.dropEffect = "copy" + dragDropStore.update(s => { + const isBottomHalf = e.offsetY > e.currentTarget.offsetHeight / 2 + s.targetComponent = component + // only allow dropping inside when container type + // is empty. If it has children, the user can drag over + // it's existing children + if (canHaveChildrenButIsEmpty) { + if (index === 0) { + // when its the first component in the screen, + // we divide into 3, so we can paste above, inside or below + const pos = e.offsetY / e.currentTarget.offsetHeight + if (pos < 0.4) { + s.dropPosition = "above" + } else if (pos > 0.8) { + // purposely giving this the least space as it is often covered + // by the component below's "above" space + s.dropPosition = "below" + } else { + s.dropPosition = "inside" + } + } else { + s.dropPosition = isBottomHalf ? "below" : "inside" + } + } else { + s.dropPosition = isBottomHalf ? "below" : "above" + } + return s + }) + return false + } + + const drop = () => { + if ($dragDropStore.targetComponent !== $dragDropStore.componentToDrop) { + store.storeComponentForCopy($dragDropStore.componentToDrop, true) + store.pasteComponent( + $dragDropStore.targetComponent, + $dragDropStore.dropPosition + ) + } + dragDropStore.update(s => { + s.dropPosition = "" + s.targetComponent = null + s.componentToDrop = null + return s + }) + } @@ -78,6 +182,11 @@ align-items: center; } + .drop-item { + background: var(--blue-light); + height: 36px; + } + .actions { display: none; height: 24px;