Prevent italic or strikethrough emojis on Android

android/src/main/java/com/expensify/livemarkdown/MarkdownRange.java

@@ -0,0 +1,21 @@
+package com.expensify.livemarkdown;
+
+import org.json.JSONException;
+import org.json.JSONObject;
+
+public class MarkdownRange {
+  public final String type;
+  public final int start;
+  public final int end;
+  public final int length;
+  public final int depth;
+
+  public MarkdownRange(String type, int start, int length, int depth) {
+    this.type = type;
+    this.start = start;
+    this.length = length;
+    this.end = start + length;
+    this.depth = depth;
+  }
+}
+

android/src/main/java/com/expensify/livemarkdown/MarkdownUtils.java

@@ -16,7 +16,10 @@
 
 import java.io.IOException;
 import java.io.InputStream;
+import java.util.ArrayList;
+import java.util.List;
 import java.util.Objects;
+import java.util.TreeMap;
 
 public class MarkdownUtils {
   static {
@@ -66,37 +69,83 @@ public void setMarkdownStyle(@NonNull MarkdownStyle markdownStyle) {
     mMarkdownStyle = markdownStyle;
   }
 
-  public void applyMarkdownFormatting(SpannableStringBuilder ssb) {
-    Objects.requireNonNull(mMarkdownStyle, "mMarkdownStyle is null");
+  private void splitRangesOnEmojis(List<MarkdownRange> markdownRanges, String type) {
+    List<MarkdownRange> emojiRanges = new ArrayList<>();
+    for (MarkdownRange range : markdownRanges) {
+      if (range.type.equals("emoji")) {
+        emojiRanges.add(range);
+      }
+    }
 
-    removeSpans(ssb);
+    int i = 0;
+    int j = 0;
+    while (i < markdownRanges.size() && j < emojiRanges.size()) {
+      MarkdownRange currentRange = markdownRanges.get(i);
+      MarkdownRange emojiRange = emojiRanges.get(j);
 
-    String input = ssb.toString();
-    String output;
-    if (input.equals(mPrevInput)) {
-      output = mPrevOutput;
-    } else {
-      output = parseMarkdown(input);
-      mPrevInput = input;
-      mPrevOutput = output;
+      if (!currentRange.type.equals(type) || currentRange.end < emojiRange.start) {
+          i += 1;
+          continue;
+      } else if (emojiRange.start >= currentRange.start && emojiRange.end <= currentRange.end) {
+        // Split range
+        MarkdownRange startRange = new MarkdownRange(currentRange.type, currentRange.start, emojiRange.start - currentRange.start, currentRange.depth);
+        MarkdownRange endRange = new MarkdownRange(currentRange.type, emojiRange.end, currentRange.end - emojiRange.end, currentRange.depth);
+
+        markdownRanges.add(i + 1, startRange);
+        markdownRanges.add(i + 2, endRange);
+        markdownRanges.remove(i);
+        i = i + 1;
+      }
+      j += 1;
     }
+  }
+
 
+  private List<MarkdownRange> parseRanges(String rangesJSON, String innerText) {
+    List<MarkdownRange> markdownRanges = new ArrayList<>();
     try {
-      JSONArray ranges = new JSONArray(output);
+      JSONArray ranges = new JSONArray(rangesJSON);
       for (int i = 0; i < ranges.length(); i++) {
         JSONObject range = ranges.getJSONObject(i);
         String type = range.getString("type");
         int start = range.getInt("start");
         int length = range.getInt("length");
         int depth = range.optInt("depth", 1);
-        int end = start + length;
-        if (length == 0 || end > input.length()) {
+
+        MarkdownRange markdownRange = new MarkdownRange(type, start, length, depth);
+        if (markdownRange.length == 0 || markdownRange.end > innerText.length()) {
           continue;
         }
-        applyRange(ssb, type, start, end, depth);
+        markdownRanges.add(markdownRange);
       }
     } catch (JSONException e) {
-      // Do nothing
+      return new ArrayList<>();
+    }
+    splitRangesOnEmojis(markdownRanges, "italic");
+    splitRangesOnEmojis(markdownRanges, "strikethrough");
+    return markdownRanges;
+  }
+
+
+
+  public void applyMarkdownFormatting(SpannableStringBuilder ssb) {
+    Objects.requireNonNull(mMarkdownStyle, "mMarkdownStyle is null");
+
+    removeSpans(ssb);
+
+    String input = ssb.toString();
+    String output;
+    if (input.equals(mPrevInput)) {
+      output = mPrevOutput;
+    } else {
+      output = parseMarkdown(input);
+      mPrevInput = input;
+      mPrevOutput = output;
+    }
+
+    List<MarkdownRange> ranges = parseRanges(output, input);
+    for (MarkdownRange range : ranges) {
+      applyRange(ssb, range.type, range.start, range.end, range.depth);
     }
   }
 

parser/index.ts

@@ -241,6 +241,8 @@ function getTagPriority(tag: string) {
       return 2;
     case 'h1':
       return 1;
+    case 'emoji':
+      return -1;
     default:
       return 0;
   }

src/web/utils/blockUtils.ts

@@ -26,7 +26,13 @@ function addStyleToBlock(targetElement: HTMLElement, type: NodeType, markdownSty
       node.style.textDecoration = 'line-through';
       break;
     case 'emoji':
-      Object.assign(node.style, {...markdownStyle.emoji, verticalAlign: 'middle'});
+      Object.assign(node.style, {
+        ...markdownStyle.emoji,
+        verticalAlign: 'middle',
+        fontStyle: 'normal',
+        textDecoration: 'none',
+        display: 'inline-block',
+      });
       break;
     case 'mention-here':
       Object.assign(node.style, markdownStyle.mentionHere);
@@ -49,7 +55,6 @@ function addStyleToBlock(targetElement: HTMLElement, type: NodeType, markdownSty
     case 'pre':
       Object.assign(node.style, markdownStyle.pre);
       break;
-
     case 'blockquote':
       Object.assign(node.style, {
         ...markdownStyle.blockquote,