Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Favicon - Flashing NEW! Expensify.cash and icon without white background can be seen in tab #2636

Closed
isagoico opened this issue Apr 29, 2021 · 10 comments · Fixed by #2993
Closed
Assignees

Comments

@isagoico
Copy link

isagoico commented Apr 29, 2021

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


Expected Result:

I expected no flashing with NEW! Expensify.cash and switching with Expensify.cash

Actual Result:

Flashing with NEW! Expensify.cash and switching with Expensify.cash

Action Performed:

  1. Launch the app as Account B and log in
  2. As account B, send some messages to account A
  3. Go to https://staging.expensify.cash as Account A in Web/Chrome
  4. Observe the favicons and NEW! Expensify.cash

Workaround:

No need, visual issue.

Platform:

Where is this issue occurring?

Web ✔️
iOS
Android
Desktop App
Mobile Web

Version Number: 1.0.34-0

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos:

Bug5045902_Recording__37.mp4

Expensify/Expensify Issue URL:

View all open jobs on Upwork

@isagoico isagoico changed the title Favico - Flashing NEW! Expensify.cash is seen in tab and icon without white background can be seen in tab Favico - Flashing NEW! Expensify.cash and icon without white background can be seen in tab Apr 29, 2021
@isagoico isagoico added the AutoAssignerTriage Auto assign issues for triage to an available triage team member label Apr 29, 2021
@MelvinBot
Copy link

Triggered auto assignment to @lschurr (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

@isagoico isagoico added the Daily KSv2 label Apr 29, 2021
@MelvinBot MelvinBot removed the AutoAssignerTriage Auto assign issues for triage to an available triage team member label Apr 29, 2021
@isagoico isagoico changed the title Favico - Flashing NEW! Expensify.cash and icon without white background can be seen in tab Favicon - Flashing NEW! Expensify.cash and icon without white background can be seen in tab Apr 30, 2021
@isagoico
Copy link
Author

isagoico commented May 9, 2021

Issue reproducible during today's KI retests

@MelvinBot
Copy link

Triggered auto assignment to @joelbettner (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@lschurr lschurr added Weekly KSv2 and removed Daily KSv2 labels May 10, 2021
@lschurr lschurr removed their assignment May 10, 2021
@joelbettner
Copy link
Contributor

Looks like it has what it needs. Not labeling with "Demolition" because that label does not appear to be an option:

image

@joelbettner joelbettner removed their assignment May 10, 2021
@deetergp deetergp self-assigned this May 13, 2021
@deetergp
Copy link
Contributor

This seems like a safe enough one for me to use to teach myself a bit more about ReactNative. Snagging it!

@isagoico
Copy link
Author

Issue reproducible during today's KI retests

@deetergp
Copy link
Contributor

I've spent some time digging into this today. The issue apperas to be that, although we are overriding the a listener inside componentDidMount in AuthScreens.js, something else is briefly overriding it after the first it mounts, setting the title back to Expensify.cash, until the component re-mounts.

I'm not entirely certain what that could be but I'll keep poking around to see if I can figure it out, and if I can't, I'll most likely end up pestering @marcaaron who, like a zen master, will present me with React Native koans till I stumble upon enlightenment.

@marcaaron
Copy link
Contributor

What is a page title?

An alarm clock when the power returns.

@deetergp
Copy link
Contributor

Ah-HAH! So it's line 43 in NavigationRoot.js that is squashing the (NEW!)… title. If I change the string from Expensify.cash to NavigationContainer this is what I see:

NavigationRoot.mp4

@marcaaron
Copy link
Contributor

my work here is done

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

Successfully merging a pull request may close this issue.

6 participants