Skip to content

Commit

Permalink
Merge pull request #50119 from Expensify/jasper-fixEmojiIndexing
Browse files Browse the repository at this point in the history
Fix: Yellow color tones not applied for emoji
(cherry picked from commit df17d79)

(CP triggered by jasperhuangg)
  • Loading branch information
jasperhuangg authored and OSBotify committed Oct 2, 2024
1 parent bca1826 commit 8a2b06d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/libs/EmojiUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -462,7 +462,7 @@ const getPreferredSkinToneIndex = (value: OnyxEntry<string | number>): number =>
*/
const getPreferredEmojiCode = (emoji: Emoji, preferredSkinTone: OnyxEntry<string | number>): string => {
if (emoji.types && typeof preferredSkinTone === 'number') {
const emojiCodeWithSkinTone = emoji.types.at(preferredSkinTone);
const emojiCodeWithSkinTone = preferredSkinTone >= 0 ? emoji.types.at(preferredSkinTone) : undefined;

// Note: it can happen that preferredSkinTone has a outdated format,
// so it makes sense to check if we actually got a valid emoji code back
Expand Down

0 comments on commit 8a2b06d

Please sign in to comment.