Livestream category improvements #7115

Merged
infinite-persistence merged 15 commits from ip/category.livestream into master 2021-09-24 16:26:22 +02:00

15 commits

Author SHA1 Message Date
infinite-persistence
1a8f82b92f
Use better tile-count on larger screens.
Used the same method as how the homepage does it.
2021-09-24 12:46:13 +08:00
infinite-persistence
111dc695ca
WildWest: limit livestream tiles + add ability to show more
Most likely this behavior will change in the future, so we'll leave `ClaimListDiscover` untouched and handle the logic at the page level.

This solution uses 2 `ClaimListDiscover` -- if the reduced livestream list is visible, it handles the header; else the normal list handles the header.
2021-09-24 12:35:59 +08:00
infinite-persistence
39c7159f2e
doFetchActiveLivestreams: add option check
We'll need to support different 'orderBy', so adding an "options check" when determining if we just made the same fetch.
2021-09-24 12:35:58 +08:00
infinite-persistence
8340783e04
doFetchActiveLivestreams: add interval check
- Added a default minimum of 5 minutes between fetches. Clients can bypass this through `forceFetch` if needed.
2021-09-24 12:35:58 +08:00
infinite-persistence
dcb44e647d
Re-enable active livestream tiles using the new method 2021-09-24 12:35:57 +08:00
infinite-persistence
48073d411b
ClaimListDiscover: add prefixUris, similar to ClaimTilesDiscover
This will be initially used to append livestreams at the top.
2021-09-24 12:35:57 +08:00
infinite-persistence
ad158bcd93
ClaimListDiscover: revert and cleanup
## Revert
- Removed the 'finalUris' stuff that was meant to "pause" visual changes when fetching. I think it'll be cleaner to use React.memo to achieve that.

## Alterations
- Added `renderUri` to make it clear which array that this component will render.
- Re-do the way we fetch view counts now that 'finalUris' is gone. Not the best method, but at least correct for now.
2021-09-24 12:35:56 +08:00
infinite-persistence
1d8609d649
Add 'useFetchViewCount' to handle fetching from lists
This effect also stashes fetched uris, so that we won't re-fetch the same uris during the same instance (e.g. during infinite scroll).
2021-09-24 12:35:56 +08:00
infinite-persistence
d47718da8e
ClaimTilesDiscover: fill with placeholder while waiting for claim_search
## Issue
Livestream claims are fetched seperately, so they might already exists. While claim_search is running, the list only consists of livestreams (collapsed).

## Fix
Fill up the space with placeholders to prevent layout shift.
2021-09-24 12:35:55 +08:00
infinite-persistence
6cfdff7eec
ClaimTilesDiscover: reduce ~17 renders at startup to just 2. 2021-09-24 12:35:54 +08:00
infinite-persistence
44ca2f147d
ClaimTilesDiscover: factor out options
## Change
Move the `option` code outside and passed in as a pre-calculated prop.

## Reason
To skip rendering while waiting for `claim_search`, we need to add `React.memo(areEqual)`. However, the flag that determines if we are fetching `claim_search` (fetchingClaimSearchByQuery[]) depends on the derived options as the key.

Instead of calculating `options` twice, we moved it to the props so both sides can use it.

It also makes the component a bit more readable.

The downside is that the prop-passing might not be clear.
2021-09-24 12:35:54 +08:00
infinite-persistence
673d679083
ClaimTilesDiscover: revert and cleanup
## Simplify
- Simplify to just `uris` instead of having multiple arrays (`uris`, `modifiedUris`, `prevUris`)
- The `prevUris` is for CLS prevention. With this removal, the CLS issue is back, but we'll handle it differently later.
- Temporarily disable the view-count fetching. Code is left there so that I don't forget.

## Fix
- `shouldPerformSearch` was never true when `prefixUris` is present. Corrected the logic.
- Aside: prefix and pin is so similar in function. Hm ....
2021-09-24 12:35:53 +08:00
infinite-persistence
16b33fe05f
Tiles can now query active-livestream state from redux instead of getting from parent. 2021-09-24 12:35:53 +08:00
infinite-persistence
b29f56d53f
Fetch and store active-livestream info in redux 2021-09-24 12:35:52 +08:00
infinite-persistence
ebda47576d
Remove old method of displaying active livestreams
Completely remove it for now to make the commit deltas clearer.
We'll replace it with the new method at the end.
2021-09-24 12:35:52 +08:00