From 5b930f8db2089c3aa2725fe9ca8a3b66d8866d48 Mon Sep 17 00:00:00 2001
From: Bruno Alves de Faria <brunoalvesdefaria@gmail.com>
Date: Wed, 22 Jun 2016 11:06:12 -0300
Subject: [PATCH] LT-251: Showing only one quick switck if there is no second
 viewport

---
 .../studySeriesQuickSwitch.html                   | 10 +++++-----
 .../studySeriesQuickSwitch.styl                   |  4 +++-
 .../components/toolbarSection/toolbarSection.html |  8 ++++++--
 .../components/toolbarSection/toolbarSection.js   | 15 +++++++++++++++
 Packages/viewerbase/lib/helpers/isSeriesActive.js |  3 +--
 5 files changed, 30 insertions(+), 10 deletions(-)

diff --git a/Packages/lesiontracker/client/components/studySeriesQuickSwitch/studySeriesQuickSwitch.html b/Packages/lesiontracker/client/components/studySeriesQuickSwitch/studySeriesQuickSwitch.html
index 70433b474..b51656c78 100644
--- a/Packages/lesiontracker/client/components/studySeriesQuickSwitch/studySeriesQuickSwitch.html
+++ b/Packages/lesiontracker/client/components/studySeriesQuickSwitch/studySeriesQuickSwitch.html
@@ -1,7 +1,7 @@
 <template name="studySeriesQuickSwitch">
     <div class="quickSwitchWrapper {{side}}">
-        <div class="quickSwitch js-quick-switch clearfix rp-t-1 {{#if eq side 'left'}}rp-r-3{{else}}rp-l-3{{/if}}">
-            <div class="switchSection switchSectionStudy rp-x-1 {{#if eq side 'left'}}pull-right{{else}}pull-left{{/if}}">
+        <div class="quickSwitch js-quick-switch clearfix rp-t-1 {{#if eq side 'right'}}rp-l-3{{/if}}{{#if eq side 'left'}}rp-r-3{{/if}}">
+            <div class="switchSection switchSectionStudy rp-x-1 {{#if eq side 'right'}}pull-left{{else}}pull-right{{/if}}">
                 <div class="label">Study</div>
                 <div class="studySwitch">
                     <div class="studyBox"></div>
@@ -12,16 +12,16 @@
                     </div>
                 </div>
             </div>
-            <div class="switchSection switchSectionSeries rp-x-1 {{#if eq side 'left'}}pull-right{{else}}pull-left{{/if}}">
+            <div class="switchSection switchSectionSeries rp-x-1 {{#if eq side 'right'}}pull-left{{else}}pull-right{{/if}}">
                 {{#let thumbnailsList=(studyThumbnails currentStudy)}}
                     <div class="label">Series</div>
                     <div class="seriesSwitch clearfix">
                         {{#each thumbnail in thumbnailsList}}
                             {{#if lt @index 8}}
-                                <div class="seriesItem {{#if eq side 'left'}}pull-right{{else}}pull-left{{/if}} {{#if isSeriesActive thumbnail.stack.seriesInstanceUid viewportIndex}}active{{/if}}"></div>
+                                <div class="seriesItem {{#if eq side 'right'}}pull-left{{else}}pull-right{{/if}} {{#if isSeriesActive thumbnail.stack.seriesInstanceUid viewportIndex}}active{{/if}}"></div>
                             {{else}}
                                 {{#if eq @index 8}}
-                                    <div class="seriesItem count {{#if eq side 'left'}}pull-right{{else}}pull-left{{/if}}">{{thumbnailsList.length}}</div>
+                                    <div class="seriesItem count {{#if eq side 'right'}}pull-left{{else}}pull-right{{/if}}">{{thumbnailsList.length}}</div>
                                 {{/if}}
                             {{/if}}
                         {{/each}}
diff --git a/Packages/lesiontracker/client/components/studySeriesQuickSwitch/studySeriesQuickSwitch.styl b/Packages/lesiontracker/client/components/studySeriesQuickSwitch/studySeriesQuickSwitch.styl
index f135f38f1..c2572faba 100644
--- a/Packages/lesiontracker/client/components/studySeriesQuickSwitch/studySeriesQuickSwitch.styl
+++ b/Packages/lesiontracker/client/components/studySeriesQuickSwitch/studySeriesQuickSwitch.styl
@@ -9,7 +9,9 @@ $seriesSpacing = 2px
 
   &.middle
     left: 50%
-    transform(translateX(-50%))
+
+    .quickSwitch
+      transform(translateX(-50%))
 
   &.left
     border-right: 1px solid $uiBorderColor
diff --git a/Packages/lesiontracker/client/components/toolbarSection/toolbarSection.html b/Packages/lesiontracker/client/components/toolbarSection/toolbarSection.html
index 733d68d6e..22e056c3a 100644
--- a/Packages/lesiontracker/client/components/toolbarSection/toolbarSection.html
+++ b/Packages/lesiontracker/client/components/toolbarSection/toolbarSection.html
@@ -21,8 +21,12 @@
                 </div>
             </div>
 
-            {{ >studySeriesQuickSwitch side="left" viewportIndex=0}}
-            {{ >studySeriesQuickSwitch side="right" viewportIndex=1}}
+            {{#if splitView}}
+                {{>studySeriesQuickSwitch side="right" viewportIndex=1}}
+                {{>studySeriesQuickSwitch side="left" viewportIndex=0}}
+            {{else}}
+                {{>studySeriesQuickSwitch side="middle" viewportIndex=0}}
+            {{/if}}
 
             <div class="toolbarSectionEntry pull-right rm-l-1 p-x-1">
                 {{ >caseProgress }}
diff --git a/Packages/lesiontracker/client/components/toolbarSection/toolbarSection.js b/Packages/lesiontracker/client/components/toolbarSection/toolbarSection.js
index 0e6e19fdb..9a3053226 100644
--- a/Packages/lesiontracker/client/components/toolbarSection/toolbarSection.js
+++ b/Packages/lesiontracker/client/components/toolbarSection/toolbarSection.js
@@ -1,4 +1,17 @@
 Template.toolbarSection.helpers({
+    // Returns true if the view shall be split in two viewports
+    splitView() {
+        // Stops here if layout manager is not defined yet
+        if (!window.layoutManager) {
+            return;
+        }
+
+        // Run this computation everytime the viewports are updated
+        Session.get('LayoutManagerUpdated');
+
+        return layoutManager.viewportData.length > 1;
+    },
+
     leftSidebarToggleButtonData() {
         const instance = Template.instance();
         return {
@@ -14,6 +27,7 @@ Template.toolbarSection.helpers({
             }]
         };
     },
+
     rightSidebarToggleButtonData() {
         const instance = Template.instance();
         return {
@@ -35,6 +49,7 @@ Template.toolbarSection.helpers({
             }]
         };
     },
+
     toolbarButtons: function() {
         var buttonData = [];
         buttonData.push({
diff --git a/Packages/viewerbase/lib/helpers/isSeriesActive.js b/Packages/viewerbase/lib/helpers/isSeriesActive.js
index 2d88709ad..5bca4784b 100644
--- a/Packages/viewerbase/lib/helpers/isSeriesActive.js
+++ b/Packages/viewerbase/lib/helpers/isSeriesActive.js
@@ -2,8 +2,7 @@
  * Boolean helper to identify if a series instance is active in some viewport
  */
 Template.registerHelper('isSeriesActive', (seriesInstanceUid, viewportIndex) => {
-    // Register dependency on Session key that is changed
-    // when viewports are updated
+    // Run this computation everytime the viewports are updated
     Session.get('LayoutManagerUpdated');
 
     // Stop here if layoutManager is not defined yet