Skip to content
This repository has been archived by the owner on Jun 16, 2022. It is now read-only.

LL-8162 - Fix safe area issues #1995

Merged
merged 1 commit into from
Nov 22, 2021
Merged

LL-8162 - Fix safe area issues #1995

merged 1 commit into from
Nov 22, 2021

Conversation

jelbaz-ledger
Copy link
Contributor

@jelbaz-ledger jelbaz-ledger commented Nov 19, 2021

Use SafeAreaView from react-native-safe-area-context to prevent escaping from the safe area.

React navigation recommends it for various reasons: https://reactnavigation.org/docs/handling-safe-area/

Type

Bug Fix

Context

LL-8162

Parts of the app affected / Test plan

  • Portfolio screen empty states
  • Portfolio screen when Carousel is displayed
  • Platform -> apps catalog screen

@jelbaz-ledger jelbaz-ledger requested review from a team November 19, 2021 13:52
@LFBarreto LFBarreto changed the base branch from develop to ENGB2C-115 November 22, 2021 09:26
@LFBarreto LFBarreto merged commit 438cf54 into ENGB2C-115 Nov 22, 2021
valpinkman pushed a commit that referenced this pull request Nov 22, 2021
* fix: use safe area views from react-native-safe-area-context (#1995)

gets along better with react navigation, see https://reactnavigation.org/docs/handling-safe-area/

* fix: exclude empty buttons generated by separator components (#1993)

* fix(router): add accounts route to AccountSettingsNavigator (#2000)

* fix lint

Co-authored-by: Julien Elbaz <[email protected]>
Co-authored-by: J <[email protected]>
@valpinkman valpinkman deleted the LL-8162 branch November 23, 2021 13:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants