[HOLD for payment 2024-05-03] [$250] Chat - Text in the composer is highlighted after adding an emoji to text with mark down

[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!

---

Version Number: 1.4.58-0
Reproducible in staging?: y
Reproducible in production?: n
Issue reported by: Applause - Internal Team

Action Performed:

1. Go to https://staging.new.expensify.com/
2. Go to any chat.
3. Enter a text in bold mark down.
4. Create a new line.
5. Open emoji picker.
6. Select an emoji.

Expected Result:

The text in the composer will not be highlighted.

Actual Result:

The entire text in the composer is highlighted after adding an emoji to text with mark down.

Workaround:

n/a

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

bandicam.2024-03-29.11-21-01-144.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~012f7f1b9abead474c
• Upwork Job ID: 1776300079521869824
• Last Price Increase: 2024-04-12
• Automatic offers:
• eh2077 | Reviewer | 0

[melvin-bot]

Triggered auto assignment to @cristipaval (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.

[kbecciv]

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

[thienlnam]

Not a blocker - added to the tracking issue

[thienlnam]

Opening this to external contributors!

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~012f7f1b9abead474c

[melvin-bot]

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

[MrMuzyk]

I am Michał from Callstack - expert contributor group. I’d like to work on this job.

[MrMuzyk]

@kbecciv Hey, can you try to reproduce this again? I wasn't able to do that.

39237_no_repro.mp4

[kbecciv]

@MrMuzyk Checking

[melvin-bot]

@thienlnam, @eh2077 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[melvin-bot]

📣 @eh2077 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job

[melvin-bot]

@tomekzaw, @thienlnam, @eh2077 Eep! 4 days overdue now. Issues have feelings too...

[Skalakid]

Hi, to confirm my Expensify/react-native-live-markdown#313 in react-native-live-markdown library, fixes this issue. I will create the PR that bumps the version of the library in our App

[Skalakid]

Here is the PR with the fix

[MrMuzyk]

I've checked on latest main and issue is no longer reproducible. @kbecciv Can you confirm that?

selection_after_emoji_fixed.mp4

[kavimuru]

@MrMuzyk Issue is fixed now.

[eh2077]

Hi, to confirm my Expensify/react-native-live-markdown#313 in react-native-live-markdown library, fixes this issue. I will create the PR that bumps the version of the library in our App

Yes, this is fixed by @Skalakid 's PR #40923 which was merged yesterday

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.66-5 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Fix text highlighting and cursor position in Live Markdown input on web #40923

If no regressions arise, payment will be issued on 2024-05-03. 🎊

For reference, here are some details about the assignees on this issue:

• @tomekzaw does not require payment (Contractor)
• @eh2077 requires payment automatic offer (Reviewer)

[melvin-bot]

Issue is ready for payment but no BZ is assigned. @laurenreidexpensify you are the lucky winner! Please verify the payment summary looks correct and complete the checklist. Thanks!

[melvin-bot]

@tomekzaw, @thienlnam, @laurenreidexpensify, @eh2077 Whoops! This issue is 2 days overdue. Let's get this updated quick!

[laurenreidexpensify]

Payment Summary:

Contributor: @tomekzaw agency worker - no payment
C+: @eh2077 $250 paid in Upwork

@eh2077 pls confirm if a regression test is required

[eh2077]

@laurenreidexpensify This is an edge case, so I don't think it's necessary to add a regression test for it.