[$250] Android - Chat - Changing skin tone and selecting emoji moves cursor to left

[lanitochka17]

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

---

Version Number: 1.4.63
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team

Action Performed:

1. Launch app
2. Tap on a report
3. Enter :+1 and then click emoji picker
4. Change the skin tone
5. Select the emoji
6. Note the cursor position in compose box

Expected Result:

Changing skin tone and selecting emoji must not move cursor to left

Actual Result:

Changing skin tone and selecting emoji moves cursor to left

Workaround:

Unknown

Platforms:

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

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6454236_1713471923855.curson.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~0198233b222f3033ae
• Upwork Job ID: 1782441601731379200
• Last Price Increase: 2024-04-22

[melvin-bot]

Triggered auto assignment to @tgolen (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

[github-actions]

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[lanitochka17]

@tgolen FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

[lanitochka17]

We think that this bug might be related to #vip-vsp

[lanitochka17]

Production:

Record_2024-04-18-22-55-08_4f9154176b47c00da84e32064abf1c48.mp4

[tgolen]

I don't think this needs to be a blocker, and I think it can be fixed by contributors. I suspect it could have something to do with #39597

@Skalakid would you mind checking to see if it relates to your PR or not?

[Skalakid]

Hi, @tgolen I was able to reproduce this issue. I think there is a problem inside react-native-live-markdown implementation for Android and I prepared a small fix for it

[melvin-bot]

Triggered auto assignment to @garrettmknight (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~0198233b222f3033ae

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @paultsimura (External)

[tgolen]

@Skalakid is this a regression from your PR or is it a separate fix?

bump @Skalakid

[Skalakid]

@tgolen Sorry, I totally missed the notification. This is a separate fix, since we didn't catch bug with cursor positioning on Android when changing text and style simultaneously

[tgolen]

OK, thanks. I'm going to add the C+ and BZ people back onto this issue then and you can attach your fix to this issue.

[melvin-bot]

Current assignee @garrettmknight is eligible for the Bug assigner, not assigning anyone new.

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (External)

[parasharrajat]

No PR yet.

[Skalakid]

The PR with the fix was already merged to the react-native-live-markdown library. This PR will bump markdown version to the latest in E/App

[melvin-bot]

@tgolen, @garrettmknight, @parasharrajat, @Skalakid Whoops! This issue is 2 days overdue. Let's get this updated quick!

[garrettmknight]

Are we still waiting on #41103 for this one?

[melvin-bot]

@tgolen @garrettmknight @parasharrajat @Skalakid this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

[tgolen]

We are still waiting on #41103 for this, yeah.

[Skalakid]

The react-native-live-markdown version has been bumped in other PR and merged, so the issue should be fixed in E/App

[tgolen]

OK, so can this GH be closed out then?

[Skalakid]

@tgolen since it is fixed I think yes 👍

[melvin-bot]

@tgolen @garrettmknight Be sure to fill out the Contact List!