Made tabs open before study loads. Added study loading indicator

LesionTracker/client/components/viewer.html

@@ -1,5 +1,5 @@
 <template name="viewer">
-    <div id="viewer" class="viewer">
+    <div id="viewer">
         {{ >lesionDialog }}
         {{ >hidingPanel }}
         <div id="viewportAndLesionTable">

LesionTracker/client/components/viewer.styl

@@ -1,4 +1,4 @@
-.viewer
+#viewer
   width: 100%
   height: 100%
   margin: 0

Packages/cornerstone/client/hangingProtocol.js

@@ -5,6 +5,11 @@ function defaultHangingProtocol(inputData) {
     var viewportRows = inputData.viewportRows;
     var viewportColumns = inputData.viewportColumns;
 
+    if (!studies.length) {
+        log.warn("No studies provided to Hanging Protocol");
+        return;
+    }
+
     // This is the most basic hanging protocol.
     var stacks = [];
     studies.forEach(function(study) {

Packages/viewerbase/components/basic/loadingText/loadingText.styl

@@ -1,4 +1,5 @@
 .loadingTextDiv
   margin: 30px
+
   h5
     text-align: center

Packages/worklist/components/tabContent/tabContent.html

@@ -1,4 +1,5 @@
 <template name="tabContent">
     <div role="tabpanel" class="tab-pane {{active}}" id="{{contentid}}">
+        {{ >loadingText }}
     </div>
 </template>

Packages/worklist/components/tabContent/tabContent.styl

@@ -0,0 +1,4 @@
+.tab-pane
+  .loadingTextDiv
+    h5
+      color: #777

Packages/worklist/components/tabTitle/tabTitle.styl

@@ -1 +1,4 @@
- 
+.tabTitle
+  button.close    
+    // Prevent blue glow around button in Chrome
+    outline: none 

Packages/worklist/components/worklist.js

@@ -54,6 +54,8 @@ switchToTab = function(contentId) {
 
         Session.set('studies', data.studies);
 
+        // Remove the loading text template that is inside by default
+        container.innerHTML = "";
         UI.renderWithData(Template.viewer, data, container);
         var imageViewer = $("#viewer");
         if (imageViewer) {
@@ -67,24 +69,20 @@ switchToTab = function(contentId) {
 };
 
 
-openNewTab = function(studyInstanceUid) {
-    getStudyMetadata(studyInstanceUid, function(study) {
-        var title = study.seriesList[0].instances[0].patientName;
-        var contentid = generateUUID();
+openNewTab = function(studyInstanceUid, title) {
+    var contentid = generateUUID();
+    var data = {
+        title: title,
+        contentid: contentid,
+    };
+    tabs.insert(data);
 
-        var data = {
-            title: title,
-            contentid: contentid,
-        };
-        tabs.insert(data);
-
-        ViewerData[contentid] = {
-            title: title,
-            contentid: contentid,
-            studyInstanceUid: studyInstanceUid
-        };
-        switchToTab(contentid);
-    });
+    ViewerData[contentid] = {
+        title: title,
+        contentid: contentid,
+        studyInstanceUid: studyInstanceUid
+    };
+    switchToTab(contentid);
 };
 
 Template.worklist.onRendered(function() {

Packages/worklist/components/worklistStudy/worklistStudy.js

@@ -1,5 +1,5 @@
 Template.worklistStudy.events({
     'click': function () {
-        openNewTab(this.studyInstanceUid);
+        openNewTab(this.studyInstanceUid, this.patientName);
     }
 });

Packages/worklist/package.js

@@ -29,6 +29,7 @@ Package.onUse(function (api) {
     api.addFiles('components/tabTitle/tabTitle.styl', 'client');
 
     api.addFiles('components/tabContent/tabContent.html', 'client');
+    api.addFiles('components/tabContent/tabContent.styl', 'client');
 
     api.addFiles('components/worklistStudy/worklistStudy.html', 'client');
     api.addFiles('components/worklistStudy/worklistStudy.js', 'client');