From cee09fcf8d4bd7c288e09ee6a4ace06208e38500 Mon Sep 17 00:00:00 2001
From: Rodrigo Antinarelli <rodrigoantinarelli@gmail.com>
Date: Wed, 29 Apr 2020 15:44:19 -0300
Subject: [PATCH] feat: OHIF-120 - Study List Route Template (#1693)

* fix pagination size

* clean up, start a very basic route configuration

* quick navigation between routes

* configure webpack to read .env variables

* export studylist filter

* split studylist filter into a component

* minor fix in studylist pagination

* update studylistfilter imports

* create studylist container with study list content

* moving file

* study list filter doc page

* catch change in ui-v2

* fix: 1px gap that shows text underneath

* Shift to routes folder

Co-authored-by: Danny Brown <danny.ri.brown@gmail.com>
---
 platform/ui/index.js                          |   1 +
 .../StudyListFilter/StudyListFilter.jsx}      |   7 +-
 .../StudyListFilter/StudyListFilter.mdx       | 118 +++++++
 .../src/components/StudyListFilter/index.js   |   1 +
 .../StudyListPagination.js                    |  17 +-
 platform/ui/src/components/index.js           |   2 +
 platform/ui/src/views/StudyList/StudyList.js  |  16 +-
 platform/ui/src/views/StudyList/StudyList.mdx |   6 +-
 platform/viewer/.webpack/webpack.pwa.js       |   2 +-
 .../StudyListContainer/StudyListContainer.jsx | 319 +++++++++++++++++-
 platform/viewer/src/routes/index.js           |   1 +
 11 files changed, 460 insertions(+), 30 deletions(-)
 rename platform/ui/src/{views/StudyList/components/StudyListFilter.js => components/StudyListFilter/StudyListFilter.jsx} (96%)
 create mode 100644 platform/ui/src/components/StudyListFilter/StudyListFilter.mdx
 create mode 100644 platform/ui/src/components/StudyListFilter/index.js

diff --git a/platform/ui/index.js b/platform/ui/index.js
index df6706cf9..0097336b1 100644
--- a/platform/ui/index.js
+++ b/platform/ui/index.js
@@ -35,6 +35,7 @@ export {
   StudyBrowser,
   StudyItem,
   StudyListExpandedRow,
+  StudyListFilter,
   StudyListPagination,
   StudyListTable,
   Svg,
diff --git a/platform/ui/src/views/StudyList/components/StudyListFilter.js b/platform/ui/src/components/StudyListFilter/StudyListFilter.jsx
similarity index 96%
rename from platform/ui/src/views/StudyList/components/StudyListFilter.js
rename to platform/ui/src/components/StudyListFilter/StudyListFilter.jsx
index 229e084a0..eed2cb795 100644
--- a/platform/ui/src/views/StudyList/components/StudyListFilter.js
+++ b/platform/ui/src/components/StudyListFilter/StudyListFilter.jsx
@@ -73,8 +73,11 @@ const StudyListFilter = ({
           </div>
         </div>
       </div>
-      <div className="sticky z-10 border-b-4 border-black" style={{ top: 58 }}>
-        <div className="bg-custom-navyDark pt-3 pb-3 ">
+      <div
+        className="sticky z-10 border-b-4 border-black"
+        style={{ top: '57px' }}
+      >
+        <div className="bg-primary-dark pt-3 pb-3 ">
           <InputGroup
             inputMeta={filtersMeta}
             values={filterValues}
diff --git a/platform/ui/src/components/StudyListFilter/StudyListFilter.mdx b/platform/ui/src/components/StudyListFilter/StudyListFilter.mdx
new file mode 100644
index 000000000..ca4ba702c
--- /dev/null
+++ b/platform/ui/src/components/StudyListFilter/StudyListFilter.mdx
@@ -0,0 +1,118 @@
+---
+name: Study List Filter
+menu: Data Display
+route: components/studyListFilter
+---
+
+import { useState } from 'react';
+import { Playground, Props } from 'docz';
+import { StudyListFilter } from '@ohif/ui';
+
+# Study List Filter
+
+## Import
+
+```javascript
+import { StudyListFilter } from '@ohif/ui';
+```
+
+## Basic usage
+
+<Playground>
+  {() => {
+    const defaultFilterValues = {
+      patientName: '',
+      mrn: '',
+      studyDate: {
+        startDate: null,
+        endDate: null,
+      },
+      description: '',
+      modality: undefined,
+      accession: '',
+      sortBy: '',
+      sortDirection: 'none',
+      page: 0,
+      resultsPerPage: 25,
+    };
+    const [filterValues, setFilterValues] = useState(defaultFilterValues);
+    const filtersMeta = [
+      {
+        name: 'patientName',
+        displayName: 'Patient Name',
+        inputType: 'Text',
+        isSortable: true,
+        gridCol: 4,
+      },
+      {
+        name: 'mrn',
+        displayName: 'MRN',
+        inputType: 'Text',
+        isSortable: true,
+        gridCol: 2,
+      },
+      {
+        name: 'studyDate',
+        displayName: 'Study date',
+        inputType: 'DateRange',
+        isSortable: true,
+        gridCol: 5,
+      },
+      {
+        name: 'description',
+        displayName: 'Description',
+        inputType: 'Text',
+        isSortable: true,
+        gridCol: 4,
+      },
+      {
+        name: 'modality',
+        displayName: 'Modality',
+        inputType: 'MultiSelect',
+        inputProps: {
+          options: [
+            { value: 'SEG', label: 'SEG' },
+            { value: 'CT', label: 'CT' },
+            { value: 'MR', label: 'MR' },
+            { value: 'SR', label: 'SR' },
+          ],
+        },
+        isSortable: true,
+        gridCol: 3,
+      },
+      {
+        name: 'accession',
+        displayName: 'Accession',
+        inputType: 'Text',
+        isSortable: true,
+        gridCol: 4,
+      },
+      {
+        name: 'instances',
+        displayName: 'Instances',
+        inputType: 'None',
+        isSortable: true,
+        gridCol: 2,
+      },
+    ];
+    const isFiltering = (filterValues, defaultFilterValues) => {
+      return Object.keys(defaultFilterValues).some(name => {
+        return filterValues[name] !== defaultFilterValues[name];
+      });
+    };
+    return (
+      <StudyListFilter
+        numOfStudies={100}
+        filtersMeta={filtersMeta}
+        filterValues={filterValues}
+        setFilterValues={setFilterValues}
+        clearFilters={() => setFilterValues(defaultFilterValues)}
+        isFiltering={isFiltering(filterValues, defaultFilterValues)}
+      />
+    );
+  }}
+</Playground>
+
+## Properties
+
+<Props of={StudyListFilter} />
diff --git a/platform/ui/src/components/StudyListFilter/index.js b/platform/ui/src/components/StudyListFilter/index.js
new file mode 100644
index 000000000..c16fbbcb6
--- /dev/null
+++ b/platform/ui/src/components/StudyListFilter/index.js
@@ -0,0 +1 @@
+export { default } from './StudyListFilter';
diff --git a/platform/ui/src/components/StudyListPagination/StudyListPagination.js b/platform/ui/src/components/StudyListPagination/StudyListPagination.js
index 05f2f64f7..4adbe1a07 100644
--- a/platform/ui/src/components/StudyListPagination/StudyListPagination.js
+++ b/platform/ui/src/components/StudyListPagination/StudyListPagination.js
@@ -1,6 +1,6 @@
 import React from 'react';
 import PropTypes from 'prop-types';
-import { Button, ButtonGroup, Icon, IconButton, Typography } from '@ohif/ui';
+import { Button, ButtonGroup, Icon, Typography } from '@ohif/ui';
 
 const StudyListPagination = ({
   onChangePage,
@@ -21,8 +21,7 @@ const StudyListPagination = ({
             <div className="relative mr-3">
               <select
                 defaultValue={perPage}
-                className="block appearance-none w-full bg-transparent border border-custom-darkSlateBlue text-white text-base px-2 pr-4 rounded leading-tight focus:outline-none"
-                style={{ height: 28 }}
+                className="block appearance-none w-full bg-transparent border border-common-active text-white text-base px-2 pr-4 rounded leading-tight focus:outline-none h-8"
                 onChange={e => onChangePerPage(e.target.value)}
                 onBlur={() => {}}
               >
@@ -48,22 +47,22 @@ const StudyListPagination = ({
                 Page {currentPage}
               </Typography>
               <ButtonGroup color="primary">
-                <IconButton
-                  size="inherit"
+                <Button
+                  size="initial"
                   className="border-common-active px-4 py-2 text-base"
                   color="white"
                   onClick={() => navigateToPage(1)}
                 >
-                  <>{`<<`}</>
-                </IconButton>
+                  {`<<`}
+                </Button>
                 <Button
-                  size="inherit"
+                  size="initial"
                   className="border-common-active py-2 px-2 text-base"
                   color="white"
                   onClick={() => navigateToPage(currentPage - 1)}
                 >{`< Previous`}</Button>
                 <Button
-                  size="inherit"
+                  size="initial"
                   className="border-common-active py-2 px-4 text-base"
                   color="white"
                   onClick={() => navigateToPage(currentPage + 1)}
diff --git a/platform/ui/src/components/index.js b/platform/ui/src/components/index.js
index bf9f590aa..72e217170 100644
--- a/platform/ui/src/components/index.js
+++ b/platform/ui/src/components/index.js
@@ -24,6 +24,7 @@ import StudyListPagination from './StudyListPagination';
 import StudyListTable from './StudyListTable';
 import Svg from './Svg';
 import StudyItem from './StudyItem';
+import StudyListFilter from './StudyListFilter';
 import Table from './Table';
 import TableBody from './TableBody';
 import TableCell from './TableCell';
@@ -66,6 +67,7 @@ export {
   StudyBrowser,
   StudyItem,
   StudyListExpandedRow,
+  StudyListFilter,
   StudyListPagination,
   StudyListTable,
   Svg,
diff --git a/platform/ui/src/views/StudyList/StudyList.js b/platform/ui/src/views/StudyList/StudyList.js
index d66d2bb30..24cf4ee8a 100644
--- a/platform/ui/src/views/StudyList/StudyList.js
+++ b/platform/ui/src/views/StudyList/StudyList.js
@@ -13,11 +13,11 @@ import {
   Button,
   StudyListPagination,
   StudyListTable,
+  StudyListFilter,
 } from '@ohif/ui';
 
 // fix imports after refactor
 import Header from './components/Header';
-import StudyListFilter from './components/StudyListFilter';
 
 const filtersMeta = [
   {
@@ -96,7 +96,7 @@ const defaultFilterValues = {
 };
 
 const isFiltering = (filterValues, defaultFilterValues) => {
-  return Object.keys(defaultFilterValues).some((name) => {
+  return Object.keys(defaultFilterValues).some(name => {
     return filterValues[name] !== defaultFilterValues[name];
   });
 };
@@ -109,7 +109,7 @@ const StudyList = () => {
 
   const tableDataSource = studies.map((study, key) => {
     const rowKey = key + 1;
-    const isExpanded = expandedRows.some((k) => k === rowKey);
+    const isExpanded = expandedRows.some(k => k === rowKey);
     const {
       AccessionNumber,
       Modalities,
@@ -128,7 +128,7 @@ const StudyList = () => {
       Instances: 'Instances',
     };
 
-    const seriesTableDataSource = series.map((seriesItem) => {
+    const seriesTableDataSource = series.map(seriesItem => {
       const { SeriesNumber, Modality, instances } = seriesItem;
       return {
         description: 'Patient Protocol',
@@ -238,8 +238,8 @@ const StudyList = () => {
         </StudyListExpandedRow>
       ),
       onClickRow: () =>
-        setExpandedRows((s) =>
-          isExpanded ? s.filter((n) => rowKey !== n) : [...s, rowKey]
+        setExpandedRows(s =>
+          isExpanded ? s.filter(n => rowKey !== n) : [...s, rowKey]
         ),
       isExpanded,
     };
@@ -248,13 +248,13 @@ const StudyList = () => {
   const [currentPage, setCurrentPage] = useState(1);
   const [perPage, setPerPage] = useState(25);
   const totalPages = Math.floor(numOfStudies / perPage);
-  const onChangePage = (page) => {
+  const onChangePage = page => {
     if (page > totalPages) {
       return;
     }
     setCurrentPage(page);
   };
-  const onChangePerPage = (perPage) => {
+  const onChangePerPage = perPage => {
     setPerPage(perPage);
     setCurrentPage(1);
   };
diff --git a/platform/ui/src/views/StudyList/StudyList.mdx b/platform/ui/src/views/StudyList/StudyList.mdx
index 1efd6ec7e..cb5d7979e 100644
--- a/platform/ui/src/views/StudyList/StudyList.mdx
+++ b/platform/ui/src/views/StudyList/StudyList.mdx
@@ -18,12 +18,10 @@ import {
   EmptyStudies,
   StudyListTable,
   StudyListPagination,
+  StudyListFilter
 } from '@ohif/ui';
 
-<!-- TEMPORARY IMPORTING -- SHOULD USE InputGroup after PR #1580 gets merged  -->
 
-import StudyListFilter from './components/StudyListFilter';
-import { getMockedStudies } from '../../utils/';
 import classnames from 'classnames';
 import moment from 'moment';
 
@@ -259,7 +257,7 @@ components.
       };
     });
     const handleStudyList = (number) => {
-      const studies = getMockedStudies(number);
+      const studies = utils.getMockedStudies(number);
       setStudies(studies);
       setCurrentPage(1);
     };
diff --git a/platform/viewer/.webpack/webpack.pwa.js b/platform/viewer/.webpack/webpack.pwa.js
index 014688769..6de09253d 100644
--- a/platform/viewer/.webpack/webpack.pwa.js
+++ b/platform/viewer/.webpack/webpack.pwa.js
@@ -94,7 +94,7 @@ module.exports = (env, argv) => {
         swDest: 'sw.js',
         swSrc: path.join(SRC_DIR, 'service-worker.js'),
         // Increase the limit to 4mb:
-        // maximumFileSizeToCacheInBytes: 4 * 1024 * 1024
+        maximumFileSizeToCacheInBytes: 4 * 1024 * 1024,
       }),
     ],
     // https://webpack.js.org/configuration/dev-server/
diff --git a/platform/viewer/src/routes/StudyListContainer/StudyListContainer.jsx b/platform/viewer/src/routes/StudyListContainer/StudyListContainer.jsx
index abd02330c..fd4457b57 100644
--- a/platform/viewer/src/routes/StudyListContainer/StudyListContainer.jsx
+++ b/platform/viewer/src/routes/StudyListContainer/StudyListContainer.jsx
@@ -1,13 +1,320 @@
-import React from 'react';
+import React, { useState } from 'react';
+import classnames from 'classnames';
 import { Link } from 'react-router-dom';
+import moment from 'moment';
+
+import {
+  utils,
+  Icon,
+  StudyListExpandedRow,
+  Button,
+  NavBar,
+  Svg,
+  IconButton,
+  EmptyStudies,
+  StudyListTable,
+  StudyListPagination,
+  StudyListFilter,
+} from '@ohif/ui';
+
+function StudyListContainer() {
+  const defaultFilterValues = {
+    patientName: '',
+    mrn: '',
+    studyDate: {
+      startDate: null,
+      endDate: null,
+    },
+    description: '',
+    modality: undefined,
+    accession: '',
+    sortBy: '',
+    sortDirection: 'none',
+    page: 0,
+    resultsPerPage: 25,
+  };
+  const [filterValues, setFilterValues] = useState(defaultFilterValues);
+  const studies = utils.getMockedStudies(100);
+  const numOfStudies = studies.length;
+  const [expandedRows, setExpandedRows] = useState([]);
+  const filtersMeta = [
+    {
+      name: 'patientName',
+      displayName: 'Patient Name',
+      inputType: 'Text',
+      isSortable: true,
+      gridCol: 4,
+    },
+    {
+      name: 'mrn',
+      displayName: 'MRN',
+      inputType: 'Text',
+      isSortable: true,
+      gridCol: 2,
+    },
+    {
+      name: 'studyDate',
+      displayName: 'Study date',
+      inputType: 'DateRange',
+      isSortable: true,
+      gridCol: 5,
+    },
+    {
+      name: 'description',
+      displayName: 'Description',
+      inputType: 'Text',
+      isSortable: true,
+      gridCol: 4,
+    },
+    {
+      name: 'modality',
+      displayName: 'Modality',
+      inputType: 'MultiSelect',
+      inputProps: {
+        options: [
+          { value: 'SEG', label: 'SEG' },
+          { value: 'CT', label: 'CT' },
+          { value: 'MR', label: 'MR' },
+          { value: 'SR', label: 'SR' },
+        ],
+      },
+      isSortable: true,
+      gridCol: 3,
+    },
+    {
+      name: 'accession',
+      displayName: 'Accession',
+      inputType: 'Text',
+      isSortable: true,
+      gridCol: 4,
+    },
+    {
+      name: 'instances',
+      displayName: 'Instances',
+      inputType: 'None',
+      isSortable: true,
+      gridCol: 2,
+    },
+  ];
+  const isFiltering = (filterValues, defaultFilterValues) => {
+    return Object.keys(defaultFilterValues).some(name => {
+      return filterValues[name] !== defaultFilterValues[name];
+    });
+  };
+  const tableDataSource = studies.map((study, key) => {
+    const rowKey = key + 1;
+    const isExpanded = expandedRows.some(k => k === rowKey);
+    const {
+      AccessionNumber,
+      Modalities,
+      Instances,
+      StudyDescription,
+      PatientId,
+      PatientName,
+      StudyDate,
+      series,
+    } = study;
+    const seriesTableColumns = {
+      description: 'Description',
+      seriesNumber: 'Series',
+      modality: 'Modality',
+      Instances: 'Instances',
+    };
+    const seriesTableDataSource = series.map(seriesItem => {
+      const { SeriesNumber, Modality, instances } = seriesItem;
+      return {
+        description: 'Patient Protocol',
+        seriesNumber: SeriesNumber,
+        modality: Modality,
+        Instances: instances.length,
+      };
+    });
+    return {
+      row: [
+        {
+          key: 'patientName',
+          content: (
+            <>
+              <Icon
+                name={isExpanded ? 'chevron-down' : 'chevron-right'}
+                className="mr-4"
+              />
+              {PatientName}
+            </>
+          ),
+          gridCol: 4,
+        },
+        {
+          key: 'mrn',
+          content: PatientId,
+          gridCol: 2,
+        },
+        {
+          key: 'studyDate',
+          content: (
+            <div>
+              <span className="mr-4">
+                {moment(StudyDate).format('MMM-DD-YYYY')}
+              </span>
+              <span>{moment(StudyDate).format('hh:mm A')}</span>
+            </div>
+          ),
+          gridCol: 5,
+        },
+        {
+          key: 'description',
+          content: StudyDescription,
+          gridCol: 4,
+        },
+        {
+          key: 'modality',
+          content: Modalities,
+          gridCol: 3,
+        },
+        {
+          key: 'accession',
+          content: AccessionNumber,
+          gridCol: 4,
+        },
+        {
+          key: 'instances',
+          content: (
+            <>
+              <Icon
+                name="group-layers"
+                className={classnames('inline-flex mr-2 w-4', {
+                  'text-primary-active': isExpanded,
+                  'text-secondary-light': !isExpanded,
+                })}
+              />
+              {Instances}
+            </>
+          ),
+          gridCol: 4,
+        },
+      ],
+      expandedContent: (
+        <StudyListExpandedRow
+          seriesTableColumns={seriesTableColumns}
+          seriesTableDataSource={seriesTableDataSource}
+        >
+          <Button
+            rounded="full"
+            variant="contained"
+            className="mr-4 font-bold"
+            endIcon={<Icon name="launch-arrow" style={{ color: '#21a7c6' }} />}
+          >
+            <Link to="/viewer/123">Basic Viewer</Link>
+          </Button>
+          <Button
+            rounded="full"
+            variant="contained"
+            className="mr-4 font-bold"
+            endIcon={<Icon name="launch-arrow" style={{ color: '#21a7c6' }} />}
+          >
+            <Link to="/viewer/123">Segmentation</Link>
+          </Button>
+          <Button
+            rounded="full"
+            variant="outlined"
+            endIcon={<Icon name="launch-info" />}
+            className="font-bold"
+          >
+            Module 3
+          </Button>
+          <div className="ml-5 text-lg text-common-bright inline-flex items-center">
+            <Icon name="notificationwarning-diamond" className="mr-2 w-5 h-5" />
+            Feedback text lorem ipsum dolor sit amet
+          </div>
+        </StudyListExpandedRow>
+      ),
+      onClickRow: () =>
+        setExpandedRows(s =>
+          isExpanded ? s.filter(n => rowKey !== n) : [...s, rowKey]
+        ),
+      isExpanded,
+    };
+  });
+
+  const [currentPage, setCurrentPage] = useState(1);
+  const [perPage, setPerPage] = useState(25);
+  const totalPages = Math.floor(numOfStudies / perPage);
+
+  const onChangePage = page => {
+    if (page > totalPages) {
+      return;
+    }
+    setCurrentPage(page);
+  };
+
+  const onChangePerPage = perPage => {
+    setPerPage(perPage);
+    setCurrentPage(1);
+  };
+
+  const hasStudies = numOfStudies > 0;
 
-const StudyListContainer = () => {
   return (
-    <div>
-      <div>StudyListContainer</div>
-      <Link to="/viewer/123">Go to Viewer Route</Link>
+    <div
+      className={classnames('bg-black h-full', {
+        'h-screen': !hasStudies,
+      })}
+    >
+      <NavBar className="justify-between border-b-4 border-black" isSticky>
+        <div className="flex items-center">
+          <div className="mx-3">
+            <Svg name="logo-ohif" />
+          </div>
+        </div>
+        <div className="flex items-center">
+          <span className="mr-3 text-common-light text-lg">
+            FOR INVESTIGATIONAL USE ONLY
+          </span>
+          <IconButton
+            variant="text"
+            color="inherit"
+            className="text-primary-active"
+            onClick={() => {}}
+          >
+            <React.Fragment>
+              <Icon name="settings" />
+              <Icon name="chevron-down" />
+            </React.Fragment>
+          </IconButton>
+        </div>
+      </NavBar>
+      <StudyListFilter
+        numOfStudies={numOfStudies}
+        filtersMeta={filtersMeta}
+        filterValues={filterValues}
+        setFilterValues={setFilterValues}
+        clearFilters={() => setFilterValues(defaultFilterValues)}
+        isFiltering={isFiltering(filterValues, defaultFilterValues)}
+      />
+      {hasStudies ? (
+        <>
+          <StudyListTable
+            tableDataSource={tableDataSource.slice(
+              (currentPage - 1) * perPage,
+              (currentPage - 1) * perPage + perPage
+            )}
+            numOfStudies={numOfStudies}
+            filtersMeta={filtersMeta}
+          />
+          <StudyListPagination
+            onChangePage={onChangePage}
+            onChangePerPage={onChangePerPage}
+            currentPage={currentPage}
+            perPage={perPage}
+          />
+        </>
+      ) : (
+        <div className="flex flex-col items-center justify-center pt-48">
+          <EmptyStudies />
+        </div>
+      )}
     </div>
   );
-};
+}
 
 export default StudyListContainer;
diff --git a/platform/viewer/src/routes/index.js b/platform/viewer/src/routes/index.js
index 6c941e640..2741337b8 100644
--- a/platform/viewer/src/routes/index.js
+++ b/platform/viewer/src/routes/index.js
@@ -1,6 +1,7 @@
 import React from 'react';
 import { Switch, Route } from 'react-router-dom';
 
+// Route Components
 import StudyListContainer from './StudyListContainer';
 import NotFound from './NotFound';