FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(ios): safeAreaLayoutGuide fallback for iOS 10 cases by vakrilov · Pull Request #5960 · NativeScript/NativeScript · GitHub

fix(ios): safeAreaLayoutGuide fallback for iOS 10 cases - #5960

Merged
vakrilov merged 4 commits into
masterfrom
safe-area-fallback
Jun 20, 2018
Merged

vakrilov merged 4 commits into
masterfrom
safe-area-fallback

Conversation

Copy link
Copy Markdown
Contributor

PR Checklist

What is the current behavior?

Crash reported in #5899.

What is the new behavior?

A fallback mechanism to init a layout guide during the layoutView in iOS 10.

Normal execution should not go trough the fallback as the safeAreaLayoutGuide property should already have been initialized in updateConstraints called in viewWillLayoutSubviews of the controller before the layout. This fallback is for cases layoutView was called before updateConstraints (perhaps manually called).

CC @NathanWalker

Copy link
Copy Markdown
Contributor Author

test

1 similar comment

Copy link
Copy Markdown
Contributor Author

test

ghost assigned manoldonev Jun 19, 2018
vakrilov merged commit 4b5754a into master Jun 20, 2018
ghost removed the in progress label Jun 20, 2018
vakrilov deleted the safe-area-fallback branch June 20, 2018 04:28

lock Bot commented Aug 26, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

lock Bot locked and limited conversation to collaborators Aug 26, 2019
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL