From 458d310c78d6d8d1f4b211f55c4f5ea2c0ceb33b Mon Sep 17 00:00:00 2001
From: Sanders DeNardi <sedenardi@users.noreply.github.com>
Date: Fri, 4 Oct 2019 09:26:05 -0400
Subject: [PATCH] fix: Move Series Information to Separate Row (#990)

* fix ThumbnailEntry alignment

* move series information to separate row
---
 .../components/studyBrowser/ThumbnailEntry.js | 15 ++++++-----
 .../studyBrowser/ThumbnailEntry.styl          | 26 +++++++------------
 2 files changed, 17 insertions(+), 24 deletions(-)

diff --git a/platform/ui/src/components/studyBrowser/ThumbnailEntry.js b/platform/ui/src/components/studyBrowser/ThumbnailEntry.js
index 19f4add7a..30f15e6c9 100644
--- a/platform/ui/src/components/studyBrowser/ThumbnailEntry.js
+++ b/platform/ui/src/components/studyBrowser/ThumbnailEntry.js
@@ -38,7 +38,6 @@ class ThumbnailEntry extends Component {
     let className = classnames('ThumbnailEntry noselect', {
       active: this.props.active,
     });
-    const infoOnly = false;
 
     let contents = null;
     if (this.props.imageSrc || this.props.imageId) {
@@ -54,8 +53,10 @@ class ThumbnailEntry extends Component {
       );
     } else if (this.props.altImageText) {
       contents = (
-        <div className={'alt-image-text p-x-1'}>
-          <h1>{this.props.altImageText}</h1>
+        <div className="p-x-1">
+          <div className="alt-image-text">
+            <h1>{this.props.altImageText}</h1>
+          </div>
         </div>
       );
     }
@@ -69,23 +70,23 @@ class ThumbnailEntry extends Component {
       >
         {contents}
         <div
-          className={infoOnly ? 'series-details info-only' : 'series-details'}
+          className="series-details"
         >
           <div className="series-description">
             {this.props.seriesDescription}
           </div>
           <div className="series-information">
-            <div className="item item-series clearfix">
+            <div className="item item-series">
               <div className="icon">S:</div>
               <div className="value">{this.props.seriesNumber}</div>
             </div>
             {hasInstanceNumber && (
-              <div className="item item-series clearfix">
+              <div className="item item-series">
                 <div className="icon">I:</div>
                 <div className="value">{this.props.instanceNumber}</div>
               </div>
             )}
-            <div className="item item-frames clearfix">
+            <div className="item item-frames">
               <div className="icon">
                 <div />
               </div>
diff --git a/platform/ui/src/components/studyBrowser/ThumbnailEntry.styl b/platform/ui/src/components/studyBrowser/ThumbnailEntry.styl
index d51a035a7..3ccc4a59e 100644
--- a/platform/ui/src/components/studyBrowser/ThumbnailEntry.styl
+++ b/platform/ui/src/components/studyBrowser/ThumbnailEntry.styl
@@ -31,8 +31,6 @@
       color: var(--text-primary-color);
 
   .series-details
-    display: flex;
-    flex-direction: row;
     color: var(--text-primary-color);
     font-size: 14px
     line-height: 1.3em
@@ -44,9 +42,6 @@
     position: relative
     word-wrap: break-word
 
-    .series-description
-        flex-grow: 1
-
     &.info-only
       .series-description
         display: none
@@ -67,36 +62,33 @@
             float: none
             line-height: 25px
 
-          .value
-            margin-left: 0
-            width: auto
 
   .series-information
-    padding-right: 4px
-    max-width: 50px
+    display: flex
+    flex-direction: row
+    align-items: center
+
+    .item
+      flex: 1
+      text-align: center
 
     .item-frames .icon
       height: 18px
 
     .value
       color: var(--text-secondary-color);
-      display: inline-block
-      float: right
+      display: inline
       font-size: 12px
       margin-left: 4px
-      overflow: hidden
       text-overflow: ellipsis
       white-space: nowrap
-      width: calc(100% - 15px)
 
     .icon
       color: var(--active-color);
-      display: inline-block
-      float: left
+      display: inline
       font-size: 10px
       font-weight: 900
       text-align: right
-      width: 11px
 
       div
         background-color: var(--series-count-background-color);