fix: use ?? instead of || so empty thumbnail string skips generation

2026-05-05 17:50:09 +01:00
parent fa3e9f5db6
commit e2e37695cd
1 changed files with 3 additions and 2 deletions
@@ -92,9 +92,10 @@ export function IndexedDBProvider({ children }: IndexedDBProviderProps) {
      fileId: FileId,
      existingThumbnail?: string,
    ): Promise<StirlingFileStub> => {
-      // Use existing thumbnail or generate new one if none provided (empty string = no raster thumbnail)
+      // existingThumbnail="" means caller explicitly opted out of a raster thumbnail;
+      // only generate when the caller passed nothing (undefined).
      const generated =
-        existingThumbnail || (await generateThumbnailForFile(file));
+        existingThumbnail ?? (await generateThumbnailForFile(file));
      const thumbnail = generated || undefined;

      // History is handled via direct fileStorage calls, not here