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
  • Loading branch information
jasperhuangg authored Oct 2, 2024
2 parents a36e74f + d053555 commit df17d79
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 df17d79

Please sign in to comment.