[$250] Chat - Text in compose box blends with background when theme was changed in device settings

[izarutskaya]

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-4
Reproducible in staging?: Y
Reproducible in production?: N
Email or phone of affected tester (no customers): applausetester+omq23@applause.expensifail.com
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team

Action Performed:

Precondition: user should be Signed In and "Use Device Settings" option should be selected for theme option in Preferences

Web, Desktop and mWeb steps:

1. Open https://staging.new.expensify.com/

2. Open any chat

3. Navigate to device settings and change the theme to the opposite color

4. Navigate back to browser or app

5. Write any text in compose box

iOS steps:

1. Open https://staging.new.expensify.com/

2. Open any chat

3. Write any text in the compose box

4. Navigate to device settings and change the theme to the opposite color

5. Navigate back to app

Expected Result:

Text should be properly displayed in compose box.

Actual Result:

The text in the compose box retains its original color despite changes to the background color. Consequently, it blends with the background when the device theme is switched to the opposite color.

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

Bug6432761_1711819047019.Screen_Recording_2024-03-30_at_19.06.10.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01b5c49154255020bb
• Upwork Job ID: 1776299979557351424
• Last Price Increase: 2024-04-05

[melvin-bot]

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

[melvin-bot]

Triggered auto assignment to @anmurali (Bug), see https://stackoverflow.com/c/expensify/questions/14418 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.

[izarutskaya]

@anmurali 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.

[izarutskaya]

We think this issue might be related to the #vip-vsb.

[izarutskaya]

Production

Bandicam.2024-04-01.13-52-24-147.mp4

[MariaHCD]

Able to reproduce the issue when switching to dark mode on staging web Chrome:

Screen.Recording.2024-04-01.at.4.30.53.PM.mov

Wasn't able to repro on desktop, however.

[melvin-bot]

Unable to auto-create job on Upwork. The BZ team member should create it manually for this issue.

[melvin-bot]

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

[nkdengineer]

I can raise the PR ASAP once assigned

Proposal

Please re-state the problem that we are trying to solve in this issue.

The text in the compose box retains its original color despite changes to the background color. Consequently, it blends with the background when the device theme is switched to the opposite color.

What is the root cause of that problem?

In here, we forgot to put flattenedStyle.color to the dependency list. So when the flattenedStyle.color changes due to theme changes, it's not reflected in updateTextColor, so the next time updateTextColor is run, it will update the text to the old color of the previous theme.

What changes do you think we should make in order to solve the problem?

In here, put flattenedStyle.color to the dependency list.

What alternative solutions did you explore? (Optional)

NA

[melvin-bot]

⚠️ Could not update price automatically because there is no linked Upwork Job ID. The BZ team member will need to update the price manually in Upwork.

[shubham1206agra]

This is fixed on staging for the web.
cc @thienlnam

[nkdengineer]

Based on this comment #39348 (comment), The react-native-live-markdown repo hasn't opened for external contributors yet. @thienlnam Is this issue still open for external contributors?

@hoangzinh Based on this thread, we've started to open up the react-native-live-markdown issues to external contributors, including this one.

[shubham1206agra]

Based on this comment #39348 (comment), The react-native-live-markdown repo hasn't opened for external contributors yet. @thienlnam Is this issue still open for external contributors?

@hoangzinh Based on this thread, we've started to open up the react-native-live-markdown issues to external contributors, including this one.

@nkdengineer You have misinterpreted the comment here.

[nkdengineer]

Based on this comment #39348 (comment), The react-native-live-markdown repo hasn't opened for external contributors yet. @thienlnam Is this issue still open for external contributors?

I think @thienlnam can help confirm here.

[tomekzaw]

I was able to reproduce the issue in Live Markdown example app, working on a fix for iOS native right now.

[tomekzaw]

Found the root cause in react-native-live-markdown, fix incoming

[thienlnam]

Sorry for the confusion - I have opened up some issues related this externally, but seems like @tomekzaw is already working on this and the fix requires changes in the original repo so I will be assigning to him

[tomekzaw]

Should be fixed with Expensify/react-native-live-markdown#269.

I will submit a PR with version bump soon.

[tomekzaw]

Should be fixed with #39872.

[melvin-bot]

This issue has not been updated in over 15 days. @hoangzinh, @tomekzaw, @thienlnam, @BartoszGrajdek eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

[BartoszGrajdek]

@thienlnam can we get this tested & closed?

[mallenexpensify]

Added @anmurali back as BZ (not sure if they're needed though, we just need BZs added to all bugs, in case they are).

[mvtglobally]

Issue not reproducible during KI retests. (First week)

[mallenexpensify]

Closing, also unable to repro (and.. now I'm in dark mode!)