fix(mention): No extra whitespace in markdown before/after mentions

Fixes: #6314 Signed-off-by: Jonas <jonas@freesources.org>
nextcloud · Sep 4, 2024 · d64187c · d64187c
1 parent 496c37e
commit d64187c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/extensions/Mention.js b/src/extensions/Mention.js
@@ -40,8 +40,6 @@ export default TipTapMention.extend({
 	},
 
 	toMarkdown(state, node) {
-		state.write(' ')
 		state.write(`@[${node.attrs.label}](mention://user/${encodeURIComponent(node.attrs.id)})`)
-		state.write(' ')
 	},
 })
diff --git a/src/tests/markdown.spec.js b/src/tests/markdown.spec.js
@@ -123,8 +123,9 @@ describe('Markdown though editor', () => {
 		expect(markdownThroughEditor('---\n---')).toBe('---\n---')
 	})
 
-	test('mentions', () => {
-		expect(markdownThroughEditor('@[username](mention://user/id)')).toBe(' @[username](mention://user/id) ')
+	test.only('mentions', () => {
+		expect(markdownThroughEditor('@[username](mention://user/id)')).toBe('@[username](mention://user/id)')
+		expect(markdownThroughEditor('pretext @[username](mention://user/id) posttext')).toBe('pretext @[username](mention://user/id) posttext')
 	})
 
 	test('details', () => {
@@ -196,9 +197,10 @@ describe('Markdown serializer from html', () => {
 		expect(markdownThroughEditorHtml('<pre id="frontmatter"><code>---</code></pre><h1>Heading</h1>')).toBe('----\n---\n----\n\n# Heading')
 	})
 
-	test('mentions', () => {
-		expect(markdownThroughEditorHtml('<span class="mention" data-label="username" data-type="user" data-id="id">username</span>')).toBe(' @[username](mention://user/id) ')
-		expect(markdownThroughEditorHtml('<span class="mention" data-label="whitespace user" data-type="user" data-id="whitespace user">whitespace user</span>')).toBe(' @[whitespace user](mention://user/whitespace%20user) ')
+	test.only('mentions', () => {
+		expect(markdownThroughEditorHtml('<span class="mention" data-label="username" data-type="user" data-id="id">username</span>')).toBe('@[username](mention://user/id)')
+		expect(markdownThroughEditorHtml('<span class="mention" data-label="whitespace user" data-type="user" data-id="whitespace user">whitespace user</span>')).toBe('@[whitespace user](mention://user/whitespace%20user)')
+		expect(markdownThroughEditorHtml('pretext <span class="mention" data-label="username" data-type="user" data-id="id">username</span> posttext')).toBe('pretext @[username](mention://user/id) posttext')
 	})
 
 	test('details', () => {