From a6c45ddd2782e30d6f372452315591b2b2f0b55f Mon Sep 17 00:00:00 2001 From: Erik Ziegler Date: Fri, 22 Jul 2016 14:26:29 +0200 Subject: [PATCH] Updating design of OHIF Viewer using (OHIF-49) --- LesionTracker/client/body.styl | 21 +--- LesionTracker/client/debugReactivity.js | 2 + LesionTracker/client/head.html | 2 - LesionTracker/defaultSettings.js | 2 + OHIFViewer/.meteor/packages | 3 +- OHIFViewer/.meteor/versions | 1 + .../flexboxLayout/flexboxLayout.html | 13 ++ .../components/flexboxLayout/flexboxLayout.js | 19 +++ .../flexboxLayout/flexboxLayout.styl | 80 ++++++++++++ .../client/components/layout/layout.html | 29 ++--- .../client/components/layout/layout.styl | 63 +++------- .../client/components/viewer/viewer.html | 26 ++-- OHIFViewer/client/components/viewer/viewer.js | 59 ++++++--- .../client/components/viewer/viewer.styl | 58 ++++----- OHIFViewer/client/routeHooks.js | 5 - OHIFViewer/client/routes.js | 6 + OHIFViewer/defaultSettings.js | 2 + Packages/cornerstone/package.js | 2 +- Packages/design/package.js | 2 +- Packages/design/styles/common/global.styl | 8 ++ Packages/design/styles/imports/variables.styl | 7 ++ .../hangingProtocolButtons.html | 4 +- .../matchedProtocols/matchedProtocols.html | 6 - .../matchedProtocols/matchedProtocols.js | 35 +----- .../protocolEditor/protocolEditor.html | 2 +- .../protocolEditor/protocolEditor.styl | 52 +++----- .../components/ruleTable/ruleTable.styl | 36 +++--- .../settingsTable/settingsTable.styl | 40 +++--- .../components/stageDetails/stageDetails.styl | 9 +- .../stageSortable/stageSortable.styl | 54 ++++---- Packages/hangingprotocols/package.js | 4 +- .../lesionTrackerViewportOverlay.styl | 102 --------------- .../client/components/viewer/viewer.styl | 28 +---- Packages/lesiontracker/package.js | 7 +- Packages/lesiontracker/server/collections.js | 2 + Packages/ohif-core/package.js | 2 +- Packages/validatejs/package.js | 2 +- .../viewer/gridLayout/gridLayout.styl | 8 +- .../imageViewerViewport.styl | 27 ++-- .../viewer/layoutChooser/layoutChooser.styl | 10 +- .../loadingIndicator/loadingIndicator.styl | 6 +- .../viewer/playClipButton/playClipButton.html | 18 ++- .../simpleToolbarButton.html | 8 +- .../simpleToolbarButton.js | 13 ++ .../components/viewer/toolbar/toolbar.html | 58 +++++---- .../components/viewer/toolbar/toolbar.styl | 33 ++--- .../toolbarGroupButton.html | 12 -- .../toolbarGroupButton/toolbarGroupButton.js | 0 .../toolbarGroupButton.styl | 14 --- .../viewer}/viewerMain/viewerMain.html | 0 .../viewer}/viewerMain/viewerMain.js | 0 .../viewer}/viewerMain/viewerMain.styl | 0 .../viewportOverlay/viewportOverlay.styl | 116 +++++++++++++++--- Packages/viewerbase/lib/isImage.js | 59 +++++++++ Packages/viewerbase/lib/sopClassDictionary.js | 115 +++++++++++++++++ Packages/viewerbase/package.js | 28 ++--- .../worklist/client/components/worklist.styl | 9 -- .../worklistResult/worklistResult.js | 4 +- Packages/worklist/package.js | 8 +- 59 files changed, 776 insertions(+), 565 deletions(-) mode change 100755 => 100644 OHIFViewer/.meteor/packages create mode 100644 OHIFViewer/client/components/flexboxLayout/flexboxLayout.html create mode 100644 OHIFViewer/client/components/flexboxLayout/flexboxLayout.js create mode 100644 OHIFViewer/client/components/flexboxLayout/flexboxLayout.styl delete mode 100644 OHIFViewer/client/routeHooks.js delete mode 100644 Packages/lesiontracker/client/components/lesionTrackerViewportOverlay/lesionTrackerViewportOverlay.styl create mode 100644 Packages/viewerbase/client/components/viewer/simpleToolbarButton/simpleToolbarButton.js delete mode 100644 Packages/viewerbase/client/components/viewer/toolbarGroupButton/toolbarGroupButton.html delete mode 100644 Packages/viewerbase/client/components/viewer/toolbarGroupButton/toolbarGroupButton.js delete mode 100644 Packages/viewerbase/client/components/viewer/toolbarGroupButton/toolbarGroupButton.styl rename Packages/{lesiontracker/client/components => viewerbase/client/components/viewer}/viewerMain/viewerMain.html (100%) rename Packages/{lesiontracker/client/components => viewerbase/client/components/viewer}/viewerMain/viewerMain.js (100%) rename Packages/{lesiontracker/client/components => viewerbase/client/components/viewer}/viewerMain/viewerMain.styl (100%) create mode 100644 Packages/viewerbase/lib/isImage.js create mode 100644 Packages/viewerbase/lib/sopClassDictionary.js diff --git a/LesionTracker/client/body.styl b/LesionTracker/client/body.styl index c4b7c9834..a33168d3d 100644 --- a/LesionTracker/client/body.styl +++ b/LesionTracker/client/body.styl @@ -6,18 +6,6 @@ body .logoContainer width: 15% - .navbar-brand - img - height: 20px - margin-top: 10px - margin-bottom: 10px - margin-right: 5px - - h4.name - padding: 1px - font-size: 0.3em - width: 50% - table#tblStudyList table-layout: fixed font-size: 0.7em @@ -25,14 +13,7 @@ body #tablist width: 85% overflow-x: auto - - #toolbar .btn-group button - width: 40px - height: 40px - - .studyDateList - margin-top: 5px - + @media (max-device-width: 667px) #toolbar .btn-group button width: 35px diff --git a/LesionTracker/client/debugReactivity.js b/LesionTracker/client/debugReactivity.js index d8878b45d..25e1e6d79 100644 --- a/LesionTracker/client/debugReactivity.js +++ b/LesionTracker/client/debugReactivity.js @@ -1,3 +1,5 @@ +import { Meteor } from 'meteor/meteor'; + Meteor.startup(function() { var debug = false; diff --git a/LesionTracker/client/head.html b/LesionTracker/client/head.html index bdacefdfe..aa80f2467 100644 --- a/LesionTracker/client/head.html +++ b/LesionTracker/client/head.html @@ -5,8 +5,6 @@ - - diff --git a/LesionTracker/defaultSettings.js b/LesionTracker/defaultSettings.js index c3843d55f..5358d950c 100644 --- a/LesionTracker/defaultSettings.js +++ b/LesionTracker/defaultSettings.js @@ -1,3 +1,5 @@ +import { Meteor } from 'meteor/meteor'; + Meteor.startup(function() { if (Object.keys(Meteor.settings).length > 1) { console.log('Using custom LesionTracker settings'); diff --git a/OHIFViewer/.meteor/packages b/OHIFViewer/.meteor/packages old mode 100755 new mode 100644 index fd3a53c9a..7ab512e73 --- a/OHIFViewer/.meteor/packages +++ b/OHIFViewer/.meteor/packages @@ -27,13 +27,14 @@ check cornerstone dicomweb design +ohif:core +ecmascript viewerbase worklist reactive-var practicalmeteor:loglevel hangingprotocols reactive-dict -gilbertwat:bootstrap3-daterangepicker clinical:router fastclick standard-minifier-css diff --git a/OHIFViewer/.meteor/versions b/OHIFViewer/.meteor/versions index 22d5de02b..5fcd74e6c 100644 --- a/OHIFViewer/.meteor/versions +++ b/OHIFViewer/.meteor/versions @@ -66,6 +66,7 @@ mrt:moment@2.8.1 natestrauser:select2@4.0.2 npm-mongo@1.4.45 observe-sequence@1.0.12 +ohif:core@0.0.1 ordered-dict@1.0.8 orthanc-remote@0.0.1 practicalmeteor:chai@2.1.0_1 diff --git a/OHIFViewer/client/components/flexboxLayout/flexboxLayout.html b/OHIFViewer/client/components/flexboxLayout/flexboxLayout.html new file mode 100644 index 000000000..28277e263 --- /dev/null +++ b/OHIFViewer/client/components/flexboxLayout/flexboxLayout.html @@ -0,0 +1,13 @@ + diff --git a/OHIFViewer/client/components/flexboxLayout/flexboxLayout.js b/OHIFViewer/client/components/flexboxLayout/flexboxLayout.js new file mode 100644 index 000000000..b5747af7c --- /dev/null +++ b/OHIFViewer/client/components/flexboxLayout/flexboxLayout.js @@ -0,0 +1,19 @@ +Template.flexboxLayout.events({ + 'transitionend .sidebarMenu'(event, instance) { + handleResize(); + }, + 'transitionend .sidebar-option'(event, instance) { + // Prevent this event from bubbling + event.stopPropagation(); + } +}); + +Template.flexboxLayout.helpers({ + leftSidebarOpen() { + return Template.instance().data.state.get('leftSidebar'); + }, + + rightSidebarOpen() { + return Template.instance().data.state.get('rightSidebar'); + } +}); diff --git a/OHIFViewer/client/components/flexboxLayout/flexboxLayout.styl b/OHIFViewer/client/components/flexboxLayout/flexboxLayout.styl new file mode 100644 index 000000000..fe7a544d5 --- /dev/null +++ b/OHIFViewer/client/components/flexboxLayout/flexboxLayout.styl @@ -0,0 +1,80 @@ +@import "{design}/app" + +$lesionsSidebarMenuWidth = 450px + +.viewerSection + display: flex + flex: 1 + flex-flow: row nowrap + align-items: stretch + height: calc(100% - 78px); + width: 100% + + .sidebarMenu + height: 100% + // required transformation to make inner fixed elements relative to this one + transform(scale(1)) + transition($sidebarTransition) + + .sidebar-option + height: 100% + position: fixed + transform(translateX(100%)) + transition($sidebarTransition) + width: 100% + + &.active + transform(translateX(0%)) + + .sidebar-left + max-width: $studiesSidebarMenuWidth + order: 1 + flex: 1 + margin-left: - $studiesSidebarMenuWidth + border-right: $uiBorderThickness $uiBorderColor solid + + &.sidebar-open + margin-left: 0 + + .mainContent + flex: 1 + order: 2 + width: 100% + height: 100% + overflow: hidden + transition($sidebarTransition) + + .sidebar-right + flex: 1 + margin-left: -6px + margin-right: - $lesionsSidebarMenuWidth + max-width: $lesionsSidebarMenuWidth + order: 3 + padding-left: 6px + position: relative + + &[data-timepoints="3"] + margin-right: - ($lesionsSidebarMenuWidth + 135.5px) + max-width: $lesionsSidebarMenuWidth + 135.5px + + &[data-timepoints="4"] + margin-right: - ($lesionsSidebarMenuWidth + 270px) + max-width: $lesionsSidebarMenuWidth + 270px + + &:before + background-color: $uiBorderColor + content: '' + display: block + height: 100% + left: 5px + position: absolute + top: 0 + width: $uiBorderThickness + + &.sidebar-open + margin-right: 0 + + .studiesListedChanger + padding: 20px 10px + text-align: center + border-bottom: $uiBorderThickness $uiBorderColor solid diff --git a/OHIFViewer/client/components/layout/layout.html b/OHIFViewer/client/components/layout/layout.html index dda5b3fa5..02519a366 100644 --- a/OHIFViewer/client/components/layout/layout.html +++ b/OHIFViewer/client/components/layout/layout.html @@ -1,25 +1,12 @@