@portabletext/editor 1.0.15 → 1.0.17

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@portabletext/editor",
-  "version": "1.0.15",
+  "version": "1.0.17",
   "description": "Portable Text Editor made in React",
   "keywords": [
     "sanity",
@@ -51,18 +51,19 @@
   },
   "devDependencies": {
     "@jest/globals": "^29.7.0",
-    "@playwright/test": "1.46.0",
+    "@jest/types": "^29.6.3",
+    "@playwright/test": "1.46.1",
     "@portabletext/toolkit": "^2.0.15",
-    "@sanity/block-tools": "^3.54.0",
+    "@sanity/block-tools": "^3.55.0",
     "@sanity/diff-match-patch": "^3.1.1",
     "@sanity/eslint-config-i18n": "^1.1.0",
     "@sanity/eslint-config-studio": "^4.0.0",
-    "@sanity/pkg-utils": "^6.10.9",
-    "@sanity/schema": "^3.54.0",
+    "@sanity/pkg-utils": "^6.10.10",
+    "@sanity/schema": "^3.55.0",
     "@sanity/test": "0.0.1-alpha.1",
-    "@sanity/types": "^3.54.0",
+    "@sanity/types": "^3.55.0",
     "@sanity/ui": "^2.8.8",
-    "@sanity/util": "^3.54.0",
+    "@sanity/util": "^3.55.0",
     "@testing-library/dom": "^10.4.0",
     "@testing-library/react": "^16.0.0",
     "@types/debug": "^4.1.5",
@@ -74,8 +75,8 @@
     "@types/react": "^18.3.3",
     "@types/react-dom": "^18.3.0",
     "@types/ws": "~8.5.12",
-    "@typescript-eslint/eslint-plugin": "^8.1.0",
-    "@typescript-eslint/parser": "^8.1.0",
+    "@typescript-eslint/eslint-plugin": "^8.2.0",
+    "@typescript-eslint/parser": "^8.2.0",
     "@vitejs/plugin-react": "^4.3.1",
     "dotenv": "^16.4.5",
     "eslint": "^8.57.0",
@@ -84,14 +85,14 @@
     "eslint-import-resolver-typescript": "^3.6.1",
     "eslint-plugin-import": "^2.29.1",
     "eslint-plugin-prettier": "^5.2.1",
-    "eslint-plugin-react-compiler": "0.0.0-experimental-d0e920e-20240815",
+    "eslint-plugin-react-compiler": "0.0.0-experimental-eeb1b2a-20240818",
     "eslint-plugin-tsdoc": "^0.3.0",
     "eslint-plugin-unicorn": "^55.0.0",
     "eslint-plugin-unused-imports": "^4.1.3",
     "express": "^4.19.2",
     "express-ws": "^5.0.2",
     "jest": "^29.7.0",
-    "jest-dev-server": "^10.0.0",
+    "jest-dev-server": "^10.1.0",
     "jest-environment-jsdom": "^29.7.0",
     "jest-environment-node": "^29.7.0",
     "node-ipc": "npm:@node-ipc/compat@9.2.5",
@@ -101,7 +102,7 @@
     "styled-components": "^6.1.12",
     "tsx": "^4.17.0",
     "typescript": "5.5.4",
-    "vite": "^5.4.1"
+    "vite": "^5.4.2"
   },
   "peerDependencies": {
     "@sanity/block-tools": "^3.47.1",

package/src/editor/plugins/createWithEditableAPI.ts

@@ -419,16 +419,7 @@ export function createWithEditableAPI(
                 )
               }
               editor.onChange()
-              if (editor.selection) {
-                // Insert an empty string to continue writing non-annotated text
-                Transforms.insertNodes(
-                  editor,
-                  [{_type: 'span', text: '', marks: [], _key: keyGenerator()}],
-                  {
-                    at: Range.end(editor.selection),
-                  },
-                )
-              }
+
               const newPortableTextEditorSelection = toPortableTextRange(
                 fromSlateValue(editor.children, types.block.name, KEY_TO_VALUE_ELEMENT.get(editor)),
                 editor.selection,

package/src/editor/plugins/createWithPortableTextMarkModel.ts

@@ -205,15 +205,6 @@ export function createWithPortableTextMarkModel(
             }
           }
         }
-        // Empty marks if text is empty
-        if (
-          isSpan &&
-          Array.isArray(node.marks) &&
-          (!node.marks || (node.marks.length > 0 && node.text === ''))
-        ) {
-          Transforms.setNodes(editor, {marks: []}, {at: path, voids: false})
-          return
-        }
       }
       // Check consistency of markDefs (unless we are merging two nodes)
       if (
@@ -354,6 +345,18 @@ export function createWithPortableTextMarkModel(
             editor.onChange()
             return
           }
+
+          const nodeHasMarks = node.marks !== undefined && node.marks.length > 0
+
+          if (nodeHasMarks && deletingAllText) {
+            Editor.withoutNormalizing(editor, () => {
+              apply(op)
+              Transforms.setNodes(editor, {marks: []}, {at: op.path})
+            })
+
+            editor.onChange()
+            return
+          }
         }
       }