Merge branch 'markdown-toolbar-spaces' into 'master'

Remove spaces from markdown toolbar items

Closes #216612

See merge request gitlab-org/gitlab!31288

app/assets/javascripts/lib/utils/text_markdown.js

@@ -225,7 +225,7 @@ export function insertMarkdownText({
   } else if (tag.indexOf(textPlaceholder) > -1) {
     textToInsert = tag.replace(textPlaceholder, selected);
   } else {
-    textToInsert = String(startChar) + tag + selected + (wrap ? tag : ' ');
+    textToInsert = String(startChar) + tag + selected + (wrap ? tag : '');
   }
 
   if (removedFirstNewLine) {

changelogs/unreleased/markdown-toolbar-spaces.yml

+---
+title: Remove extra spaces from markdown toolbar items
+merge_request: 31288
+author:
+type: other

spec/frontend/lib/utils/text_markdown_spec.js

@@ -25,7 +25,7 @@ describe('init markdown', () => {
         insertMarkdownText({
           textArea,
           text: textArea.value,
-          tag: '*',
+          tag: '* ',
           blockTag: null,
           selected: '',
           wrap: false,
@@ -43,7 +43,7 @@ describe('init markdown', () => {
         insertMarkdownText({
           textArea,
           text: textArea.value,
-          tag: '*',
+          tag: '* ',
           blockTag: null,
           selected: '',
           wrap: false,
@@ -61,7 +61,7 @@ describe('init markdown', () => {
         insertMarkdownText({
           textArea,
           text: textArea.value,
-          tag: '*',
+          tag: '* ',
           blockTag: null,
           selected: '',
           wrap: false,
@@ -79,7 +79,7 @@ describe('init markdown', () => {
         insertMarkdownText({
           textArea,
           text: textArea.value,
-          tag: '*',
+          tag: '* ',
           blockTag: null,
           selected: '',
           wrap: false,

spec/frontend/vue_shared/components/markdown/field_spec.js

@@ -185,7 +185,7 @@ describe('Markdown field component', () => {
         markdownButton.trigger('click');
 
         return wrapper.vm.$nextTick(() => {
-          expect(textarea.value).toContain('*  testing');
+          expect(textarea.value).toContain('* testing');
         });
       });