From 6988c1f458f2b4e8f59c800af2645f2a2494b7d4 Mon Sep 17 00:00:00 2001
From: AleksanderSklorz <115619721+AleksanderSklorz@users.noreply.github.com>
Date: Tue, 14 Nov 2023 12:44:48 +0100
Subject: [PATCH] [ACS-6067] viewer thumbnails not refresh on file change
 (#9075)

* ACS-6067 Update thumbnails when file is changed

* ACS-6067 Added unit tests

* ACS-6067 Removed redundant line
---
 .../src/lib/viewer/components/pdf-viewer.component.html  | 6 ++----
 .../lib/viewer/components/pdf-viewer.component.spec.ts   | 9 +++++++++
 .../src/lib/viewer/components/pdf-viewer.component.ts    | 1 +
 3 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/lib/core/src/lib/viewer/components/pdf-viewer.component.html b/lib/core/src/lib/viewer/components/pdf-viewer.component.html
index cf83f4e135..9844cf5173 100644
--- a/lib/core/src/lib/viewer/components/pdf-viewer.component.html
+++ b/lib/core/src/lib/viewer/components/pdf-viewer.component.html
@@ -16,10 +16,8 @@
                 <ng-container *ngIf="thumbnailsTemplate">
                     <ng-container *ngTemplateOutlet="thumbnailsTemplate;context:pdfThumbnailsContext"></ng-container>
                 </ng-container>
-
-                <ng-container *ngIf="!thumbnailsTemplate">
-                    <adf-pdf-thumbnails (close)="toggleThumbnails()" [pdfViewer]="pdfViewer"></adf-pdf-thumbnails>
-                </ng-container>
+                <adf-pdf-thumbnails *ngIf="!thumbnailsTemplate && !isPanelDisabled" (close)="toggleThumbnails()" [pdfViewer]="pdfViewer">
+                </adf-pdf-thumbnails>
             </div>
         </div>
     </ng-container>
diff --git a/lib/core/src/lib/viewer/components/pdf-viewer.component.spec.ts b/lib/core/src/lib/viewer/components/pdf-viewer.component.spec.ts
index a3361d0837..e33a9e0f22 100644
--- a/lib/core/src/lib/viewer/components/pdf-viewer.component.spec.ts
+++ b/lib/core/src/lib/viewer/components/pdf-viewer.component.spec.ts
@@ -27,6 +27,8 @@ import { take } from 'rxjs/operators';
 import { AppConfigService } from '../../app-config/app-config.service';
 import { CoreTestingModule } from '../../testing/core.testing.module';
 import { TranslateModule } from '@ngx-translate/core';
+import { PdfThumbListComponent } from '@alfresco/adf-core';
+import { By } from '@angular/platform-browser';
 
 declare const pdfjsLib: any;
 
@@ -354,6 +356,13 @@ describe('Test PdfViewer component', () => {
                     done();
                 });
             }, 55000);
+
+            it('should not render PdfThumbListComponent during initialization of new pdfViewer', () => {
+                componentUrlTestComponent.pdfViewerComponent.toggleThumbnails();
+                componentUrlTestComponent.urlFile = 'file.pdf';
+                fixtureUrlTestComponent.detectChanges();
+                expect(fixtureUrlTestComponent.debugElement.query(By.directive(PdfThumbListComponent))).toBeNull();
+            });
         });
 
         describe('Viewer events', () => {
diff --git a/lib/core/src/lib/viewer/components/pdf-viewer.component.ts b/lib/core/src/lib/viewer/components/pdf-viewer.component.ts
index 63893ac3f9..b9afb6c376 100644
--- a/lib/core/src/lib/viewer/components/pdf-viewer.component.ts
+++ b/lib/core/src/lib/viewer/components/pdf-viewer.component.ts
@@ -205,6 +205,7 @@ export class PdfViewerComponent implements OnChanges, OnDestroy {
             this.loadingPercent = Math.round(level * 100);
         };
 
+        this.isPanelDisabled = true;
         this.loadingTask.promise
             .then((pdfDocument: PDFDocumentProxy) => {
                 this.totalPages = pdfDocument.numPages;