compare with prev text value to avoid multiple applyMarkdownFormattin…

…g calls
Expensify · Sep 11, 2024 · c176a5e · c176a5e
1 parent 0dfd7b9
commit c176a5e
Showing 1 changed file with 9 additions and 15 deletions.
diff --git a/android/src/main/java/com/expensify/livemarkdown/MarkdownTextWatcher.java b/android/src/main/java/com/expensify/livemarkdown/MarkdownTextWatcher.java
@@ -8,8 +8,8 @@
 
 public class MarkdownTextWatcher implements TextWatcher {
   private final MarkdownUtils mMarkdownUtils;
-
   private boolean mShouldSkip = false;
+  private Editable mPreviousEditable;
 
   public MarkdownTextWatcher(@NonNull MarkdownUtils markdownUtils) {
     mMarkdownUtils = markdownUtils;
@@ -22,23 +22,17 @@ public void beforeTextChanged(CharSequence s, int start, int count, int after) {
 
   @Override
   public void onTextChanged(CharSequence s, int start, int before, int count) {
-    if (mShouldSkip) {
-      return;
-    }
-    // Set the flag to indicate text is being changed
-    mShouldSkip = true;
+
   }
 
   @Override
   public void afterTextChanged(Editable editable) {
-      if (!mShouldSkip) {
-          return;
-      }
-
       if (editable instanceof SpannableStringBuilder) {
+          if (mPreviousEditable != null && mPreviousEditable.toString().equals(editable.toString())) {
+              return;
+          }
           mMarkdownUtils.applyMarkdownFormatting((SpannableStringBuilder) editable);
-      }
-
-      // Reset the flag after formatting is applied
-      mShouldSkip = false;
-  }}
+          mPreviousEditable = new SpannableStringBuilder(editable);
+      }      
+  }
+}