Add headers and menu button to collection-panel-files
authorMichal Klobukowski <michal.klobukowski@contractors.roche.com>
Mon, 30 Jul 2018 10:50:16 +0000 (12:50 +0200)
committerMichal Klobukowski <michal.klobukowski@contractors.roche.com>
Mon, 30 Jul 2018 10:50:16 +0000 (12:50 +0200)
Feature #13855

Arvados-DCO-1.1-Signed-off-by: Michal Klobukowski <michal.klobukowski@contractors.roche.com>

src/components/collection-panel-files/collection-panel-files.tsx
src/components/file-tree/file-tree-item.tsx
src/components/file-tree/file-tree.tsx
src/views/workbench/workbench.tsx

index df277e33fa505f6b323d17a6de744c8e1b7ad52a..af2c19a05d0388f17abf390ebe1038649673830c 100644 (file)
@@ -6,20 +6,46 @@ import * as React from 'react';
 import { TreeItem, TreeItemStatus } from '../tree/tree';
 import { FileTreeData } from '../file-tree/file-tree-data';
 import { FileTree } from '../file-tree/file-tree';
-import { CollectionPanelFilesState } from '../../store/collection-panel/collection-panel-files/collection-panel-files-state';
+import { IconButton, Grid, Typography, StyleRulesCallback, withStyles, WithStyles } from '@material-ui/core';
+import { CustomizeTableIcon } from '../icon/icon';
 
 export interface CollectionPanelFilesProps {
     items: Array<TreeItem<FileTreeData>>;
-    onItemContextMenu: (event: React.MouseEvent<HTMLElement>, item: TreeItem<FileTreeData>) => void;
-    onCommonContextMenu: (event: React.MouseEvent<HTMLElement>) => void;
+    onItemMenuOpen: (event: React.MouseEvent<HTMLElement>, item: TreeItem<FileTreeData>) => void;
+    onOptionsMenuOpen: (event: React.MouseEvent<HTMLElement>) => void;
     onSelectionToggle: (event: React.MouseEvent<HTMLElement>, item: TreeItem<FileTreeData>) => void;
     onCollapseToggle: (id: string, status: TreeItemStatus) => void;
 }
 
-export const CollectionPanelFiles = ({ onItemContextMenu, onCommonContextMenu, ...treeProps }: CollectionPanelFilesProps) =>
-    <div>
-        <FileTree onContextMenu={onItemContextMenu} {...treeProps} />
-    </div>;
+type CssRules = 'nameHeader' | 'fileSizeHeader';
+
+const styles: StyleRulesCallback<CssRules> = theme => ({
+    nameHeader: {
+        marginLeft: '75px'
+    },
+    fileSizeHeader: {
+        marginRight: '50px'
+    }
+});
+
+export const CollectionPanelFiles = withStyles(styles)(
+    ({ onItemMenuOpen, onOptionsMenuOpen, classes, ...treeProps }: CollectionPanelFilesProps & WithStyles<CssRules>) =>
+        <div>
+            <Grid container justify="flex-end">
+                <IconButton onClick={onOptionsMenuOpen}>
+                    <CustomizeTableIcon />
+                </IconButton>
+            </Grid>
+            <Grid container justify="space-between">
+                <Typography variant="caption" className={classes.nameHeader}>
+                    Name
+                </Typography>
+                <Typography variant="caption" className={classes.fileSizeHeader}>
+                    File size
+            </Typography>
+            </Grid>
+            <FileTree onMenuOpen={onItemMenuOpen} {...treeProps} />
+        </div>);
 
 export const collectionPanelItems: Array<TreeItem<FileTreeData>> = [{
     active: false,
@@ -44,7 +70,8 @@ export const collectionPanelItems: Array<TreeItem<FileTreeData>> = [{
         active: false,
         data: {
             name: "File 1.1",
-            type: "file"
+            type: "file",
+            size: 20033
         },
         id: "File 1.1",
         open: false,
index e65c6cdfa0cb2d5d8082b716c667dab071b72924..71e37f1f570cc0afc4341ea66af5efd801179bab 100644 (file)
@@ -16,7 +16,7 @@ const fileTreeItemStyle: StyleRulesCallback<CssRules> = theme => ({
     root: {
         display: "flex",
         alignItems: "center",
-        paddingRight: `${theme.spacing.unit}px`
+        paddingRight: `${theme.spacing.unit * 1.5}px`
     },
     spacer: {
         flex: "1"
index bb70f25c7442e8da0209197f4a9281c2e52dea1c..06fc8b7855d26ad66fafd3e173fb1b05af458f37 100644 (file)
@@ -9,7 +9,7 @@ import { FileTreeItem } from "./file-tree-item";
 
 export interface FileTreeProps {
     items: Array<TreeItem<FileTreeData>>;
-    onContextMenu: (event: React.MouseEvent<HTMLElement>, item: TreeItem<FileTreeData>) => void;
+    onMenuOpen: (event: React.MouseEvent<HTMLElement>, item: TreeItem<FileTreeData>) => void;
     onSelectionToggle: (event: React.MouseEvent<HTMLElement>, item: TreeItem<FileTreeData>) => void;
     onCollapseToggle: (id: string, status: TreeItemStatus) => void;
 }
@@ -29,7 +29,7 @@ export class FileTree extends React.Component<FileTreeProps> {
 
     handleContextMenu = (event: React.MouseEvent<any>, item: TreeItem<FileTreeData>) => {
         event.stopPropagation();
-        this.props.onContextMenu(event, item);
+        this.props.onMenuOpen(event, item);
     }
 
     handleToggle = (id: string, status: TreeItemStatus) => {
index e932fba819d3a572af9ade4f43d7c273a9db2122..1b6e05dabbe032113d62aaa3892766e0961d88ad 100644 (file)
@@ -218,8 +218,8 @@ export const Workbench = withStyles(styles)(
                                         items={collectionPanelItems} 
                                         onCollapseToggle={console.log}
                                         onSelectionToggle={console.log}
-                                        onItemContextMenu={console.log}
-                                        onCommonContextMenu={console.log}
+                                        onItemMenuOpen={console.log}
+                                        onOptionsMenuOpen={console.log}
                                         />} />
                                 </Switch>
                             </div>