chore: refactor KeySelection and ValueSelection to get rid of a c…

…ouple of internal workarounds

BREAKING CHANGE:
The API's of `KeySelection`, `ValueSelection`, `createKeySelection`, `createValueSelection`,
and `EditableValue` changed.

src/lib/components/controls/EditableDiv.svelte

@@ -13,6 +13,7 @@
   const debug = createDebug('jsoneditor:EditableDiv')
 
   export let value: string
+  export let initialValue: string | undefined
   export let shortText = false
   export let label: string
   export let onChange: (newValue: string, updateSelection: UpdateSelectionAfterChange) => void
@@ -27,17 +28,12 @@
   let closed = false
 
   onMount(() => {
-    debug('onMount', { value })
-    setDomValue(value)
+    debug('onMount', { value, initialValue })
+    setDomValue(initialValue !== undefined ? initialValue : value)
 
     // focus
     if (domValue) {
       setCursorToEnd(domValue)
-
-      // The refresh method can be used to update the classnames for example
-      // eslint-disable-next-line @typescript-eslint/ban-ts-comment
-      // @ts-ignore
-      domValue.refresh = handleValueInput
     }
   })
 

src/lib/components/modes/tablemode/TableMode.svelte

@@ -673,9 +673,7 @@
     const index = parseInt(path[0], 10)
     const nextPath = [String(index + 1), ...path.slice(1)]
 
-    return existsIn(json, nextPath)
-      ? createValueSelection(nextPath, false)
-      : createValueSelection(path, false)
+    return existsIn(json, nextPath) ? createValueSelection(nextPath) : createValueSelection(path)
   }
 
   export function focus() {
@@ -736,7 +734,7 @@
         return
       }
 
-      updateSelection(createValueSelection(path, false))
+      updateSelection(createValueSelection(path))
 
       event.preventDefault()
     }
@@ -752,7 +750,7 @@
       // Select the first row, first column
       const path = ['0', ...columns[0]]
 
-      return createValueSelection(path, false)
+      return createValueSelection(path)
     } else {
       return undefined
     }
@@ -1047,7 +1045,7 @@
     if (isObjectOrArray(value)) {
       openJSONEditorModal(path)
     } else {
-      updateSelection(createValueSelection(path, true))
+      updateSelection(createValueSelection(path))
     }
   }
 
@@ -1173,7 +1171,6 @@
     await onInsertCharacter({
       char,
       selectInside: false,
-      refJsonEditor,
       json,
       selection: selection,
       readOnly,
@@ -1453,7 +1450,7 @@
   function handleSelectValidationError(error: ValidationError) {
     debug('select validation error', error)
 
-    updateSelection(createValueSelection(error.path, false))
+    updateSelection(createValueSelection(error.path))
 
     scrollTo(error.path)
   }

src/lib/components/modes/treemode/JSONKey.svelte

@@ -3,6 +3,7 @@
 <script lang="ts">
   import { initial, isEqual } from 'lodash-es'
   import {
+    createEditKeySelection,
     createKeySelection,
     createValueSelection,
     isEditingSelection,
@@ -13,7 +14,7 @@
   import { addNewLineSuffix } from '$lib/utils/domUtils.js'
   import type { ExtendedSearchResultItem, JSONSelection, TreeModeContext } from '$lib/types.js'
   import { UpdateSelectionAfterChange } from '$lib/types.js'
-  import { parseJSONPointer, type JSONPath, type JSONPointer } from 'immutable-json-patch'
+  import { type JSONPath, type JSONPointer, parseJSONPointer } from 'immutable-json-patch'
   import ContextMenuPointer from '../../../components/controls/contextmenu/ContextMenuPointer.svelte'
   import { classnames } from '$lib/utils/cssUtils.js'
 
@@ -28,15 +29,15 @@
   let path: JSONPath
   $: path = parseJSONPointer(pointer)
 
-  $: isKeySelected = selection ? isKeySelection(selection) && isEqual(selection.path, path) : false
+  $: isKeySelected = isKeySelection(selection) && isEqual(selection.path, path)
   $: isEditingKey = isKeySelected && isEditingSelection(selection)
 
   function handleKeyDoubleClick(
     event: MouseEvent & { currentTarget: EventTarget & HTMLDivElement }
   ) {
     if (!isEditingKey && !context.readOnly) {
       event.preventDefault()
-      context.onSelect(createKeySelection(path, true))
+      context.onSelect(createEditKeySelection(path))
     }
   }
 
@@ -52,8 +53,8 @@
 
     context.onSelect(
       updateSelection === UpdateSelectionAfterChange.nextInside
-        ? createValueSelection(updatedPath, false)
-        : createKeySelection(updatedPath, false)
+        ? createValueSelection(updatedPath)
+        : createKeySelection(updatedPath)
     )
 
     if (updateSelection !== UpdateSelectionAfterChange.self) {
@@ -62,14 +63,15 @@
   }
 
   function handleCancelChange() {
-    context.onSelect(createKeySelection(path, false))
+    context.onSelect(createKeySelection(path))
     context.focus()
   }
 </script>
 
 {#if !context.readOnly && isEditingKey}
   <EditableDiv
     value={context.normalization.escapeValue(key)}
+    initialValue={isEditingSelection(selection) ? selection.initialValue : undefined}
     label="Edit key"
     shortText
     onChange={handleChangeValue}

src/lib/components/modes/treemode/TreeMode.svelte

@@ -42,8 +42,9 @@
   import {
     canConvert,
     createAfterSelection,
+    createEditKeySelection,
+    createEditValueSelection,
     createInsideSelection,
-    createKeySelection,
     createSelectionFromOperations,
     createValueSelection,
     findRootPath,
@@ -312,7 +313,7 @@
   function handleSelectValidationError(error: ValidationError) {
     debug('select validation error', error)
 
-    updateSelection(createValueSelection(error.path, false))
+    updateSelection(createValueSelection(error.path))
     scrollTo(error.path)
   }
 
@@ -578,7 +579,7 @@
   function createDefaultSelection() {
     debug('createDefaultSelection')
 
-    updateSelection(createValueSelection([], false))
+    updateSelection(createValueSelection([]))
   }
 
   export function patch(
@@ -661,7 +662,7 @@
       return
     }
 
-    updateSelection(createKeySelection(getFocusPath(selection), true))
+    updateSelection(createEditKeySelection(getFocusPath(selection)))
   }
 
   function handleEditValue() {
@@ -674,7 +675,7 @@
     if (isObjectOrArray(value)) {
       openJSONEditorModal(path, value)
     } else {
-      updateSelection(createValueSelection(path, true))
+      updateSelection(createEditValueSelection(path))
     }
   }
 
@@ -834,7 +835,7 @@
     onInsert({
       insertType,
       selectInside: true,
-      refJsonEditor,
+      initialValue: undefined,
       json,
       selection,
       readOnly,
@@ -847,7 +848,7 @@
   function handleInsertFromContextMenu(type: InsertType) {
     if (isKeySelection(selection)) {
       // in this case, we do not want to rename the key, but replace the property
-      updateSelection(createValueSelection(selection.path, false))
+      updateSelection(createValueSelection(selection.path))
     }
 
     if (!selection) {
@@ -940,7 +941,6 @@
     await onInsertCharacter({
       char,
       selectInside: true,
-      refJsonEditor,
       json,
       selection,
       readOnly,
@@ -1054,7 +1054,7 @@
         handlePatch(operations, (patchedJson, patchedState) => ({
           // expand the newly replaced array and select it
           state: expandSmart(patchedJson, patchedState, rootPath),
-          selection: createValueSelection(rootPath, false)
+          selection: createValueSelection(rootPath)
         }))
       },
       onClose: () => {
@@ -1108,7 +1108,7 @@
           handlePatch(operations, (patchedJson, patchedState) => ({
             // expand the newly replaced array and select it
             state: expandSmart(patchedJson, patchedState, rootPath),
-            selection: createValueSelection(rootPath, false)
+            selection: createValueSelection(rootPath)
           }))
         }
       },
@@ -1514,7 +1514,7 @@
         const parent = getIn(json, initial(path))
         if (Array.isArray(parent)) {
           // change into selection of the value
-          updateSelection(createValueSelection(path, false))
+          updateSelection(createValueSelection(path))
         }
       }