From 1a210e3ee63155093b8044f2bb3bca86ee64b09f Mon Sep 17 00:00:00 2001
From: Andrew Kingston <andrew@kingston.dev>
Date: Fri, 14 Oct 2022 18:24:47 +0100
Subject: [PATCH] Fix issues dropping components from new component panel

---
 .../client/src/components/preview/DNDHandler.svelte |  1 +
 packages/client/src/stores/dnd.js                   | 13 ++++---------
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/packages/client/src/components/preview/DNDHandler.svelte b/packages/client/src/components/preview/DNDHandler.svelte
index 392cdceb36..61b34eefd3 100644
--- a/packages/client/src/components/preview/DNDHandler.svelte
+++ b/packages/client/src/components/preview/DNDHandler.svelte
@@ -245,6 +245,7 @@
         drop.parent,
         drop.index
       )
+      return
     }
 
     // Convert parent + index into target + mode
diff --git a/packages/client/src/stores/dnd.js b/packages/client/src/stores/dnd.js
index 1e7d7a6d50..cebdb7c6ae 100644
--- a/packages/client/src/stores/dnd.js
+++ b/packages/client/src/stores/dnd.js
@@ -2,10 +2,6 @@ import { writable, derived } from "svelte/store"
 
 const createDndStore = () => {
   const initialState = {
-    // Flags for whether we are inserting a new component or not
-    isNewComponent: false,
-    newComponentType: null,
-
     // Info about the dragged component
     source: null,
 
@@ -24,8 +20,8 @@ const createDndStore = () => {
     })
   }
 
-  const startDraggingNewComponent = ({ type, definition }) => {
-    if (!type || !definition) {
+  const startDraggingNewComponent = ({ component, definition }) => {
+    if (!component || !definition) {
       return
     }
 
@@ -35,13 +31,12 @@ const createDndStore = () => {
 
     store.set({
       ...initialState,
-      isNewComponent: true,
-      newComponentType: type,
       source: {
         id: null,
         parent: null,
         bounds: { height, width },
         index: null,
+        newComponentType: component,
       },
     })
   }
@@ -91,5 +86,5 @@ export const dndBounds = derived(
 )
 export const dndIsNewComponent = derived(
   dndStore,
-  $dndStore => $dndStore.isNewComponent
+  $dndStore => $dndStore.source?.newComponentType != null
 )