Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.33k stars 2.76k forks source link

[HybridApp] When quickly switching between apps and Hybrid App with the keyboard open, it flashes dark mode and changes my keyboard to dark mode #48299

Open m-natarajan opened 2 weeks ago

m-natarajan commented 2 weeks ago

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: Reproducible in staging?: Needs reproduction Reproducible in production?: - 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 Expensify/Expensify Issue URL: Issue reported by: @AndrewGable Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1724949894220739

Action Performed:

  1. Open the hybrid app , Initiate and compose a DM with any user (Keyboard open)
  2. Quickly swipe left or right along the bottom edge of the screen to switch between recently used apps

    Expected Result:

    No visual issues

    Actual Result:

    Quickly switching between apps and Hybrid App with the keyboard open, it flashes dark mode and changes my keyboard to dark mode

    Workaround:

    Unknown

    Platforms:

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

    • [ ] Android: Native
    • [ ] Android: mWeb Chrome
    • [X] iOS: Native
    • [ ] iOS: mWeb Safari
    • [ ] MacOS: Chrome / Safari
    • [ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/34262ae7-fb2c-4ffc-991e-1f6f0a3db960

View all open jobs on GitHub

melvin-bot[bot] commented 2 weeks ago

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

MelvinBot commented 2 weeks ago

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

melvin-bot[bot] commented 1 week ago

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

staszekscp commented 1 week ago

Hey! At the moment I am investigating this issue, and I was able to reproduce this issue also on the NewDot app:

https://github.com/user-attachments/assets/e3cc1920-8d34-49d8-8d4d-4ed307e86ba4

It seems that the workaround would be to timeout the value returned by useColorScheme, because the problem is known, and still exists. There is a chance that the problem will disappear when we enable bridgelessMode, but at the moment I think the timeout would be the best solution.

JmillsExpensify commented 1 week ago

Nice, did you want to take this issue or are we going to assign to someone internal?

war-in commented 1 week ago

Hi @JmillsExpensify 👋 I took this issue over from @staszekscp and I'm going to create a PR in a moment 🤞

staszekscp commented 1 week ago

Sorry @JmillsExpensify for missing your question yesterday! I was on the way to the React Universe Conference, but since I didn’t want to leave the issue unfinished I asked @war-in to build and test the fix! 😀 The PR is ready, so I think we’re good to go!

melvin-bot[bot] commented 1 week ago

Triggered auto assignment to @AndrewGable, see https://stackoverflow.com/c/expensify/questions/7972 for more details.