Add support for inline code with space only character as content

Signed-off-by: Tsaqif <tsaiinkwa@yahoo.com>

__tests__/ExpensiMark-HTML-test.js

@@ -79,7 +79,7 @@ test('Test italic markdown replacement', () => {
     const italicTestStartString = 'Note that _this is correctly italicized_\n' + 'Note that `_this is correctly not italicized_` and _following inline contents are italicized_';
 
     const italicTestReplacedString =
-        'Note that <em>this is correctly italicized</em><br />' + 'Note that <code>_this is correctly not italicized_</code> and <em>following inline contents are italicized</em>';
+        'Note that <em>this is correctly italicized</em><br />' + 'Note that <code>_this&nbsp;is&nbsp;correctly&nbsp;not&nbsp;italicized_</code> and <em>following inline contents are italicized</em>';
 
     expect(parser.replace(italicTestStartString)).toBe(italicTestReplacedString);
 });
@@ -402,7 +402,7 @@ test('Test critical markdown style links', () => {
         '<a href="https://necolas.github.io/react-native-web/docs/?path=/docs/components-pressable--disabled" target="_blank" rel="noreferrer noopener">second no http://</a> ' +
         '<a href="https://github.com/Expensify/Expensify.cash/issues/123#:~:text=Please%20work/Expensify.cash" target="_blank" rel="noreferrer noopener">third</a> ' +
         '<a href="https://github.com/Expensify/Expensify.cash/issues/123#:~:text=Please%20work/Expensify.cash" target="_blank" rel="noreferrer noopener">third no https://</a> ' +
-        '<a href="https://google.com" target="_blank" rel="noreferrer noopener">link <code>[inside another link](https://google.com)</code></a> ' +
+        '<a href="https://google.com" target="_blank" rel="noreferrer noopener">link <code>[inside&nbsp;another&nbsp;link](https://google.com)</code></a> ' +
         '<a href="https://google.com" target="_blank" rel="noreferrer noopener">link with an @ in it</a> ' +
         '<a href="https://google.com" target="_blank" rel="noreferrer noopener">link with [brackets] inside of it</a> ' +
         '<a href="https://google.com" target="_blank" rel="noreferrer noopener">link with smart quotes ‘’“”</a> ' +
@@ -499,7 +499,7 @@ test('Test inline code with multiple backtick symbols as content', () => {
 
 test('Test inline code blocks with ExpensiMark syntax inside', () => {
     const inlineCodeStartString = '`This is how you can write ~strikethrough~, *bold*, and _italics_`';
-    expect(parser.replace(inlineCodeStartString)).toBe('<code>This is how you can write ~strikethrough~, *bold*, and _italics_</code>');
+    expect(parser.replace(inlineCodeStartString)).toBe('<code>This&nbsp;is&nbsp;how&nbsp;you&nbsp;can&nbsp;write&nbsp;~strikethrough~,&nbsp;*bold*,&nbsp;and&nbsp;_italics_</code>');
 });
 
 test('Test inline code blocks inside ExpensiMark', () => {
@@ -1243,17 +1243,24 @@ test('Test for backticks with complete escaped backtick characters inside it', (
     expect(parser.replace(testString)).toBe(resultString);
 });
 
-// Backticks with no content are not replaced with <code>
-test('Test for backticks with no content', () => {
-    const testString = '`   `';
-    const resultString = '&#x60;   &#x60;';
+// Backticks with only tab characters inside it are not replaced with <code>
+test('Test for backticks only tab characters inside it', () => {
+    const testString = '`\u0009`';
+    const resultString = '&#x60;\u0009&#x60;';
     expect(parser.replace(testString)).toBe(resultString);
 });
 
-// Code-fence with no content is not replaced with <pre>
-test('Test for codefence with no content', () => {
+// Backticks with only space characters are replaced with <code>
+test('Test for backticks with only space characters as content', () => {
+    const testString = '`  `';
+    const resultString = '<code>&nbsp;&nbsp;</code>';
+    expect(parser.replace(testString)).toBe(resultString);
+});
+
+// Code-fence with spaces as content
+test('Test for inline code block with triple backtick with spaces as content', () => {
     const testString = '```   ```';
-    const resultString = '&#x60;&#x60;&#x60;   &#x60;&#x60;&#x60;';
+    const resultString = '<code>&#x60;&#x60;&nbsp;&nbsp;&nbsp;&#x60;&#x60;</code>';
     expect(parser.replace(testString)).toBe(resultString);
 });
 
@@ -1410,7 +1417,7 @@ test('Test for user mention with text with codefence style', () => {
 
 test('Test for user mention with text with inlineCodeBlock style', () => {
     const testString = '`hi @username@expensify.com`';
-    const resultString = '<code>hi @username@expensify.com</code>';
+    const resultString = '<code>hi&nbsp;@username@expensify.com</code>';
     expect(parser.replace(testString)).toBe(resultString);
 });
 

lib/ExpensiMark.ts

@@ -190,8 +190,12 @@ export default class ExpensiMark {
                 // Use the url escaped version of a backtick (`) symbol. Mobile platforms do not support lookbehinds,
                 // so capture the first and third group and place them in the replacement.
                 // but we should not replace backtick symbols if they include <pre> tags between them.
-                regex: /(\B|_|)&#x60;((?:&#x60;)*)(?!&#x60;)(.*?\S+?.*?)(?<!&#x60;)((?:&#x60;)*)(&#x60;)(\B|_|)(?!&#x60;|[^<]*<\/pre>|[^<]*<\/video>)/gm,
-                replacement: '$1<code>$2$3$4</code>$6',
+                // At least one non-whitespace character or a specific whitespace character (" " and "\u00A0") 
+                // must be present inside the backticks.
+                regex: /(\B|_|)&#x60;((?:&#x60;)*(?!&#x60;).*?[\S| |\u00A0]+?.*?(?<!&#x60;)(?:&#x60;)*)&#x60;(\B|_|)(?!&#x60;|[^<]*<\/pre>|[^<]*<\/video>)/gm,
+                replacement: (_extras, _match, g1, g2, g3) => {
+                    return `${g1}<code>${g2.replaceAll(' ', '&nbsp;')}</code>${g3}`;
+                },
             },
 
             /**