Merge pull request #44822 from margelo/hanno/cleanup-1

cleanup: don't run through `containsOnlyEmojis` for an empty string
Expensify · Jul 8, 2024 · e1636b1 · e1636b1
2 parents 22a122f + 0cf6951
commit e1636b1
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/hooks/useMarkdownStyle.ts b/src/hooks/useMarkdownStyle.ts
@@ -9,7 +9,8 @@ const defaultEmptyArray: Array<keyof MarkdownStyle> = [];
 
 function useMarkdownStyle(message: string | null = null, excludeStyles: Array<keyof MarkdownStyle> = defaultEmptyArray): MarkdownStyle {
     const theme = useTheme();
-    const emojiFontSize = containsOnlyEmojis(message ?? '') ? variables.fontSizeOnlyEmojis : variables.fontSizeNormal;
+    const hasMessageOnlyEmojis = message != null && message.length > 0 && containsOnlyEmojis(message);
+    const emojiFontSize = hasMessageOnlyEmojis ? variables.fontSizeOnlyEmojis : variables.fontSizeNormal;
 
     // this map is used to reset the styles that are not needed - passing undefined value can break the native side
     const nonStylingDefaultValues: Record<string, string | number> = useMemo(