From 06aa4ba09c2fa11bdd8bdaec88008d7eb69ab6b5 Mon Sep 17 00:00:00 2001
From: Andrew Kingston <andrew@kingston.dev>
Date: Sun, 23 Jun 2024 15:08:01 +0100
Subject: [PATCH] Fix issue with pasting from multi to single cell

---
 .../grid/overlays/KeyboardManager.svelte      |  4 ++--
 .../src/components/grid/stores/clipboard.js   | 20 +++++++++----------
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/packages/frontend-core/src/components/grid/overlays/KeyboardManager.svelte b/packages/frontend-core/src/components/grid/overlays/KeyboardManager.svelte
index 3f3af0a1ac..e74cddd61b 100644
--- a/packages/frontend-core/src/components/grid/overlays/KeyboardManager.svelte
+++ b/packages/frontend-core/src/components/grid/overlays/KeyboardManager.svelte
@@ -47,9 +47,9 @@
     }
 
     // Sugar for preventing default
-    const handle = async fn => {
+    const handle = fn => {
       e.preventDefault()
-      await fn()
+      fn()
     }
 
     // Handle certain key presses regardless of selection state
diff --git a/packages/frontend-core/src/components/grid/stores/clipboard.js b/packages/frontend-core/src/components/grid/stores/clipboard.js
index 793abc9ad8..c30e53a4d5 100644
--- a/packages/frontend-core/src/components/grid/stores/clipboard.js
+++ b/packages/frontend-core/src/components/grid/stores/clipboard.js
@@ -16,18 +16,19 @@ export const deriveStores = context => {
   const { clipboard, focusedCellAPI, selectedCellCount, config } = context
 
   // Derive whether or not we're able to copy
-  const copyAllowed = derived(
-    [focusedCellAPI, selectedCellCount],
-    ([$focusedCellAPI, $selectedCellCount]) => {
-      return $focusedCellAPI || $selectedCellCount
-    }
-  )
+  const copyAllowed = derived(focusedCellAPI, $focusedCellAPI => {
+    return $focusedCellAPI != null
+  })
 
   // Derive whether or not we're able to paste
   const pasteAllowed = derived(
     [clipboard, focusedCellAPI, selectedCellCount, config],
     ([$clipboard, $focusedCellAPI, $selectedCellCount, $config]) => {
-      if ($clipboard.value == null || !$config.canEditRows) {
+      if (
+        $clipboard.value == null ||
+        !$config.canEditRows ||
+        !$focusedCellAPI
+      ) {
         return false
       }
 
@@ -40,8 +41,7 @@ export const deriveStores = context => {
       ) {
         return false
       }
-
-      return $focusedCellAPI || $selectedCellCount
+      return true
     }
   )
 
@@ -183,7 +183,7 @@ export const createActions = context => {
           get(focusedCellAPI).setValue(value[0][0])
         } else {
           // Select the new cells to paste into, then paste
-          selectedCells.actions.updateTarget(targetCellId)
+          selectedCells.actions.selectRange($focusedCellId, targetCellId)
           await pasteIntoSelectedCells(value)
         }
       }