From 34961e5624a29c27f35b379478d924105b37d2ac Mon Sep 17 00:00:00 2001
From: Rodrigo Antinarelli <rodrigoantinarelli@gmail.com>
Date: Fri, 20 Mar 2020 18:56:27 -0300
Subject: [PATCH] feat: v2 Table Component

---
 platform/ui/index.js                          |  22 +---
 platform/ui/src/components/Table/Table.jsx    |  35 ++++++
 platform/ui/src/components/Table/Table.mdx    |  63 ++++++++++
 platform/ui/src/components/Table/index.js     |   2 +
 .../ui/src/components/TableBody/TableBody.jsx |  32 ++++++
 platform/ui/src/components/TableBody/index.js |   2 +
 .../ui/src/components/TableCell/TableCell.jsx |  43 +++++++
 platform/ui/src/components/TableCell/index.js |   2 +
 .../ui/src/components/TableHead/TableHead.jsx |  32 ++++++
 platform/ui/src/components/TableHead/index.js |   2 +
 .../ui/src/components/TableRow/TableRow.jsx   |  28 +++++
 platform/ui/src/components/TableRow/index.js  |   2 +
 platform/ui/src/components/index.js           |  10 ++
 .../components/Layout/index.js                |   1 +
 .../components/Sidebar/index.js               |   8 +-
 platform/ui/src/gatsby-theme-docz/theme.css   |  10 +-
 .../StudyList/components/StudyListTable.js    | 108 ++++++------------
 17 files changed, 304 insertions(+), 98 deletions(-)
 create mode 100644 platform/ui/src/components/Table/Table.jsx
 create mode 100644 platform/ui/src/components/Table/Table.mdx
 create mode 100644 platform/ui/src/components/Table/index.js
 create mode 100644 platform/ui/src/components/TableBody/TableBody.jsx
 create mode 100644 platform/ui/src/components/TableBody/index.js
 create mode 100644 platform/ui/src/components/TableCell/TableCell.jsx
 create mode 100644 platform/ui/src/components/TableCell/index.js
 create mode 100644 platform/ui/src/components/TableHead/TableHead.jsx
 create mode 100644 platform/ui/src/components/TableHead/index.js
 create mode 100644 platform/ui/src/components/TableRow/TableRow.jsx
 create mode 100644 platform/ui/src/components/TableRow/index.js

diff --git a/platform/ui/index.js b/platform/ui/index.js
index 78dc23ea6..02defd329 100644
--- a/platform/ui/index.js
+++ b/platform/ui/index.js
@@ -1,23 +1,7 @@
 import * as utils from './src/utils/';
-import {
-  ModalProvider,
-  ModalConsumer,
-  useModal,
-  withModal,
-} from './src/contextProviders';
+export * from './src/components';
+
+export { utils };
 
-export {
-  Button,
-  ButtonGroup,
-  DateRange,
-  Icon,
-  IconButton,
-  Input,
-  NavBar,
-  Select,
-  Svg,
-  ThemeWrapper,
-  Typography,
-} from './src/components';
 export { StudyList } from './src/views';
 export { ModalProvider, ModalConsumer, useModal, withModal, utils };
diff --git a/platform/ui/src/components/Table/Table.jsx b/platform/ui/src/components/Table/Table.jsx
new file mode 100644
index 000000000..86029dbf4
--- /dev/null
+++ b/platform/ui/src/components/Table/Table.jsx
@@ -0,0 +1,35 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import classnames from 'classnames';
+
+const Table = ({ children, className, fullWidth, style }) => {
+  return (
+    <div
+      className={classnames(
+        'text-lg text-white',
+        {
+          'w-full': fullWidth,
+        },
+        className
+      )}
+      style={style}
+    >
+      {children}
+    </div>
+  );
+};
+
+Table.defaultProps = {
+  className: '',
+  fullWidth: true,
+  style: {},
+};
+
+Table.propTypes = {
+  fullWidth: PropTypes.bool,
+  children: PropTypes.node.isRequired,
+  className: PropTypes.string,
+  style: PropTypes.object,
+};
+
+export default Table;
diff --git a/platform/ui/src/components/Table/Table.mdx b/platform/ui/src/components/Table/Table.mdx
new file mode 100644
index 000000000..9ee423a45
--- /dev/null
+++ b/platform/ui/src/components/Table/Table.mdx
@@ -0,0 +1,63 @@
+---
+name: Table
+menu: Components
+route: components/table
+---
+
+import { Playground, Props, Link } from 'docz';
+import { Table, TableHead, TableBody, TableRow, TableCell } from '@ohif/ui';
+
+# Table
+
+Tables display sets of data.
+
+## Import
+
+```javascript
+import { Table } from '@ohfi/ui';
+```
+
+## Basic usage
+
+<Playground>
+  <div className="p-4">
+    <Table>
+      <TableHead>
+        <TableRow>
+          <TableCell>Item 01</TableCell>
+          <TableCell>Item 02</TableCell>
+          <TableCell>Item 03</TableCell>
+          <TableCell>Item 04</TableCell>
+        </TableRow>
+      </TableHead>
+      <TableBody>
+        <TableRow>
+          <TableCell>Content 01</TableCell>
+          <TableCell>Content 02</TableCell>
+          <TableCell>Content 03</TableCell>
+          <TableCell>Content 04</TableCell>
+        </TableRow>
+      </TableBody>
+    </Table>
+  </div>
+</Playground>
+
+## Table width
+
+Table has full width by default. If you want to change its width, you will have
+to set the `fullWidth` prop to `false` and then define your own table width by
+passing a `className` or by `style`.
+
+## Table components
+
+<Link to="components/table-head">TableHead</Link>
+<br />
+<Link to="components/table-body">TableBody</Link>
+<br />
+<Link to="components/table-row">TableRow</Link>
+<br />
+<Link to="components/table-cell">TableCell</Link>
+
+## Properties
+
+<Props of={Table} />
diff --git a/platform/ui/src/components/Table/index.js b/platform/ui/src/components/Table/index.js
new file mode 100644
index 000000000..ad6fbcbd7
--- /dev/null
+++ b/platform/ui/src/components/Table/index.js
@@ -0,0 +1,2 @@
+import Table from './Table';
+export default Table;
diff --git a/platform/ui/src/components/TableBody/TableBody.jsx b/platform/ui/src/components/TableBody/TableBody.jsx
new file mode 100644
index 000000000..d3e6d6357
--- /dev/null
+++ b/platform/ui/src/components/TableBody/TableBody.jsx
@@ -0,0 +1,32 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import classnames from 'classnames';
+
+const TableBody = ({ children, className, style }) => {
+  return (
+    <div
+      className={classnames(
+        'mt-2 max-h-48 overflow-y-scroll ohif-scrollbar',
+        className
+      )}
+      style={style}
+    >
+      {React.Children.map(children, child =>
+        React.cloneElement(child, { isTableHead: false })
+      )}
+    </div>
+  );
+};
+
+TableBody.defaultProps = {
+  className: '',
+  style: {},
+};
+
+TableBody.propTypes = {
+  children: PropTypes.node.isRequired,
+  className: PropTypes.string,
+  style: PropTypes.object,
+};
+
+export default TableBody;
diff --git a/platform/ui/src/components/TableBody/index.js b/platform/ui/src/components/TableBody/index.js
new file mode 100644
index 000000000..debbe4eaf
--- /dev/null
+++ b/platform/ui/src/components/TableBody/index.js
@@ -0,0 +1,2 @@
+import TableBody from './TableBody';
+export default TableBody;
diff --git a/platform/ui/src/components/TableCell/TableCell.jsx b/platform/ui/src/components/TableCell/TableCell.jsx
new file mode 100644
index 000000000..30982abf3
--- /dev/null
+++ b/platform/ui/src/components/TableCell/TableCell.jsx
@@ -0,0 +1,43 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import classnames from 'classnames';
+
+const TableCell = ({ children, className, isTableHead, size, style }) => {
+  const sizeClasses = {
+    small: 'flex-0.3',
+    normal: 'flex-1',
+  };
+
+  return (
+    <div
+      className={classnames(
+        'px-2 last:border-r-0 break-all',
+        sizeClasses[size],
+        {
+          'border-r border-custom-violetPale': !isTableHead,
+        },
+        className
+      )}
+      style={style}
+    >
+      {children}
+    </div>
+  );
+};
+
+TableCell.defaultProps = {
+  className: '',
+  isTableHead: false,
+  size: 'normal',
+  style: {},
+};
+
+TableCell.propTypes = {
+  isTableHead: PropTypes.bool,
+  children: PropTypes.node.isRequired,
+  className: PropTypes.string,
+  size: PropTypes.oneOf(['small', 'normal']),
+  style: PropTypes.object,
+};
+
+export default TableCell;
diff --git a/platform/ui/src/components/TableCell/index.js b/platform/ui/src/components/TableCell/index.js
new file mode 100644
index 000000000..a6381757e
--- /dev/null
+++ b/platform/ui/src/components/TableCell/index.js
@@ -0,0 +1,2 @@
+import TableCell from './TableCell';
+export default TableCell;
diff --git a/platform/ui/src/components/TableHead/TableHead.jsx b/platform/ui/src/components/TableHead/TableHead.jsx
new file mode 100644
index 000000000..42fcb8e43
--- /dev/null
+++ b/platform/ui/src/components/TableHead/TableHead.jsx
@@ -0,0 +1,32 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import classnames from 'classnames';
+
+const TableHead = ({ children, className, style }) => {
+  return (
+    <div
+      className={classnames(
+        'bg-custom-navy border-b border-custom-violetPale flex font-bold',
+        className
+      )}
+      style={style}
+    >
+      {React.cloneElement(children, {
+        isTableHead: true,
+      })}
+    </div>
+  );
+};
+
+TableHead.defaultProps = {
+  className: '',
+  style: {},
+};
+
+TableHead.propTypes = {
+  children: PropTypes.node.isRequired,
+  className: PropTypes.string,
+  style: PropTypes.object,
+};
+
+export default TableHead;
diff --git a/platform/ui/src/components/TableHead/index.js b/platform/ui/src/components/TableHead/index.js
new file mode 100644
index 000000000..4befa4c86
--- /dev/null
+++ b/platform/ui/src/components/TableHead/index.js
@@ -0,0 +1,2 @@
+import TableHead from './TableHead';
+export default TableHead;
diff --git a/platform/ui/src/components/TableRow/TableRow.jsx b/platform/ui/src/components/TableRow/TableRow.jsx
new file mode 100644
index 000000000..7a9052e2b
--- /dev/null
+++ b/platform/ui/src/components/TableRow/TableRow.jsx
@@ -0,0 +1,28 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import classnames from 'classnames';
+
+const TableRow = ({ children, className, isTableHead, style }) => {
+  return (
+    <div className={classnames('w-full flex', className)} style={style}>
+      {React.Children.map(children, child =>
+        React.cloneElement(child, { isTableHead })
+      )}
+    </div>
+  );
+};
+
+TableRow.defaultProps = {
+  isTableHead: false,
+  className: '',
+  style: {},
+};
+
+TableRow.propTypes = {
+  isTableHead: PropTypes.bool,
+  children: PropTypes.node.isRequired,
+  className: PropTypes.string,
+  style: PropTypes.object,
+};
+
+export default TableRow;
diff --git a/platform/ui/src/components/TableRow/index.js b/platform/ui/src/components/TableRow/index.js
new file mode 100644
index 000000000..560f3b653
--- /dev/null
+++ b/platform/ui/src/components/TableRow/index.js
@@ -0,0 +1,2 @@
+import TableRow from './TableRow';
+export default TableRow;
diff --git a/platform/ui/src/components/index.js b/platform/ui/src/components/index.js
index 49e011fd7..add5a5313 100644
--- a/platform/ui/src/components/index.js
+++ b/platform/ui/src/components/index.js
@@ -7,6 +7,11 @@ import Input from './Input';
 import NavBar from './NavBar';
 import Select from './Select';
 import Svg from './Svg';
+import Table from './Table';
+import TableBody from './TableBody';
+import TableCell from './TableCell';
+import TableHead from './TableHead';
+import TableRow from './TableRow';
 import ThemeWrapper from './ThemeWrapper/';
 import Typography from './Typography';
 
@@ -20,6 +25,11 @@ export {
   NavBar,
   Select,
   Svg,
+  Table,
+  TableBody,
+  TableCell,
+  TableHead,
+  TableRow,
   ThemeWrapper,
   Typography,
 };
diff --git a/platform/ui/src/gatsby-theme-docz/components/Layout/index.js b/platform/ui/src/gatsby-theme-docz/components/Layout/index.js
index c22eb568a..8495bad79 100644
--- a/platform/ui/src/gatsby-theme-docz/components/Layout/index.js
+++ b/platform/ui/src/gatsby-theme-docz/components/Layout/index.js
@@ -36,6 +36,7 @@ export const Layout = ({ children }) => {
             onClick={() => setSidebarOpen(false)}
           />
           <div
+            id="main-container"
             className={classnames('py-10 w-full', {
               'px-4 lg:px-20': sidebarOpen,
               'px-4 lg:px-8': !sidebarOpen,
diff --git a/platform/ui/src/gatsby-theme-docz/components/Sidebar/index.js b/platform/ui/src/gatsby-theme-docz/components/Sidebar/index.js
index b96620bf4..7b057625f 100644
--- a/platform/ui/src/gatsby-theme-docz/components/Sidebar/index.js
+++ b/platform/ui/src/gatsby-theme-docz/components/Sidebar/index.js
@@ -31,19 +31,19 @@ export const Sidebar = React.forwardRef((props, ref) => {
           <div className="block lg:hidden">
             <div
               onClick={() => props.onClick()}
-              className="fixed z-10 left-0 w-full h-full bg-black opacity-80"
-              style={{ top: '81px' }}
+              className="fixed left-0 w-full h-full bg-black opacity-80"
+              style={{ top: 81, zIndex: 99999 }}
             />
           </div>
           <div
             ref={ref}
             data-testid="sidebar"
             className={classnames(
-              'p-8 flex-col bg-white top-0 overflow-auto z-10 border-r border-gray-400 hidden fixed left-0 bottom-0 lg:block lg:sticky lg:h-screen lg:top-0'
+              'p-8 flex-col bg-white top-0 overflow-auto border-r border-gray-400 hidden fixed left-0 bottom-0 lg:block lg:sticky lg:h-screen lg:top-0'
             )}
             style={{
               display: props.sidebarOpen ? 'block' : 'none',
-              top: '81px',
+              zIndex: 99999,
               minWidth: 250,
             }}
           >
diff --git a/platform/ui/src/gatsby-theme-docz/theme.css b/platform/ui/src/gatsby-theme-docz/theme.css
index 853f65234..9a0e034fa 100644
--- a/platform/ui/src/gatsby-theme-docz/theme.css
+++ b/platform/ui/src/gatsby-theme-docz/theme.css
@@ -6,7 +6,11 @@
   background: #000;
 }
 
-[data-testid='main-container'] > h2 {
-  margin: 20px 0;
-  font-weight: bold;
+#main-container > h2 {
+  @apply font-bold my-8 text-2xl;
+}
+
+#main-container > ul li a,
+#main-container > a {
+  @apply underline text-blue-500;
 }
diff --git a/platform/ui/src/views/StudyList/components/StudyListTable.js b/platform/ui/src/views/StudyList/components/StudyListTable.js
index e33f91b2f..1c2bf6973 100644
--- a/platform/ui/src/views/StudyList/components/StudyListTable.js
+++ b/platform/ui/src/views/StudyList/components/StudyListTable.js
@@ -3,14 +3,23 @@ import PropTypes from 'prop-types';
 import classnames from 'classnames';
 
 import { format } from 'date-fns';
-import { Button, Icon, Typography } from '@ohif/ui';
+import {
+  Button,
+  Icon,
+  Typography,
+  Table,
+  TableHead,
+  TableBody,
+  TableRow,
+  TableCell,
+} from '@ohif/ui';
 
 const getGridColClass = (filtersMeta, name) => {
   const filter = filtersMeta.find(filter => filter.name === name);
   return (filter && filter.gridCol && `w-${filter.gridCol}/24`) || '';
 };
 
-const TableRow = props => {
+const StudyTableRow = props => {
   const {
     AccessionNumber,
     Modalities,
@@ -30,11 +39,6 @@ const TableRow = props => {
     'px-4 py-2 text-base',
     { 'border-b border-custom-violetPale': !isOpened },
   ];
-  const seriesWidthClasses = {
-    normal: 'px-2 flex-1',
-    small: 'px-2 flex-0.3',
-  };
-  const seriesBodyClasses = 'border-r border-custom-violetPale';
   return (
     <>
       <tr>
@@ -176,75 +180,35 @@ const TableRow = props => {
                         </div>
                       </div>
                       <div className="mt-4">
-                        <div className="w-full text-lg">
-                          <div className="bg-custom-navy border-b border-custom-violetPale flex">
-                            <div
-                              className={classnames(
-                                seriesWidthClasses.normal,
-                                'font-bold'
-                              )}
-                            >
-                              Description
-                            </div>
-                            <div
-                              className={classnames(
-                                seriesWidthClasses.small,
-                                'font-bold'
-                              )}
-                            >
-                              Series
-                            </div>
-                            <div
-                              className={classnames(
-                                seriesWidthClasses.small,
-                                'font-bold'
-                              )}
-                            >
-                              Modality
-                            </div>
-                            <div
-                              className={classnames(
-                                seriesWidthClasses.normal,
-                                'font-bold'
-                              )}
-                            >
-                              Instances
-                            </div>
-                          </div>
-                          <div className="mt-2 max-h-48 overflow-y-scroll ohif-scrollbar">
+                        <Table>
+                          <TableHead>
+                            <TableRow>
+                              <TableCell size="normal">Description</TableCell>
+                              <TableCell size="small">Series</TableCell>
+                              <TableCell size="small">Modality</TableCell>
+                              <TableCell size="normal">Instances</TableCell>
+                            </TableRow>
+                          </TableHead>
+
+                          <TableBody>
                             {series.map((seriesItem, i) => (
-                              <div className="w-full flex" key={i}>
-                                <div
-                                  className={classnames(
-                                    seriesWidthClasses.normal,
-                                    seriesBodyClasses
-                                  )}
-                                >
+                              <TableRow key={i}>
+                                <TableCell size="normal">
                                   Patient Protocol
-                                </div>
-                                <div
-                                  className={classnames(
-                                    seriesWidthClasses.small,
-                                    seriesBodyClasses
-                                  )}
-                                >
+                                </TableCell>
+                                <TableCell size="small">
                                   {seriesItem.SeriesNumber}
-                                </div>
-                                <div
-                                  className={classnames(
-                                    seriesWidthClasses.small,
-                                    seriesBodyClasses
-                                  )}
-                                >
+                                </TableCell>
+                                <TableCell size="small">
                                   {seriesItem.Modality}
-                                </div>
-                                <div className={classnames('pl-3 flex-1')}>
+                                </TableCell>
+                                <TableCell size="normal">
                                   {seriesItem.instances.length}
-                                </div>
-                              </div>
+                                </TableCell>
+                              </TableRow>
                             ))}
-                          </div>
-                        </div>
+                          </TableBody>
+                        </Table>
                       </div>
                     </td>
                   </tr>
@@ -258,7 +222,7 @@ const TableRow = props => {
   );
 };
 
-TableRow.propTypes = {
+StudyTableRow.propTypes = {
   AccessionNumber: PropTypes.string.isRequired,
   Modalities: PropTypes.string.isRequired,
   Instances: PropTypes.number.isRequired,
@@ -275,7 +239,7 @@ const StudyListTable = ({ studies, numOfStudies, filtersMeta }) => {
       <table className="w-full text-white">
         <tbody>
           {studies.map((study, i) => (
-            <TableRow
+            <StudyTableRow
               key={i}
               AccessionNumber={study.AccessionNumber || ''}
               Modalities={study.Modalities || ''}