Skip to content

fix(android): prevent “flashing” activity on app start#9190

Merged
NathanWalker merged 4 commits into
NativeScript:mainfrom
Akylas:android_start_flash_fix
Nov 24, 2022
Merged

fix(android): prevent “flashing” activity on app start#9190
NathanWalker merged 4 commits into
NativeScript:mainfrom
Akylas:android_start_flash_fix

Conversation

@farfromrefug

Copy link
Copy Markdown
Collaborator

This was due to a change made a while back to fix an issue with resetRootView.
The consequence was that we were waiting for the frame to be “attachedToWindow” to load the first frame page.
That PR make it so that we apply that check/fix only when the activity was reset. Thus it wont happen on app start

The flash issue is currently seeable on ALL apps. Though it depends on the complexity of the first frame page. the more complex the longer you lll see the "white" screen issue (white being your activity background color)

This was due to a change made a while back to fix an issue with resetRootView.
The consequence was that we were waiting for the frame to be “attachedToWindow” to load the first frame page.
That PR make it so that we apply that check/fix only when the activity was reset. Thus it wont happen on app start
@rigor789

rigor789 commented Feb 3, 2021

Copy link
Copy Markdown
Member

We need to make sure the are no cases where _onRootViewReset is not called at all - as those cases would fail with this change. Don't know without a deeper look if a scenario like that is even possible, just leaving it here as a note.

@NathanWalker NathanWalker added this to the 8.4 milestone Aug 16, 2022
@cla-bot cla-bot Bot added the cla: yes label Nov 18, 2022
@NathanWalker NathanWalker merged commit 6a9484a into NativeScript:main Nov 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants