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

SetRoot wait for render #4688

Merged
merged 17 commits into from
Feb 6, 2019
Merged

SetRoot wait for render #4688

merged 17 commits into from
Feb 6, 2019

Conversation

yogevbd
Copy link
Collaborator

@yogevbd yogevbd commented Feb 5, 2019

No description provided.

@guyca
Copy link
Collaborator

guyca commented Feb 6, 2019

🔥 🔥 🔥

@yogevbd yogevbd merged commit 5abea28 into master Feb 6, 2019
@yogevbd yogevbd deleted the setRootWaitForRender branch February 6, 2019 14:03
vshkl pushed a commit to vshkl/react-native-navigation that referenced this pull request Feb 5, 2020
* Layout changes

* WIP

* Revert "Layout changes"

This reverts commit 4da8cfb.

* Clean topBar.component and topBar.background.component when not needed

* Move custom components login from viewController to presenters

* Change waitForRender and enabled property from primitive bool to Bool object

* Fix ios unit tests

* Add unit tests

* Remove RNNReactView observer when view ready

* Load react components from component manager

* Remove createCustomReactView duplication

* Handle render children in each Layout

* Wait for navigation presenter components render, refactor RNNTitleViewHelper

* Fix unit tests

* Add waitForRender for custom react components

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

Successfully merging this pull request may close these issues.

2 participants