Skip to content

Troubleshoot page has blue color status bar #44484

Closed

Description

@m-natarajan

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: 9.0.2-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
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: @ishpaul777
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1719384828744689

Action Performed:

  1. Go to staging.new.expensify.com
  2. Tap Settings > Trouble shoot

Expected Result:

Status bar should be same color as other pages with animation

Actual Result:

Troubleshoot page has blue color status bar

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

Screen.Recording.2024-06-26.at.12.21.10.PM.mov

View all open jobs on GitHub

Activity

  1. added
    BugSomething is broken. Auto assigns a BugZero manager.
    on Jun 26, 2024
  2. melvin-bot commented on Jun 26, 2024

    @melvin-bot

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

  3. ishpaul777 commented on Jun 26, 2024

    @ishpaul777
    Contributor

    Proposal

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

    Troubleshoot page has blue color status bar

    What is the root cause of that problem?

    Here we pass illustrationBackgroundColor=PAGE_THEMES[SCREENS.SETTINGS.TROUBLESHOOT].backgroundColor

    illustrationBackgroundColor={theme.PAGE_THEMES[SCREENS.SETTINGS.TROUBLESHOOT].backgroundColor}

    and in here we have logic to chage status bar color of all pages that has a theme color

    if (currentRoute && 'name' in currentRoute && currentRoute.name in theme.PAGE_THEMES) {
    const pageTheme = theme.PAGE_THEMES[currentRoute.name];
    newStatusBarStyle = pageTheme.statusBarStyle;

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

    we need to remove troubleshoot color from here

    [SCREENS.SETTINGS.TROUBLESHOOT]: {
    backgroundColor: colors.blue700,
    statusBarStyle: CONST.STATUS_BAR_STYLE.LIGHT_CONTENT,
    },

    and

    [SCREENS.SETTINGS.TROUBLESHOOT]: {
    backgroundColor: colors.blue700,
    statusBarStyle: CONST.STATUS_BAR_STYLE.LIGHT_CONTENT,
    },

    and add backgroundColor: colors.blue700 here:

    Desk: {
    file: require('@assets/animations/Desk.lottie'),
    w: 200,
    h: 120,

  4. isabelastisser commented on Jun 27, 2024

    @isabelastisser
    Contributor

    I cannot recreate this issue in staging or prod. Closing.

  5. ishpaul777 commented on Jun 27, 2024

    @ishpaul777
    Contributor
    image
    image

    @isabelastisser this is consistently reproducible on mweb Safari, Mind rechecking please ?

  6. isabelastisser commented on Jun 27, 2024

    @isabelastisser
    Contributor

    I can't reproduce this on Safari either:

    image
  7. ishpaul777 commented on Jun 27, 2024

    @ishpaul777
    Contributor

    Theme coloring statusbar is a feature exclusive to mobile safari only, its not supported on other platforms.

  8. isabelastisser commented on Jun 27, 2024

    @isabelastisser
    Contributor

    I see, I'm not sure if this should be considered a bug. I don't think we should change it. @shawnborton, what's your opinion here? The blue bar actually calls to switch to the New Expensify mobile app on Safari, I like it.

  9. shawnborton commented on Jun 27, 2024

    @shawnborton
    Contributor

    We're already taking care of this one here: #44048

    So we can just keep this one closed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

BugSomething is broken. Auto assigns a BugZero manager.DailyKSv2

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions