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 topBar title with subtitle view #6509

Merged
merged 5 commits into from
Sep 1, 2020
Merged

Fix topBar title with subtitle view #6509

merged 5 commits into from
Sep 1, 2020

Conversation

yogevbd
Copy link
Collaborator

@yogevbd yogevbd commented Aug 25, 2020

Move topBar title with subtitle view initialisation to when the viewController view is already initialised. Fix's cases where the titleView initialised too soon and therefor not visible because layoutSubviews never gets called.

Copy link
Collaborator

@guyca guyca left a comment

Choose a reason for hiding this comment

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

Looks good! Lets add e2e and we're good to go!

@yogevbd yogevbd merged commit 1878393 into master Sep 1, 2020
@yogevbd yogevbd deleted the fixTitleWithSubtitle branch September 1, 2020 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants