Escape more white space in appendText

Fixes #110464

src/vs/base/common/htmlContent.ts

@@ -49,7 +49,7 @@ export class MarkdownString implements IMarkdownString {
 		// escape markdown syntax tokens: http://daringfireball.net/projects/markdown/syntax#backslash
 		this.value += (this.supportThemeIcons ? escapeCodicons(value) : value)
 			.replace(/[\\`*_{}[\]()#+\-.!]/g, '\\$&')
-			.replace(/^([ \t]+)(.+)$/gm, (_match, g1, g2) => ' '.repeat(g1.length) + g2)
+			.replace(/([ \t]+)/g, (_match, g1) => ' '.repeat(g1.length))
 			.replace(/^>/gm, '\\>')
 			.replace(/\n/g, newlineStyle === MarkdownStringTextNewlineStyle.Break ? '\\\n' : '\n\n');
 

src/vs/base/test/browser/markdownRenderer.test.ts

@@ -57,7 +57,7 @@ suite('MarkdownRenderer', () => {
 			mds.appendText('$(zap) $(not a theme icon) $(add)');
 
 			let result: HTMLElement = renderMarkdown(mds);
-			assert.strictEqual(result.innerHTML, `<p>$(zap) $(not a theme icon) $(add)</p>`);
+			assert.strictEqual(result.innerHTML, `<p>$(zap)&nbsp;$(not&nbsp;a&nbsp;theme&nbsp;icon)&nbsp;$(add)</p>`);
 		});
 
 		test('render appendMarkdown', () => {
@@ -85,7 +85,7 @@ suite('MarkdownRenderer', () => {
 			mds.appendText('$(zap) $(not a theme icon) $(add)');
 
 			let result: HTMLElement = renderMarkdown(mds);
-			assert.strictEqual(result.innerHTML, `<p>$(zap) $(not a theme icon) $(add)</p>`);
+			assert.strictEqual(result.innerHTML, `<p>$(zap)&nbsp;$(not&nbsp;a&nbsp;theme&nbsp;icon)&nbsp;$(add)</p>`);
 		});
 
 		test('render appendMarkdown with escaped icon', () => {

src/vs/base/test/common/markdownString.test.ts

@@ -11,13 +11,13 @@ suite('MarkdownString', () => {
 	test('Escape leading whitespace', function () {
 		const mds = new MarkdownString();
 		mds.appendText('Hello\n    Not a code block');
-		assert.equal(mds.value, 'Hello\n\n    Not a code block');
+		assert.equal(mds.value, 'Hello\n\n    Not a code block');
 	});
 
 	test('MarkdownString.appendText doesn\'t escape quote #109040', function () {
 		const mds = new MarkdownString();
 		mds.appendText('> Text\n>More');
-		assert.equal(mds.value, '\\> Text\n\n\\>More');
+		assert.equal(mds.value, '\\> Text\n\n\\>More');
 	});
 
 	test('appendText', () => {
@@ -25,7 +25,7 @@ suite('MarkdownString', () => {
 		const mds = new MarkdownString();
 		mds.appendText('# foo\n*bar*');
 
-		assert.equal(mds.value, '\\# foo\n\n\\*bar\\*');
+		assert.equal(mds.value, '\\# foo\n\n\\*bar\\*');
 	});
 
 	suite('ThemeIcons', () => {
@@ -36,7 +36,7 @@ suite('MarkdownString', () => {
 				const mds = new MarkdownString(undefined, { supportThemeIcons: true });
 				mds.appendText('$(zap) $(not a theme icon) $(add)');
 
-				assert.equal(mds.value, '\\\\$\\(zap\\) $\\(not a theme icon\\) \\\\$\\(add\\)');
+				assert.equal(mds.value, '\\\\$\\(zap\\) $\\(not a theme icon\\) \\\\$\\(add\\)');
 			});
 
 			test('appendMarkdown', () => {
@@ -61,7 +61,7 @@ suite('MarkdownString', () => {
 				const mds = new MarkdownString(undefined, { supportThemeIcons: false });
 				mds.appendText('$(zap) $(not a theme icon) $(add)');
 
-				assert.equal(mds.value, '$\\(zap\\) $\\(not a theme icon\\) $\\(add\\)');
+				assert.equal(mds.value, '$\\(zap\\) $\\(not a theme icon\\) $\\(add\\)');
 			});
 
 			test('appendMarkdown', () => {