[ARVADOS-WORKBENCH2] updated: 1.2.0-156-gfb15af5

Git user git at public.curoverse.com
Tue Aug 28 06:41:41 EDT 2018


Summary of changes:
 src/views/process-panel/process-panel.tsx | 27 ++++++++++++++-------------
 src/views/workbench/workbench.tsx         |  3 ---
 2 files changed, 14 insertions(+), 16 deletions(-)

       via  fb15af5c500ad8469240c59b17ff73b889bb022b (commit)
      from  4476731a9ec985c5abd97da453978af0dfa4406b (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit fb15af5c500ad8469240c59b17ff73b889bb022b
Author: Pawel Kowalczyk <pawel.kowalczyk at contractors.roche.com>
Date:   Tue Aug 28 12:41:27 2018 +0200

    clean code
    
    Feature #13858
    
    Arvados-DCO-1.1-Signed-off-by: Pawel Kowalczyk <pawel.kowalczyk at contractors.roche.com>

diff --git a/src/views/process-panel/process-panel.tsx b/src/views/process-panel/process-panel.tsx
index 1ae8747..08bd370 100644
--- a/src/views/process-panel/process-panel.tsx
+++ b/src/views/process-panel/process-panel.tsx
@@ -15,7 +15,7 @@ import { MoreOptionsIcon, ProcessIcon } from '~/components/icon/icon';
 import { DetailsAttribute } from '~/components/details-attribute/details-attribute';
 import { RootState } from '~/store/store';
 
-type CssRules = 'card' | 'iconHeader' | 'label' | 'value' | 'content' | 'chip' | 'headerText';
+type CssRules = 'card' | 'iconHeader' | 'label' | 'value' | 'content' | 'chip' | 'headerText' | 'link';
 
 const styles: StyleRulesCallback<CssRules> = (theme: ArvadosTheme) => ({
     card: {
@@ -27,7 +27,7 @@ const styles: StyleRulesCallback<CssRules> = (theme: ArvadosTheme) => ({
         color: theme.customs.colors.green700
     },
     label: {
-        fontSize: '0.875rem'
+        fontSize: '0.875rem',
     },
     value: {
         textTransform: 'none',
@@ -41,6 +41,13 @@ const styles: StyleRulesCallback<CssRules> = (theme: ArvadosTheme) => ({
             paddingBottom: '0px ',
         }
     },
+    link: {
+        fontSize: '0.875rem',
+        '&:hover': {
+            color: theme.palette.primary.main,
+            cursor: 'pointer'
+        }
+    },
     chip: {
         height: theme.spacing.unit * 2.5,
         width: theme.spacing.unit * 12,
@@ -53,9 +60,9 @@ const styles: StyleRulesCallback<CssRules> = (theme: ArvadosTheme) => ({
         fontSize: '0.875rem',
         display: 'flex',
         position: 'relative',
-        justifyContent: 'flex-end',
+        justifyContent: 'flex-start',
         top: -theme.spacing.unit * 4.5,
-        right: theme.spacing.unit * 2,
+        left: theme.spacing.unit * 3,
     }
 });
 
@@ -64,7 +71,6 @@ interface ProcessPanelDataProps {
 }
 
 interface ProcessPanelActionProps {
-    onItemRouteChange: (processId: string) => void;
     onContextMenu: (event: React.MouseEvent<HTMLElement>, item: ProcessResource) => void;
 }
 
@@ -105,11 +111,11 @@ export const ProcessPanel = withStyles(styles)(
                             </Grid>
                             <Grid container direction="column">
                                 <Grid item xs={8}>
-                                    <DetailsAttribute classLabel={classes.label} classValue={classes.value}
+                                    <DetailsAttribute classLabel={classes.link} classValue={classes.value}
                                         label='Container output' />
-                                    <DetailsAttribute classLabel={classes.label} classValue={classes.value}
+                                    <DetailsAttribute classLabel={classes.link} classValue={classes.value}
                                         label='Show inputs' />
-                                    <DetailsAttribute classLabel={classes.label} classValue={classes.value}
+                                    <DetailsAttribute classLabel={classes.link} classValue={classes.value}
                                         label='Show command' />
                                 </Grid>
                             </Grid>
@@ -118,11 +124,6 @@ export const ProcessPanel = withStyles(styles)(
                     </Card>
                 </div>;
             }
-            componentWillReceiveProps({ match, item, onItemRouteChange }: ProcessPanelProps) {
-                if (!item || match.params.id !== item.uuid) {
-                    onItemRouteChange(match.params.id);
-                }
-            }
         }
     )
 );
\ No newline at end of file
diff --git a/src/views/workbench/workbench.tsx b/src/views/workbench/workbench.tsx
index 856a626..d8daed5 100644
--- a/src/views/workbench/workbench.tsx
+++ b/src/views/workbench/workbench.tsx
@@ -267,9 +267,6 @@ export const Workbench = withStyles(styles)(
             }
 
             renderProcessPanel = (props: RouteComponentProps<{ id: string }>) => <ProcessPanel
-                onItemRouteChange={(processId) => {
-                    return <span>a</span>;
-                }}
                 onContextMenu={(event, item) => {
                     this.openContextMenu(event, {
                         uuid: 'item.uuid',

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list