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 top bar layout frame #5008

Merged
merged 3 commits into from
Apr 21, 2019
Merged

Fix top bar layout frame #5008

merged 3 commits into from
Apr 21, 2019

Conversation

yogevbd
Copy link
Collaborator

@yogevbd yogevbd commented Apr 21, 2019

• Fix top bar component layout frame size - Closes #5003
• Clean duplicate initWithLayoutInfo implementations
• Initialize parent child controllers in the constructors and not with setViewControllers

@guyca guyca changed the title Fix top bar layout frame, Closes #5003 Fix top bar layout frame Apr 21, 2019
@yogevbd yogevbd merged commit be00c4c into master Apr 21, 2019
@yogevbd yogevbd deleted the fixRegression branch April 21, 2019 13:32
@exentrich
Copy link

Unfortunately on iOS alignment center is not fixed. On first appearance of screen is aligned correctly, but after push and pop it out of center.
Снимок экрана 2019-04-22 в 20 31 02
Снимок экрана 2019-04-22 в 20 31 08

@yogevbd
Copy link
Collaborator Author

yogevbd commented Apr 22, 2019

@exentrich Managed to reproduce.. pushing a fix. Thanks!

vshkl pushed a commit to vshkl/react-native-navigation that referenced this pull request Feb 5, 2020
* Fix top bar layout frame, Closes wix#5003

* Set layout child view controllers on initialization

* Remove duplicate initWithLayoutInfo: function
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.

Custom top bar layout is broken on iOS on latest release 2.17.0
2 participants