From 4dca8ebc2ac8483d9fb9ac3a9b778b903e79963b Mon Sep 17 00:00:00 2001
From: Suzana Dirla <dirla.silvia.suzana@gmail.com>
Date: Wed, 5 Dec 2018 13:31:55 +0200
Subject: [PATCH] [ACA-2060] arrange viewer toolbar icons (#849)

* [ACA-2060] checks if empty

* [ACA-2060] hide last divider

* [ACA-2060] switch info drawer button & more menu

* [ACA-2060] cspell change

* [ACA-2060] apply changes only to viewer top toolbar
---
 src/app/components/preview/preview.component.html |  9 +++++----
 src/app/components/preview/preview.component.scss | 14 ++++++++++++++
 src/app/components/preview/preview.component.ts   |  1 +
 3 files changed, 20 insertions(+), 4 deletions(-)

diff --git a/src/app/components/preview/preview.component.html b/src/app/components/preview/preview.component.html
index dfe66d3ce..0e3eb9a5e 100644
--- a/src/app/components/preview/preview.component.html
+++ b/src/app/components/preview/preview.component.html
@@ -3,7 +3,7 @@
     <adf-viewer
         [fileNodeId]="nodeId"
         [allowNavigate]="navigateMultiple"
-        [allowSidebar]="true"
+        [allowSidebar]="hasRightSidebar"
         [allowPrint] ="false"
         [allowDownload]="false"
         [allowFullScreen]="false"
@@ -12,13 +12,14 @@
         [overlayMode]="true"
         (showViewerChange)="onVisibilityChanged($event)"
         (navigateBefore)="onNavigateBefore()"
-        (navigateNext)="onNavigateNext()">
+        (navigateNext)="onNavigateNext()"
+        [ngClass]="hasRightSidebar ? 'hide-last-divider': ''">
 
         <adf-viewer-sidebar>
             <aca-info-drawer [node]="selection.file"></aca-info-drawer>
         </adf-viewer-sidebar>
 
-        <adf-viewer-open-with>
+        <adf-viewer-open-with *ngIf="openWith.length">
             <ng-container *ngFor="let action of openWith; trackBy: trackByActionId">
                 <aca-toolbar-action type="menu-item" [actionRef]="action"></aca-toolbar-action>
             </ng-container>
@@ -30,7 +31,7 @@
             </ng-container>
         </adf-viewer-toolbar-actions>
 
-        <adf-viewer-more-actions>
+        <adf-viewer-more-actions *ngIf="viewerToolbarMoreActions.length">
             <ng-container *ngFor="let action of viewerToolbarMoreActions; trackBy: trackByActionId">
                 <aca-toolbar-action type="menu-item" [actionRef]="action"></aca-toolbar-action>
             </ng-container>
diff --git a/src/app/components/preview/preview.component.scss b/src/app/components/preview/preview.component.scss
index 714aad589..74f765b7d 100644
--- a/src/app/components/preview/preview.component.scss
+++ b/src/app/components/preview/preview.component.scss
@@ -2,3 +2,17 @@
   width: 100%;
   height: 100%;
 }
+
+.hide-last-divider .adf-viewer-toolbar .mat-toolbar {
+  adf-toolbar-divider:last-of-type {
+    display: none;
+  }
+
+  > button:last-of-type {
+    right: 40px;
+  }
+}
+
+#adf-viewer-moreactions {
+  right: -40px;
+}
diff --git a/src/app/components/preview/preview.component.ts b/src/app/components/preview/preview.component.ts
index e0b326e26..9a433a5b0 100644
--- a/src/app/components/preview/preview.component.ts
+++ b/src/app/components/preview/preview.component.ts
@@ -74,6 +74,7 @@ export class PreviewComponent extends PageComponent
   navigateMultiple = false;
   openWith: Array<ContentActionRef> = [];
   contentExtensions: Array<ViewerExtensionRef> = [];
+  hasRightSidebar = true;
 
   constructor(
     private contentApi: ContentApiService,