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

Use react-native-screens #168

Merged
merged 2 commits into from
Sep 24, 2018
Merged

Conversation

MortalKastor
Copy link
Contributor

Implement the bleeding edge react-native-screens to use native navigation containers which could bring sizable performance improvements (see https://www.youtube.com/watch?v=Z0Jl1KCWiag ).

Flagged as HODL because:

@gre
Copy link
Contributor

gre commented Sep 22, 2018

let's test this again next week. we have a usage of react-native-gesture-handler now (the graph). if you can't properly interact with the graph anymore with this lib, then it's a dead end. if there is no difference, we should be good.

We might want to play with the react-native-gesture-handler buttons too before testing these, so maybe smart to first work on #208 .

For the ReactFragmentActivity thing, I think it's not a blocker to get this in but we might want to Subscribe on software-mansion/react-native-screens#7 discussions to get it solved

@gre gre removed the HODL label Sep 24, 2018
@gre gre merged commit fb9f29f into LedgerHQ:develop Sep 24, 2018
@MortalKastor MortalKastor deleted the react-native-screens branch November 5, 2018 13:31
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.

2 participants