search

language-transfer / lt-app

React Native application for Language Transfer
https://www.languagetransfer.org/
Other
291 stars 26 forks source link

Cleanup: use the `useCallback` hook where appropriate #17

Open SyntaxBlitz opened 4 years ago

SyntaxBlitz commented 4 years ago

I think we need a lot of these, but I never really bothered when I was prototyping.

schonfeld commented 4 years ago

Tacking onto this, I've noticed a lot of props.navigation.addListener that can benefit from using react-navigation's own hooks useFocusEffect and useIsFocused

edit: I actually got most of these done in #27

SyntaxBlitz commented 4 years ago

totally agree, I found out about that hook pretty late so never actually used it.