OHIF-15 Added two rendering options for cine dialog.

OHIFViewer/client/components/toolbarSection/toolbarSection.js

@@ -92,6 +92,8 @@ Template.toolbarSection.helpers({
             iconClasses: 'fa fa-undo'
         });
 
+        if (!OHIF.uiSettings.multiframeEnhancementsEnabled) {
+
             buttonData.push({
                 id: 'previousDisplaySet',
                 title: 'Previous',
@@ -122,6 +124,8 @@ Template.toolbarSection.helpers({
                 iconClasses: 'fa fa-youtube-play'
             });
 
+        }
+
         buttonData.push({
             id: 'layout',
             title: 'Layout',
@@ -221,7 +225,7 @@ Template.toolbarSection.onRendered(function() {
 
     instance.$('#layout').dropdown();
 
-    if (OHIF.uiSettings.showCineDialogOnRendered) {
+    if (OHIF.uiSettings.multiframeEnhancementsEnabled) {
         toggleCineDialog();
     }
 

Packages/viewerbase/client/components/viewer/cineDialog/cineDialog.html

@@ -1,5 +1,5 @@
 <template name="cineDialog">
-    {{#form id='cineDialog' class='dialog-animated noselect' schema=instance.schema api=instance.api}}
+    {{#form id='cineDialog' class=(getClassNames 'dialog-animated noselect') schema=instance.schema api=instance.api}}
         {{>inputHidden key='intervalId'}}
         <div class="cine-navigation">
             <div class="btn-group">

Packages/viewerbase/client/components/viewer/cineDialog/cineDialog.js

@@ -142,5 +142,10 @@ Template.cineDialog.helpers({
     displaySetDisabled(isNext) {
         Session.get('LayoutManagerUpdated');
         return !OHIF.viewer.canMoveDisplaySets(isNext) ? 'disabled' : '';
+    },
+
+    getClassNames(baseCls) {
+        return baseCls + ' ' + (OHIF.uiSettings.multiframeEnhancementsEnabled ? 'single' : 'double') + '-row-style'
     }
+
 });

Packages/viewerbase/client/components/viewer/cineDialog/cineDialog.styl

@@ -4,19 +4,10 @@
   theme('border', '2px solid $uiBorderColor', 0.95)
   theme('background', '$uiGrayDarkest', 0.95)
   theme('color', '$textSecondaryColor')
-  border-radius: 8px
-  bottom: 3px
-  box-sizing: border-box
-  height: 80px
-  left: 50%
-  overflow: hidden
-  padding: 10px
   position: absolute
-  right: 3px
+  border-radius: 8px
-  top: 2%
-  width: 300px
   z-index: 1000
-  transform(translateX(-50%) translateY(-50%))
+  overflow: hidden
 
   h5
     font-size: 20px
@@ -36,6 +27,13 @@
     &:active, &.active
       theme('color', '$activeColor')
 
+#cineDialog.double-row-style
+  box-sizing: border-box
+  width: 300px
+  height: 80px
+  padding: 10px
+  top: 2%
+  left: 35%
 
   .cine-navigation
     position: absolute
@@ -63,6 +61,68 @@
       float: right
       padding: 5px 10px 0 0
 
-    .loop-section
-      padding-right: 12px
 
+#cineDialog.single-row-style
+  box-sizing: content-box
+  width: 425px
+  height: 45px
+  padding: 4px 8px
+  top: 2%
+  left: 35%
+
+  .cine-navigation
+    position: relative
+    float: right
+    overflow: hidden
+
+    .btn
+      padding: 0 2px
+      margin: 0 0 0 2px
+      border: 0 none
+      &:first-of-type
+        margin-left: 0
+
+      i
+        font-size: 38px
+        line-height: 45px
+
+  .cine-controls
+    position: relative
+    float: left
+    overflow: hidden
+    .btn
+      font-size: 22px
+      line-height: 45px
+      min-width: 28px
+      padding: 0 0px
+      margin: 0 4px
+      border: 0 none
+
+  .cine-options
+    display: block
+    position: relative
+    width: 150px
+    float: left
+    overflow: hidden
+    padding: 0
+    margin: 0 0 0 10px
+
+    .fps-section
+      display: block
+      float: left
+      width: 80px
+      input[type="range"]
+        line-height: 45px
+        height: 45px
+        background-color: transparent
+        border: 0 none
+        outline: 0 none
+
+    #fps
+      display: block
+      width: 68px
+      float: left
+      padding: 0
+      margin: 0 0 0 2px
+      text-align: center
+      line-height: 45px

Packages/worklist/both/schema.js

@@ -149,9 +149,9 @@ export const UISettings = new SimpleSchema({
         label: 'The UP/DOWN display set navigation buttons will iterate over all the viewports at once?',
         defaultValue: false
     },
-    showCineDialogOnRendered: {
+    multiframeEnhancementsEnabled: {
         type: Boolean,
-        label: 'The Cine dialog will be displayed by default',
+        label: 'Enable cine dialog enhancements for multiframe images.',
         defaultValue: false
     }
 });