kamaji/nextsnap
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix gallery rename applying to wrong files

Browse Source 
After renaming a file, loadDirectory() was called which rebuilt the
galleryImages array in new alphabetical order. The gallery index
didn't change, so it now pointed to a different file — subsequent
renames hit the wrong file.

Fix: only update local state during the gallery session, defer the
file list refresh to when the gallery closes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
kamaji
2026-02-07 15:27:26 -06:00
parent 32801852cb
commit 0eef9bf2f3
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
app/static/js/filebrowser.js
View File

@@ -291,6 +291,9 @@ const FileBrowser = {
const img = document.getElementById('gallery-image'); const img = document.getElementById('gallery-image');
if (img.src && img.src.startsWith('blob:')) URL.revokeObjectURL(img.src); if (img.src && img.src.startsWith('blob:')) URL.revokeObjectURL(img.src);
img.src = ''; img.src = '';
// Refresh file list to reflect any renames
this.loadDirectory(this.currentPath);
}, },
async displayGalleryImage(index) { async displayGalleryImage(index) {
@@ -474,9 +477,6 @@ const FileBrowser = {
this.closeRenameModal(); this.closeRenameModal();
this.showGalleryToast('File renamed', 'success'); this.showGalleryToast('File renamed', 'success');
// Refresh the file list in the background
this.loadDirectory(this.currentPath);
} catch (error) { } catch (error) {
console.error('Error renaming file:', error); console.error('Error renaming file:', error);
errorEl.textContent = error.message; errorEl.textContent = error.message;