fix(web): previous/next asset navigation

[brighteyed]

This PR fixes the next/previous navigation in the AssetViewer including navigation across not loaded buckets

Example of the bug:

1. Open the main timeline (the first N bucket(s) are loaded now)
2. Click on the scrollbar somewhere near the bottom (some bucket(s) near that location are loaded. Note: _assetGridState.assets now contains assets from nonadjacent buckets)
3. Click on any asset to open AssetViewer
4. Click the Previous button multiple times. When we reach the end of the bucket(s) loaded in (2), we begin to show assets from buckets loaded in (1) thus missing a lot of assets in intermediate buckets.

Details:

I couldn't find a way to write navigateAsset function better. But I think the current implementation is at least clear and readable. It just finds the adjacent bucket when needed

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
immich	⬜️ Ignored (Inspect)			Jul 5, 2023 8:22am

[jrasm91]

Looks a lot better IMO! A few more suggestions, but looks good to me.