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

Add layout.insets option #7416

Merged
merged 33 commits into from
Jan 4, 2022
Merged

Add layout.insets option #7416

merged 33 commits into from
Jan 4, 2022

Conversation

yogevbd
Copy link
Collaborator

@yogevbd yogevbd commented Dec 28, 2021

This feature allows adding insets to the topmost view controller in the hierarchy, it is useful for presenting a banner across the app with overlay without having to re-render it each time a new screen is being pushed.

@yogevbd yogevbd merged commit 9bb70d8 into master Jan 4, 2022
@yogevbd yogevbd deleted the layoutInsets branch January 4, 2022 08:09
@N3TC4T
Copy link
Contributor

N3TC4T commented Jan 5, 2022

Thanks @yogevbd for the fix, just one more issue, setting adjustResize to false when setting the default options with setDefaultOptions won't apply the value. so adjustResize is always true.

@yogevbd
Copy link
Collaborator Author

yogevbd commented Jan 5, 2022

That's right, @swabbass we should resolve adjustResize option from default

swabbass added a commit that referenced this pull request Jan 6, 2022
commit b8d7472
Author: wixmobile <mobile1@wix.com>
Date:   Tue Jan 4 01:09:08 2022 -0800

    Update package.json version to 7.25.0 [buildkite skip]

commit 9bb70d8
Author: Yogev Ben David <yogev132@gmail.com>
Date:   Tue Jan 4 10:09:17 2022 +0200

    Add layout.insets option (#7416)

    This feature allows adding insets to the topmost view controller in the hierarchy, it is useful for presenting a banner across the app with overlay without having to re-render it each time a new screen is being pushed.

    Co-authored-by: Ward Abbass <swabbass@gmail.com>

commit 403fcf1
Author: igorgn <50628426+igorgn@users.noreply.github.com>
Date:   Mon Jan 3 13:46:50 2022 +0200

    Archive artifacts (#7417)

    * archive artifacts

    * e2e archiving

commit 6380c29
Author: Ward Abbass <swabbass@gmail.com>
Date:   Tue Dec 28 13:46:39 2021 +0200

    Exclude android unit tests for non playground projects (#7415)

    Exclude android native unit tests and their dependencies from being passed to other projects.
swabbass added a commit that referenced this pull request Jan 6, 2022
commit b8d7472
Author: wixmobile <mobile1@wix.com>
Date:   Tue Jan 4 01:09:08 2022 -0800

    Update package.json version to 7.25.0 [buildkite skip]

commit 9bb70d8
Author: Yogev Ben David <yogev132@gmail.com>
Date:   Tue Jan 4 10:09:17 2022 +0200

    Add layout.insets option (#7416)

    This feature allows adding insets to the topmost view controller in the hierarchy, it is useful for presenting a banner across the app with overlay without having to re-render it each time a new screen is being pushed.

    Co-authored-by: Ward Abbass <swabbass@gmail.com>

commit 403fcf1
Author: igorgn <50628426+igorgn@users.noreply.github.com>
Date:   Mon Jan 3 13:46:50 2022 +0200

    Archive artifacts (#7417)

    * archive artifacts

    * e2e archiving

commit 6380c29
Author: Ward Abbass <swabbass@gmail.com>
Date:   Tue Dec 28 13:46:39 2021 +0200

    Exclude android unit tests for non playground projects (#7415)

    Exclude android native unit tests and their dependencies from being passed to other projects.
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.

3 participants