Commit fce09f25 authored by Jose Ivan Vargas's avatar Jose Ivan Vargas

Merge branch 'ph/fixSuggestionSlashes' into 'master'

Fixes some characters being changed incorrectly in suggestions

See merge request gitlab-org/gitlab!68941
parents 5526b946 14678733
...@@ -233,7 +233,7 @@ export function insertMarkdownText({ ...@@ -233,7 +233,7 @@ export function insertMarkdownText({
} }
} else if (tag.indexOf(textPlaceholder) > -1) { } else if (tag.indexOf(textPlaceholder) > -1) {
textToInsert = tag.replace(textPlaceholder, () => textToInsert = tag.replace(textPlaceholder, () =>
selected.replace(/\\n/g, '\n').replace(/\/(n|t|r)/g, '\\$1'), selected.replace(/\\n/g, '\n').replace('%br', '\\n'),
); );
} else { } else {
textToInsert = String(startChar) + tag + selected + (wrap ? tag : ''); textToInsert = String(startChar) + tag + selected + (wrap ? tag : '');
......
...@@ -16,11 +16,7 @@ import MarkdownHeader from './header.vue'; ...@@ -16,11 +16,7 @@ import MarkdownHeader from './header.vue';
import MarkdownToolbar from './toolbar.vue'; import MarkdownToolbar from './toolbar.vue';
function cleanUpLine(content) { function cleanUpLine(content) {
return unescape( return unescape(stripHtml(content).replace(/\\n/g, '%br').replace(/\n/g, ''));
stripHtml(content)
.replace(/\\(n|t|r)/g, '/$1')
.replace(/\n/g, ''),
);
} }
export default { export default {
......
...@@ -100,7 +100,7 @@ describe('init markdown', () => { ...@@ -100,7 +100,7 @@ describe('init markdown', () => {
text: textArea.value, text: textArea.value,
tag: '```suggestion:-0+0\n{text}\n```', tag: '```suggestion:-0+0\n{text}\n```',
blockTag: true, blockTag: true,
selected: '# Does not parse the /n currently.', selected: '# Does not parse the %br currently.',
wrap: false, wrap: false,
}); });
......
...@@ -250,7 +250,7 @@ describe('Markdown field component', () => { ...@@ -250,7 +250,7 @@ describe('Markdown field component', () => {
createSubject([{ rich_text: 'hello world\\n' }]); createSubject([{ rich_text: 'hello world\\n' }]);
expect(subject.find('[data-testid="markdownHeader"]').props('lineContent')).toBe( expect(subject.find('[data-testid="markdownHeader"]').props('lineContent')).toBe(
'hello world/n', 'hello world%br',
); );
}); });
}); });
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment