Skip to content

Commit d0bb3a7

Browse files
author
Nikos Vasileiou
authored
Merge pull request #157 from transifex/memoize-useT
@transifex/react: Memoize useT
2 parents ff40301 + acfea28 commit d0bb3a7

1 file changed

Lines changed: 9 additions & 3 deletions

File tree

packages/react/src/hooks/useT.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
1-
import { useEffect, useState, useContext } from 'react';
1+
import {
2+
useEffect, useState, useContext, useCallback,
3+
} from 'react';
24
import {
35
tx,
46
onEvent,
@@ -29,7 +31,7 @@ export default function useT() {
2931
const context = useContext(TXNativeContext);
3032
const instance = context.instance || tx;
3133

32-
const [, setCounter] = useState(0);
34+
const [counter, setCounter] = useState(0);
3335
useEffect(() => {
3436
// Using `setCounter` will trigger a rerender
3537
function rerender(_, caller) {
@@ -44,5 +46,9 @@ export default function useT() {
4446
offEvent(TRANSLATIONS_FETCHED, rerender);
4547
};
4648
}, [setCounter, instance]);
47-
return (_str, props) => translateWithElements(_str, props, context);
49+
50+
return useCallback(
51+
(_str, props) => translateWithElements(_str, props, context),
52+
[context, counter],
53+
);
4854
}

0 commit comments

Comments
 (0)