Web - Overlapping Emojis when Changing Default Skin Tone for "CoupleKiss" Emoji

[kbecciv]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Action Performed:

1. Open a chat and type a message.
2. Hover over the message and click "Add reaction."
3. Search for "coupleKiss" emoji in the search bar.
4. Change the default skin tone for the "coupleKiss" emoji by clicking on the hand icons at the bottom of the Emoji picker.

Expected Result:

The emojis should not overlap when changing the default skin tone for the "coupleKiss" emoji.

Actual Result:

Some emojis overlap when changing the default skin tone for the "coupleKiss" emoji, causing visual issues.

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

• [ ] Android / native

• [ ] Android / Chrome

• [ ] iOS / native

• [ ] iOS / Safari

• [ ] MacOS / Chrome / Safari

• [ ] MacOS / Desktop

• [X] Edition Windows 10 Pro Version 22H2 Installed on ‎8/‎16/‎2021 OS build 19045.3086 Experience Windows Feature Experience Pack 1000.19041.1000.0

Version Number: v1.3.33-4 Reproducible in staging?: y Reproducible in production?: y If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Notes/Photos/Videos: Any additional supporting documentation

https://github.com/Expensify/App/assets/93399543/ddb26d92-42b2-46d4-9005-f186f7a54297

https://github.com/Expensify/App/assets/93399543/d099a7bf-6525-45ab-aec1-b8208c6c9737

Expensify/Expensify Issue URL: Issue reported by: @tewodrosGirmaA Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1687688275769119

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~013e72f32679d51f10
• Upwork Job ID: 1674063590840524800
• Last Price Increase: 2023-06-28

[melvin-bot[bot]]

Triggered auto assignment to @Christinadobrzyn (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[melvin-bot[bot]]

Bug0 Triage Checklist (Main S/O)

• [ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
• [x] This bug is not a duplicate report (check E/App issues and #expensify-bugs)
  • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
• [x] This bug is reproducible using the reproduction steps in the OP. S/O
  • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
  • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
• [x] This issue is filled out as thoroughly and clearly as possible
  • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
• [x] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

[Christinadobrzyn]

I can't reproduce this on MacOS / Chrome / Safari, asking if it's still happening - https://expensify.slack.com/archives/C049HHMV9SM/p1687905235179519?thread_ts=1687688275.769119&cid=C049HHMV9SM

[tewodrosGirmaA]

[tienifr]

This is reproducible on Windows.

[Christinadobrzyn]

Hey @tienifr do we test on windows?

I can't reproduce on Android, Chrome/Safari, iOS, Mac. I think that's all we test on, right?

[tienifr]

Yeah, I think the root cause is about the font family. The fall-back font on Windows is Segoe UI Emoji. This font may not have the black couplekiss emoji. That's my guess. Wait internal confirmation whether this is a bug.

[melvin-bot[bot]]

Job added to Upwork: https://www.upwork.com/jobs/~013e72f32679d51f10

[melvin-bot[bot]]

Triggered auto assignment to Contributor Plus for review of internal employee PR - @eVoloshchak (Internal)

[Christinadobrzyn]

I don't think we test on Windows products at this time so I don't think this needs to be fixed but I'll add a C+ to confirm.

Making this Internal until we determine if this should be fixed. If yes, i can make it external.

[Christinadobrzyn]

@eVoloshchak just nudging this, should we fix this windows emoji issue? only affecting windows?

[eVoloshchak]

@Christinadobrzyn, I personally think we should fix Windows, since I suspect most people access NewDot from Windows (no data to back this up, but Windows is the most popular OS). In addition to that, I remember a couple of issues in the past that were fixing Windows-specific issues. That was a while back, not sure if our policy (if there is any) has changed

[Christinadobrzyn]

Asking what version of windows was used to test this so I can do some testing - https://expensify.slack.com/archives/C049HHMV9SM/p1687974216623249?thread_ts=1687688275.769119&cid=C049HHMV9SM

[Christinadobrzyn]

I don't have any method to test this on a Windows 10 Pro - asking the team if anyone can test.

https://expensify.slack.com/archives/C01SKUP7QR0/p1688409525853629

@eVoloshchak are you able to reproduce on a Windows 10 Pro?

[Christinadobrzyn]

It sounds like Windows PC is out of scope since we can't test on it - based on this Slack chat - https://expensify.slack.com/archives/C01SKUP7QR0/p1688409525853629

So closing this for now. feel free to reopen if we start working on Windows PC issues!