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

fix: Android: Splash screen always showing behind other screens #11760

Merged
merged 1 commit into from
Oct 14, 2024

Conversation

NicolasMassart
Copy link
Contributor

@NicolasMassart NicolasMassart commented Oct 11, 2024

Description

Cause of the issue

The splash screen has been reworked recently for performances and replaced by an Android window background. But this is visible quickly between screen changes as nav stack has been reorganised.

Fix

  • remove the transparent background on MainFlow stack navigator to prevent the window background to be visible behind it

Related issues

Fixes #11697

Manual testing steps

Feature: Navigate between screens without splash screen visible
  Scenario: go to token detail screen
    Given wallet is ready
    And some tokens are available in the list (load from a test address)
    When user touches the token title or icon
    Then detail screen opens
    And no splash screen fox is visible even very quicly.

Note

The issue was on Android but do not hesitate to also test iOS to make sure nothing is broken.

Screenshots/Recordings

Before

See recording in #11697

After

android.webm

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@NicolasMassart NicolasMassart requested a review from a team as a code owner October 11, 2024 16:12
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

Copy link

sonarcloud bot commented Oct 11, 2024

@NicolasMassart NicolasMassart added the Run Smoke E2E Triggers smoke e2e on Bitrise label Oct 11, 2024
Copy link
Contributor

github-actions bot commented Oct 11, 2024

https://bitrise.io/ Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: 2e7f7e4
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/9d5a0067-9ed4-46be-aa6b-4b65d84eee27

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

@NicolasMassart NicolasMassart self-assigned this Oct 11, 2024
@salimtb
Copy link
Contributor

salimtb commented Oct 11, 2024

LGTM ✅

Copy link
Contributor

@chrisleewilcox chrisleewilcox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@salimtb salimtb added this pull request to the merge queue Oct 14, 2024
Merged via the queue into main with commit 267b107 Oct 14, 2024
55 of 56 checks passed
@salimtb salimtb deleted the fix/11697_splash_screen branch October 14, 2024 09:16
@github-actions github-actions bot locked and limited conversation to collaborators Oct 14, 2024
@NicolasMassart NicolasMassart added release-7.33.0 Issue or pull request that will be included in release 7.33.0 release-7.34.0 and removed release-7.34.0 release-7.33.0 Issue or pull request that will be included in release 7.33.0 labels Oct 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-7.33.0 Issue or pull request that will be included in release 7.33.0 Run Smoke E2E Triggers smoke e2e on Bitrise team-mobile-platform
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[Bug]: Android: Splash screen always showing when opening/closing Token view and other screens
4 participants