diff options
| author | Eric Bailey <git@esb.lol> | 2023-12-01 15:40:50 -0600 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-12-01 13:40:50 -0800 |
| commit | 9dec9d72768c6217a3878c80b11cbbd3d9ed0ff9 (patch) | |
| tree | c4e131576171c1289bdf5f95b018ee2003666be8 /src/view/com | |
| parent | f80bd30ef47ff1959e97ca8cb7652c98d115c0ab (diff) | |
| download | voidsky-9dec9d72768c6217a3878c80b11cbbd3d9ed0ff9.tar.zst | |
Only poll feed when feed page is focused (#2064)
* Do no poll when screen is not focused * Avoid polling unless focused * Handle homepage in background * Fix the intl:check to ignore comments in diffs --------- Co-authored-by: Paul Frazee <pfrazee@gmail.com>
Diffstat (limited to 'src/view/com')
| -rw-r--r-- | src/view/com/feeds/FeedPage.tsx | 2 | ||||
| -rw-r--r-- | src/view/com/posts/Feed.tsx | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/src/view/com/feeds/FeedPage.tsx b/src/view/com/feeds/FeedPage.tsx index f06716fb0..f3f07a8bd 100644 --- a/src/view/com/feeds/FeedPage.tsx +++ b/src/view/com/feeds/FeedPage.tsx @@ -158,9 +158,9 @@ export function FeedPage({ <View testID={testID} style={s.h100pct}> <Feed testID={testID ? `${testID}-feed` : undefined} + enabled={isPageFocused} feed={feed} feedParams={feedParams} - enabled={isPageFocused} pollInterval={POLL_FREQ} scrollElRef={scrollElRef} onScroll={onMainScroll} diff --git a/src/view/com/posts/Feed.tsx b/src/view/com/posts/Feed.tsx index 393c1bc91..f0f7cd919 100644 --- a/src/view/com/posts/Feed.tsx +++ b/src/view/com/posts/Feed.tsx @@ -89,7 +89,7 @@ let Feed = ({ const isEmpty = !isFetching && !data?.pages[0]?.slices.length const checkForNew = React.useCallback(async () => { - if (!data?.pages[0] || isFetching || !onHasNew) { + if (!data?.pages[0] || isFetching || !onHasNew || !enabled) { return } try { @@ -99,7 +99,7 @@ let Feed = ({ } catch (e) { logger.error('Poll latest failed', {feed, error: String(e)}) } - }, [feed, data, isFetching, onHasNew]) + }, [feed, data, isFetching, onHasNew, enabled]) React.useEffect(() => { // we store the interval handler in a ref to avoid needless |