From 2e8644b4bc320852c9ea528bb89c7b80b921c408 Mon Sep 17 00:00:00 2001
From: Eugenio Romano <eugenio.romano@alfresco.com>
Date: Tue, 2 Aug 2016 10:25:18 +0100
Subject: [PATCH] hide close button viewer in not overlay mode

---
 .../src/viewer.component.html                 |  2 +-
 .../src/viewer.component.spec.ts              | 20 ++++++++++++++++++-
 2 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/ng2-components/ng2-alfresco-viewer/src/viewer.component.html b/ng2-components/ng2-alfresco-viewer/src/viewer.component.html
index c9eb7494b9..485406c28d 100644
--- a/ng2-components/ng2-alfresco-viewer/src/viewer.component.html
+++ b/ng2-components/ng2-alfresco-viewer/src/viewer.component.html
@@ -20,7 +20,7 @@
                     <!-- Start Navigation -->
                     <nav class="mdl-navigation">
                         <div id="viewer-toolbar-view-options">
-                            <button
+                            <button *ngIf="overlayMode"
                                 class="mdl-color--black mdl-button mdl-js-button mdl-button--fab mdl-button--mini-fab mdl-button--colored" (click)="close()">
                                 <i id="viewer-close-button" class="icon material-icons">close</i>
                             </button>
diff --git a/ng2-components/ng2-alfresco-viewer/src/viewer.component.spec.ts b/ng2-components/ng2-alfresco-viewer/src/viewer.component.spec.ts
index ac01431395..57d3a30a01 100644
--- a/ng2-components/ng2-alfresco-viewer/src/viewer.component.spec.ts
+++ b/ng2-components/ng2-alfresco-viewer/src/viewer.component.spec.ts
@@ -70,13 +70,14 @@ import { EventMock } from './assets/event.mock';
                  });
          }));
 
-         it('Close button should be present', inject([TestComponentBuilder], (tcb: TestComponentBuilder) => {
+         it('Close button should be present if overlay mode', inject([TestComponentBuilder], (tcb: TestComponentBuilder) => {
              return tcb
                  .createAsync(ViewerComponent)
                  .then((fixture) => {
                      let element = fixture.nativeElement;
                      let component = fixture.componentInstance;
                      component.urlFile = 'fake-url-file';
+                     component.overlayMode = true;
 
                      fixture.detectChanges();
 
@@ -84,6 +85,22 @@ import { EventMock } from './assets/event.mock';
                  });
          }));
 
+         it('Close button should be not present if is not overlay mode', inject([TestComponentBuilder], (tcb: TestComponentBuilder) => {
+             return tcb
+                 .createAsync(ViewerComponent)
+                 .then((fixture) => {
+                     let element = fixture.nativeElement;
+                     let component = fixture.componentInstance;
+                     component.urlFile = 'fake-url-file';
+                     component.overlayMode = false;
+
+                     fixture.detectChanges();
+
+                     expect(element.querySelector('#viewer-close-button')).toBeNull();
+                 });
+         }));
+
+
          it('Click on close button should hide the viewer', inject([TestComponentBuilder], (tcb: TestComponentBuilder) => {
              return tcb
                  .createAsync(ViewerComponent)
@@ -91,6 +108,7 @@ import { EventMock } from './assets/event.mock';
                      let element = fixture.nativeElement;
                      let component = fixture.componentInstance;
                      component.urlFile = 'fake-url-file';
+                     component.overlayMode = true;
 
                      fixture.detectChanges();
                      element.querySelector('#viewer-close-button').click();