wallet icon indicating copy to clipboard operation
wallet copied to clipboard

feat: use @react-navigation/native-stack

Open MuckT opened this issue 3 years ago • 0 comments

Description

Moves to @react-navigation/native-stack from @react-navigation/stack for increased performance. Initial testing shows a ~20% memory usage reduction from 350MB to 275MB.

Todo

  • [ ] Fix build errors
  • [ ] Update failing tests
  • [ ] Check if @react-navigation/drawer & @react-navigation/material-top-tabs need to be updated also.

Other changes

Updated @react-navigation/native Added @react-navigation/elements

Tested

Tested locally on iOS and Android

How others should test

Devs

  • Run the app in profile mode on Android and check the memory usage yarn dev:android -e mainnet -r -t.
  • Follow the steps in the profile section of the WALLET.md.
  • Check for misaligned buttons, incorrect background colors, or text in headers.

QA

  • Navigate through onboarding screens and ensure that the proper header is displayed.
  • Navigate through the Add & Withdraw flows and ensure the proper header is displayed.
  • Navigate through setting items and ensure that the proper header is displayed.
  • Navigate through the send and receive flows and ensure that the proper header is displayed.
  • Check the QR code scanner header is displayed correctly.

Related issues

N/A

Backwards compatibility

Yes

MuckT avatar Oct 18 '22 01:10 MuckT