From 48cba5493a650bee436c120e0d780ed548962bed Mon Sep 17 00:00:00 2001 From: Armando Guarino Date: Tue, 29 Oct 2024 13:49:21 +0100 Subject: [PATCH] fix: preserve attributes of set node --- .changeset/chatty-pianos-learn.md | 5 +++++ packages/core/src/commands/setNode.ts | 11 +++++++++-- 2 files changed, 14 insertions(+), 2 deletions(-) create mode 100644 .changeset/chatty-pianos-learn.md diff --git a/.changeset/chatty-pianos-learn.md b/.changeset/chatty-pianos-learn.md new file mode 100644 index 000000000..201b5076c --- /dev/null +++ b/.changeset/chatty-pianos-learn.md @@ -0,0 +1,5 @@ +--- +"@tiptap/core": patch +--- + +preserve existing node attributes when running setNode diff --git a/packages/core/src/commands/setNode.ts b/packages/core/src/commands/setNode.ts index b48e1a376..d3828d05f 100644 --- a/packages/core/src/commands/setNode.ts +++ b/packages/core/src/commands/setNode.ts @@ -21,6 +21,13 @@ declare module '@tiptap/core' { export const setNode: RawCommands['setNode'] = (typeOrName, attributes = {}) => ({ state, dispatch, chain }) => { const type = getNodeType(typeOrName, state.schema) + let attributesToCopy: Record | undefined + + if (state.selection.$anchor.sameParent(state.selection.$head)) { + // only copy attributes if the selection is pointing to a node of the same type + attributesToCopy = state.selection.$anchor.parent.attrs + } + // TODO: use a fallback like insertContent? if (!type.isTextblock) { console.warn('[tiptap warn]: Currently "setNode()" only supports text block nodes.') @@ -32,7 +39,7 @@ export const setNode: RawCommands['setNode'] = (typeOrName, attributes = {}) => chain() // try to convert node to default node if needed .command(({ commands }) => { - const canSetBlock = setBlockType(type, attributes)(state) + const canSetBlock = setBlockType(type, { ...attributesToCopy, ...attributes })(state) if (canSetBlock) { return true @@ -41,7 +48,7 @@ export const setNode: RawCommands['setNode'] = (typeOrName, attributes = {}) => return commands.clearNodes() }) .command(({ state: updatedState }) => { - return setBlockType(type, attributes)(updatedState, dispatch) + return setBlockType(type, { ...attributesToCopy, ...attributes })(updatedState, dispatch) }) .run() )