From 22e99e5c0fed9b69faad1ca5b4ce2b165eaf1d6e Mon Sep 17 00:00:00 2001
From: domsteinbach <36757218+domsteinbach@users.noreply.github.com>
Date: Tue, 25 Jul 2023 10:10:30 +0200
Subject: [PATCH] fix(pdf viewer): menubar is broken (#1160)

---
 .../representation/document/document.component.html         | 5 +----
 .../representation/document/document.component.scss         | 4 +---
 .../resource/representation/document/document.component.ts  | 6 +++---
 apps/dsp-app/src/styles/_elements.scss                      | 1 +
 4 files changed, 6 insertions(+), 10 deletions(-)

diff --git a/apps/dsp-app/src/app/workspace/resource/representation/document/document.component.html b/apps/dsp-app/src/app/workspace/resource/representation/document/document.component.html
index fbee83ea7a..22ff236bed 100644
--- a/apps/dsp-app/src/app/workspace/resource/representation/document/document.component.html
+++ b/apps/dsp-app/src/app/workspace/resource/representation/document/document.component.html
@@ -20,7 +20,7 @@
     >
     </pdf-viewer>
 
-    <div class="toolbar">
+    <div>
         <div class="action horizontal bottom">
             <!-- caption -->
             <span>
@@ -94,9 +94,6 @@
 
             <!-- empty placeholders to center the zoom buttons -->
             <span class="fill-remaining-space"></span>
-            <span class="empty-space"></span>
-            <span class="empty-space"></span>
-            <span class="empty-space"></span>
 
             <!-- full screen button -->
             <span>
diff --git a/apps/dsp-app/src/app/workspace/resource/representation/document/document.component.scss b/apps/dsp-app/src/app/workspace/resource/representation/document/document.component.scss
index 0359c78543..5f29f46d22 100644
--- a/apps/dsp-app/src/app/workspace/resource/representation/document/document.component.scss
+++ b/apps/dsp-app/src/app/workspace/resource/representation/document/document.component.scss
@@ -21,14 +21,12 @@ $osd-height: 460px;
   }
 
   .pdf-container {
-    color: $bright;
-    height: 90%;
+    height: 100%;
     background-color: $dark;
     border-radius: 8px 8px 0px 0px;
 
     pdf-viewer {
       position: inherit;
-      height: 100%;
 
       ::ng-deep .ng2-pdf-viewer-container {
         height: 90%;
diff --git a/apps/dsp-app/src/app/workspace/resource/representation/document/document.component.ts b/apps/dsp-app/src/app/workspace/resource/representation/document/document.component.ts
index 6738a137d6..1615392ff0 100644
--- a/apps/dsp-app/src/app/workspace/resource/representation/document/document.component.ts
+++ b/apps/dsp-app/src/app/workspace/resource/representation/document/document.component.ts
@@ -73,9 +73,6 @@ export class DocumentComponent implements OnInit, AfterViewInit {
 
     ngOnInit(): void {
         this.fileType = this._getFileType(this.src.fileValue.filename);
-        if (this.fileType === 'pdf') {
-            this.elem = document.getElementsByClassName('pdf-viewer')[0];
-        }
 
         this._rs
             .getFileInfo(this.src.fileValue.fileUrl)
@@ -88,6 +85,9 @@ export class DocumentComponent implements OnInit, AfterViewInit {
     }
 
     ngAfterViewInit() {
+        if (this.fileType === 'pdf') {
+            this.elem = document.getElementsByClassName('pdf-viewer')[0];
+        }
         this.loaded.emit(true);
     }
 
diff --git a/apps/dsp-app/src/styles/_elements.scss b/apps/dsp-app/src/styles/_elements.scss
index 73b687e7b9..a5ac732897 100644
--- a/apps/dsp-app/src/styles/_elements.scss
+++ b/apps/dsp-app/src/styles/_elements.scss
@@ -327,6 +327,7 @@ a,
       }
 
       &.bottom {
+        background-color: inherit;
         bottom: 0;
       }
     }