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

LL-8170 - exclude empty buttons generated by separator components #1993

Merged
merged 1 commit into from
Nov 22, 2021

Conversation

jelbaz-ledger
Copy link
Contributor

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

Prevents "empty" buttons in the account page.

Capture d’écran 2021-11-22 à 09 34 49

The issue arises when separator components are in the first 2 elements of the actions array they are converted into empty buttons.

Type

Bug Fix

Context

LL-8170

Parts of the app affected / Test plan

  • Read only mode (Onboarding without paring a device, import data from LLD)
  • Ethereum Account screen

@jelbaz-ledger jelbaz-ledger requested review from a team November 19, 2021 11:27
@LFBarreto LFBarreto changed the base branch from develop to ENGB2C-115 November 22, 2021 09:27
@LFBarreto LFBarreto merged commit b60c61b 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-8170 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