fix(web): folder view navigation refresh #16492
Open
+10
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
In the gallery-viewer component in the web directory, I saved the "currentViewAssetIndex" variable to localStorage whenever it was changed and set the variable to the value in localStorage upon page load (if it exists).
This change fixes a bug, where upon loading the page while navigating between images in folder view, the currentViewAssetIndex would be set to 0 by default, so using the right/left arrows to navigate between images results in the wrong ones.
Fixes #15222
How Has This Been Tested?
I went to folder view opened up images at varying positions, navigated between them, and deleted them, refreshing the page after each action to check if I continue at the proper index. I also checked navigation on other views to ensure these changes did not break them
Screenshots (if appropriate)
N/A
Checklist:
src/services
uses repositories implementations for database calls, filesystem operations, etc.src/repositories/
is pretty basic/simple and does not have any immich specific logic (that belongs insrc/services
)