Merge branch '21688-io-panel-style-fixes' into main. Closes #21688
[arvados.git] / services / workbench2 / src / views / process-panel / process-io-card.tsx
index 607bdeb763a95ae84d0d82a4b00209ef8e906e6c..9fce7e83d4bd516cb5f9e1247fd320a7b0b09404 100644 (file)
@@ -2,8 +2,8 @@
 //
 // SPDX-License-Identifier: AGPL-3.0
 
-import React, { ReactElement, memo, useState } from 'react';
-import { Dispatch } from 'redux';
+import React, { ReactElement, memo, useState } from "react";
+import { Dispatch } from "redux";
 import {
     StyleRulesCallback,
     WithStyles,
@@ -25,19 +25,10 @@ import {
     Grid,
     Chip,
     CircularProgress,
-} from '@material-ui/core';
-import { ArvadosTheme } from 'common/custom-theme';
-import {
-    CloseIcon,
-    ImageIcon,
-    InputIcon,
-    ImageOffIcon,
-    OutputIcon,
-    MaximizeIcon,
-    UnMaximizeIcon,
-    InfoIcon
-} from 'components/icon/icon';
-import { MPVPanelProps } from 'components/multi-panel-view/multi-panel-view';
+} from "@material-ui/core";
+import { ArvadosTheme } from "common/custom-theme";
+import { CloseIcon, InputIcon, OutputIcon, MaximizeIcon, UnMaximizeIcon, InfoIcon } from "components/icon/icon";
+import { MPVPanelProps } from "components/multi-panel-view/multi-panel-view";
 import {
     BooleanCommandInputParameter,
     CommandInputParameter,
@@ -56,26 +47,29 @@ import {
     isPrimitiveOfType,
     StringArrayCommandInputParameter,
     StringCommandInputParameter,
-    getEnumType
+    getEnumType,
 } from "models/workflow";
-import { CommandOutputParameter } from 'cwlts/mappings/v1.0/CommandOutputParameter';
-import { File } from 'models/workflow';
-import { getInlineFileUrl } from 'views-components/context-menu/actions/helpers';
-import { AuthState } from 'store/auth/auth-reducer';
-import mime from 'mime';
-import { DefaultView } from 'components/default-view/default-view';
-import { getNavUrl } from 'routes/routes';
-import { Link as RouterLink } from 'react-router-dom';
-import { Link as MuiLink } from '@material-ui/core';
-import { InputCollectionMount } from 'store/processes/processes-actions';
-import { connect } from 'react-redux';
-import { RootState } from 'store/store';
-import { ProcessOutputCollectionFiles } from './process-output-collection-files';
-import { Process } from 'store/processes/process';
-import { navigateTo } from 'store/navigation/navigation-action';
-import classNames from 'classnames';
-import { DefaultCodeSnippet } from 'components/default-code-snippet/default-code-snippet';
-import { KEEP_URL_REGEX } from 'models/resource';
+import { CommandOutputParameter } from "cwlts/mappings/v1.0/CommandOutputParameter";
+import { File } from "models/workflow";
+import { getInlineFileUrl } from "views-components/context-menu/actions/helpers";
+import { AuthState } from "store/auth/auth-reducer";
+import mime from "mime";
+import { DefaultView } from "components/default-view/default-view";
+import { getNavUrl } from "routes/routes";
+import { Link as RouterLink } from "react-router-dom";
+import { Link as MuiLink } from "@material-ui/core";
+import { InputCollectionMount } from "store/processes/processes-actions";
+import { connect } from "react-redux";
+import { RootState } from "store/store";
+import { ProcessOutputCollectionFiles } from "./process-output-collection-files";
+import { Process } from "store/processes/process";
+import { navigateTo } from "store/navigation/navigation-action";
+import classNames from "classnames";
+import { DefaultVirtualCodeSnippet } from "components/default-code-snippet/default-virtual-code-snippet";
+import { KEEP_URL_REGEX } from "models/resource";
+import { FixedSizeList } from 'react-window';
+import AutoSizer from "react-virtualized-auto-sizer";
+import { LinkProps } from "@material-ui/core/Link";
 
 type CssRules =
     | "card"
@@ -85,141 +79,184 @@ type CssRules =
     | "avatar"
     | "iconHeader"
     | "tableWrapper"
-    | "tableRoot"
-    | "paramValue"
+    | "paramTableRoot"
+    | "paramTableCellText"
+    | "mountsTableRoot"
+    | "jsonWrapper"
     | "keepLink"
     | "collectionLink"
-    | "imagePreview"
-    | "valArray"
     | "secondaryVal"
-    | "secondaryRow"
     | "emptyValue"
     | "noBorderRow"
     | "symmetricTabs"
-    | "imagePlaceholder"
-    | "rowWithPreview"
-    | "labelColumn";
+    | "wrapTooltip";
 
 const styles: StyleRulesCallback<CssRules> = (theme: ArvadosTheme) => ({
     card: {
-        height: '100%'
+        height: "100%",
     },
     header: {
         paddingTop: theme.spacing.unit,
         paddingBottom: 0,
     },
     iconHeader: {
-        fontSize: '1.875rem',
+        fontSize: "1.875rem",
         color: theme.customs.colors.greyL,
     },
     avatar: {
-        alignSelf: 'flex-start',
-        paddingTop: theme.spacing.unit * 0.5
+        alignSelf: "flex-start",
+        paddingTop: theme.spacing.unit * 0.5,
     },
+    // Card content
     content: {
-        height: `calc(100% - ${theme.spacing.unit * 7}px - ${theme.spacing.unit * 1.5}px)`,
+        height: `calc(100% - ${theme.spacing.unit * 6}px)`,
         padding: theme.spacing.unit * 1.0,
         paddingTop: 0,
-        '&:last-child': {
+        "&:last-child": {
             paddingBottom: theme.spacing.unit * 1,
-        }
+        },
     },
+    // Card title
     title: {
-        overflow: 'hidden',
+        overflow: "hidden",
         paddingTop: theme.spacing.unit * 0.5,
         color: theme.customs.colors.greyD,
-        fontSize: '1.875rem'
+        fontSize: "1.875rem",
     },
+    // Applies to table tab and collection table content
     tableWrapper: {
-        height: 'auto',
-        maxHeight: `calc(100% - ${theme.spacing.unit * 4.5}px)`,
-        overflow: 'auto',
+        height: "auto",
+        maxHeight: `calc(100% - ${theme.spacing.unit * 6}px)`,
+        overflow: "auto",
+        // Use flexbox to keep scrolling at the virtual list level
+        display: "flex",
+        flexDirection: "column",
+        alignItems: "stretch", // Stretches output collection to full width
+
+    },
+
+    // Param table virtual list styles
+    paramTableRoot: {
+        display: "flex",
+        flexDirection: "column",
+        overflow: "hidden",
+        // Flex header
+        "& thead tr": {
+            alignItems: "end",
+            "& th": {
+                padding: "4px 25px 10px",
+            },
+        },
+        "& tbody": {
+            height: "100vh", // Must be constrained by panel maxHeight
+        },
+        // Flex header/body rows
+        "& thead tr, & > tbody tr": {
+            display: "flex",
+            // Flex header/body cells
+            "& th, & td": {
+                flexGrow: 1,
+                flexShrink: 1,
+                flexBasis: 0,
+                overflow: "hidden",
+            },
+            // Column width overrides
+            "& th:nth-of-type(1), & td:nth-of-type(1)": {
+                flexGrow: 0.7,
+            },
+            "& th:nth-last-of-type(1), & td:nth-last-of-type(1)": {
+                flexGrow: 2,
+            },
+        },
+        // Flex body rows
+        "& tbody tr": {
+            height: "40px",
+            // Flex body cells
+            "& td": {
+                padding: "2px 25px 2px",
+                overflow: "hidden",
+                display: "flex",
+                flexDirection: "row",
+                alignItems: "center",
+                whiteSpace: "nowrap",
+            },
+        },
     },
-    tableRoot: {
-        width: '100%',
-        '& thead th': {
-            verticalAlign: 'bottom',
-            paddingBottom: '10px',
+    // Param value cell typography styles
+    paramTableCellText: {
+        overflow: "hidden",
+        display: "flex",
+        // Every cell contents requires a wrapper for the ellipsis
+        // since adding ellipses to an anchor element parent results in misaligned tooltip
+        "& a, & span": {
+            overflow: "hidden",
+            textOverflow: "ellipsis",
+        },
+        '& pre': {
+            margin: 0,
+            overflow: "hidden",
+            textOverflow: "ellipsis",
+        },
+    },
+    mountsTableRoot: {
+        width: "100%",
+        "& thead th": {
+            verticalAlign: "bottom",
+            paddingBottom: "10px",
+        },
+        "& td, & th": {
+            paddingRight: "25px",
         },
-        '& td, & th': {
-            paddingRight: '25px',
-        }
     },
-    paramValue: {
-        display: 'flex',
-        alignItems: 'flex-start',
-        flexDirection: 'column',
+    // JSON tab wrapper
+    jsonWrapper: {
+        height: `calc(100% - ${theme.spacing.unit * 6}px)`,
     },
     keepLink: {
         color: theme.palette.primary.main,
-        textDecoration: 'none',
-        overflowWrap: 'break-word',
-        cursor: 'pointer',
+        textDecoration: "none",
+        // Overflow wrap for mounts table
+        overflowWrap: "break-word",
+        cursor: "pointer",
     },
+    // Output collection tab link
     collectionLink: {
-        margin: '10px',
-        '& a': {
+        margin: "10px",
+        "& a": {
             color: theme.palette.primary.main,
-            textDecoration: 'none',
-            overflowWrap: 'break-word',
-            cursor: 'pointer',
-        }
-    },
-    imagePreview: {
-        maxHeight: '15em',
-        maxWidth: '15em',
-        marginBottom: theme.spacing.unit,
-    },
-    valArray: {
-        display: 'flex',
-        gap: '10px',
-        flexWrap: 'wrap',
-        '& span': {
-            display: 'inline',
-        }
+            textDecoration: "none",
+            overflowWrap: "break-word",
+            cursor: "pointer",
+        },
     },
     secondaryVal: {
-        paddingLeft: '20px',
-    },
-    secondaryRow: {
-        height: '29px',
-        verticalAlign: 'top',
-        position: 'relative',
-        top: '-9px',
+        paddingLeft: "20px",
     },
     emptyValue: {
         color: theme.customs.colors.grey700,
     },
     noBorderRow: {
-        '& td': {
-            borderBottom: 'none',
-        }
+        "& td": {
+            borderBottom: "none",
+            paddingTop: "2px",
+            paddingBottom: "2px",
+        },
+        height: "24px",
     },
     symmetricTabs: {
-        '& button': {
-            flexBasis: '0',
-        }
-    },
-    imagePlaceholder: {
-        width: '60px',
-        height: '60px',
-        display: 'flex',
-        alignItems: 'center',
-        justifyContent: 'center',
-        backgroundColor: '#cecece',
-        borderRadius: '10px',
-    },
-    rowWithPreview: {
-        verticalAlign: 'bottom',
+        "& button": {
+            flexBasis: "0",
+        },
     },
-    labelColumn: {
-        minWidth: '120px',
+    wrapTooltip: {
+        maxWidth: "600px",
+        wordWrap: "break-word",
     },
 });
 
 export enum ProcessIOCardType {
-    INPUT = 'Inputs',
-    OUTPUT = 'Outputs',
+    INPUT = "Input Parameters",
+    OUTPUT = "Output Parameters",
 }
 export interface ProcessIOCardDataProps {
     process?: Process;
@@ -228,7 +265,7 @@ export interface ProcessIOCardDataProps {
     raw: any;
     mounts?: InputCollectionMount[];
     outputUuid?: string;
-    showParams?: boolean;
+    forceShowParams?: boolean;
 }
 
 export interface ProcessIOCardActionProps {
@@ -236,240 +273,400 @@ export interface ProcessIOCardActionProps {
 }
 
 const mapDispatchToProps = (dispatch: Dispatch): ProcessIOCardActionProps => ({
-    navigateTo: (uuid) => dispatch<any>(navigateTo(uuid)),
+    navigateTo: uuid => dispatch<any>(navigateTo(uuid)),
 });
 
 type ProcessIOCardProps = ProcessIOCardDataProps & ProcessIOCardActionProps & WithStyles<CssRules> & MPVPanelProps;
 
-export const ProcessIOCard = withStyles(styles)(connect(null, mapDispatchToProps)(
-    ({ classes, label, params, raw, mounts, outputUuid, doHidePanel, doMaximizePanel,
-        doUnMaximizePanel, panelMaximized, panelName, process, navigateTo, showParams }: ProcessIOCardProps) => {
-        const [mainProcTabState, setMainProcTabState] = useState(0);
-        const [subProcTabState, setSubProcTabState] = useState(0);
-        const handleMainProcTabChange = (event: React.MouseEvent<HTMLElement>, value: number) => {
-            setMainProcTabState(value);
-        }
-        const handleSubProcTabChange = (event: React.MouseEvent<HTMLElement>, value: number) => {
-            setSubProcTabState(value);
-        }
-
-        const [showImagePreview, setShowImagePreview] = useState(false);
-
-        const PanelIcon = label === ProcessIOCardType.INPUT ? InputIcon : OutputIcon;
-        const mainProcess = !(process && process!.containerRequest.requestingContainerUuid);
-
-        const loading = raw === null || raw === undefined || params === null;
-        const hasRaw = !!(raw && Object.keys(raw).length > 0);
-        const hasParams = !!(params && params.length > 0);
-
-        // Subprocess
-        const hasInputMounts = !!(label === ProcessIOCardType.INPUT && mounts && mounts.length);
-        const hasOutputCollecton = !!(label === ProcessIOCardType.OUTPUT && outputUuid);
-
-        return <Card className={classes.card} data-cy="process-io-card">
-            <CardHeader
-                className={classes.header}
-                classes={{
-                    content: classes.title,
-                    avatar: classes.avatar,
-                }}
-                avatar={<PanelIcon className={classes.iconHeader} />}
-                title={
-                    <Typography noWrap variant='h6' color='inherit'>
-                        {label}
-                    </Typography>
-                }
-                action={
-                    <div>
-                        {mainProcess && <Tooltip title={"Toggle Image Preview"} disableFocusListener>
-                            <IconButton data-cy="io-preview-image-toggle" onClick={() => { setShowImagePreview(!showImagePreview) }}>{showImagePreview ? <ImageIcon /> : <ImageOffIcon />}</IconButton>
-                        </Tooltip>}
-                        {doUnMaximizePanel && panelMaximized &&
-                            <Tooltip title={`Unmaximize ${panelName || 'panel'}`} disableFocusListener>
-                                <IconButton onClick={doUnMaximizePanel}><UnMaximizeIcon /></IconButton>
-                            </Tooltip>}
-                        {doMaximizePanel && !panelMaximized &&
-                            <Tooltip title={`Maximize ${panelName || 'panel'}`} disableFocusListener>
-                                <IconButton onClick={doMaximizePanel}><MaximizeIcon /></IconButton>
-                            </Tooltip>}
-                        {doHidePanel &&
-                            <Tooltip title={`Close ${panelName || 'panel'}`} disableFocusListener>
-                                <IconButton disabled={panelMaximized} onClick={doHidePanel}><CloseIcon /></IconButton>
-                            </Tooltip>}
-                    </div>
-                } />
-            <CardContent className={classes.content}>
-                {(mainProcess || showParams) ?
-                    (<>
-                        {/* raw is undefined until params are loaded */}
-                        {loading && <Grid container item alignItems='center' justify='center'>
-                            <CircularProgress />
-                        </Grid>}
-                        {/* Once loaded, either raw or params may still be empty
-                         *   Raw when all params are empty
-                         *   Params when raw is provided by containerRequest properties but workflow mount is absent for preview
-                         */}
-                        {(!loading && (hasRaw || hasParams)) &&
+export const ProcessIOCard = withStyles(styles)(
+    connect(
+        null,
+        mapDispatchToProps
+    )(
+        ({
+            classes,
+            label,
+            params,
+            raw,
+            mounts,
+            outputUuid,
+            doHidePanel,
+            doMaximizePanel,
+            doUnMaximizePanel,
+            panelMaximized,
+            panelName,
+            process,
+            navigateTo,
+            forceShowParams,
+        }: ProcessIOCardProps) => {
+            const [mainProcTabState, setMainProcTabState] = useState(0);
+            const [subProcTabState, setSubProcTabState] = useState(0);
+            const handleMainProcTabChange = (event: React.MouseEvent<HTMLElement>, value: number) => {
+                setMainProcTabState(value);
+            };
+            const handleSubProcTabChange = (event: React.MouseEvent<HTMLElement>, value: number) => {
+                setSubProcTabState(value);
+            };
+
+            const PanelIcon = label === ProcessIOCardType.INPUT ? InputIcon : OutputIcon;
+            const mainProcess = !(process && process!.containerRequest.requestingContainerUuid);
+            const showParamTable = mainProcess || forceShowParams;
+
+            const loading = raw === null || raw === undefined || params === null;
+
+            const hasRaw = !!(raw && Object.keys(raw).length > 0);
+            const hasParams = !!(params && params.length > 0);
+            // isRawLoaded allows subprocess panel to display raw even if it's {}
+            const isRawLoaded = !!(raw && Object.keys(raw).length >= 0);
+
+            // Subprocess
+            const hasInputMounts = !!(label === ProcessIOCardType.INPUT && mounts && mounts.length);
+            const hasOutputCollecton = !!(label === ProcessIOCardType.OUTPUT && outputUuid);
+            // Subprocess should not show loading if hasOutputCollection or hasInputMounts
+            const subProcessLoading = loading && !hasOutputCollecton && !hasInputMounts;
+
+            return (
+                <Card
+                    className={classes.card}
+                    data-cy="process-io-card"
+                >
+                    <CardHeader
+                        className={classes.header}
+                        classes={{
+                            content: classes.title,
+                            avatar: classes.avatar,
+                        }}
+                        avatar={<PanelIcon className={classes.iconHeader} />}
+                        title={
+                            <Typography
+                                noWrap
+                                variant="h6"
+                                color="inherit"
+                            >
+                                {label}
+                            </Typography>
+                        }
+                        action={
+                            <div>
+                                {doUnMaximizePanel && panelMaximized && (
+                                    <Tooltip
+                                        title={`Unmaximize ${panelName || "panel"}`}
+                                        disableFocusListener
+                                    >
+                                        <IconButton onClick={doUnMaximizePanel}>
+                                            <UnMaximizeIcon />
+                                        </IconButton>
+                                    </Tooltip>
+                                )}
+                                {doMaximizePanel && !panelMaximized && (
+                                    <Tooltip
+                                        title={`Maximize ${panelName || "panel"}`}
+                                        disableFocusListener
+                                    >
+                                        <IconButton onClick={doMaximizePanel}>
+                                            <MaximizeIcon />
+                                        </IconButton>
+                                    </Tooltip>
+                                )}
+                                {doHidePanel && (
+                                    <Tooltip
+                                        title={`Close ${panelName || "panel"}`}
+                                        disableFocusListener
+                                    >
+                                        <IconButton
+                                            disabled={panelMaximized}
+                                            onClick={doHidePanel}
+                                        >
+                                            <CloseIcon />
+                                        </IconButton>
+                                    </Tooltip>
+                                )}
+                            </div>
+                        }
+                    />
+                    <CardContent className={classes.content}>
+                        {showParamTable ? (
                             <>
-                                <Tabs value={mainProcTabState} onChange={handleMainProcTabChange} variant="fullWidth" className={classes.symmetricTabs}>
-                                    {/* params will be empty on processes without workflow definitions in mounts, so we only show raw */}
-                                    {hasParams && <Tab label="Parameters" />}
-                                    {!showParams && <Tab label="JSON" />}
-                                </Tabs>
-                                {(mainProcTabState === 0 && params && hasParams) && <div className={classes.tableWrapper}>
-                                    <ProcessIOPreview data={params} showImagePreview={showImagePreview} valueLabel={showParams ? "Default value" : "Value"} />
-                                </div>}
-                                {(mainProcTabState === 1 || !hasParams) && <div className={classes.tableWrapper}>
-                                    <ProcessIORaw data={raw} />
-                                </div>}
-                            </>}
-                        {!loading && !hasRaw && !hasParams && <Grid container item alignItems='center' justify='center'>
-                            <DefaultView messages={["No parameters found"]} />
-                        </Grid>}
-                    </>) :
-                    // Subprocess
-                    (<>
-                        {loading && <Grid container item alignItems='center' justify='center'>
-                            <CircularProgress />
-                        </Grid>}
-                        {!loading && (hasInputMounts || hasOutputCollecton || hasRaw) ?
+                                {/* raw is undefined until params are loaded */}
+                                {loading && (
+                                    <Grid
+                                        container
+                                        item
+                                        alignItems="center"
+                                        justify="center"
+                                    >
+                                        <CircularProgress />
+                                    </Grid>
+                                )}
+                                {/* Once loaded, either raw or params may still be empty
+                                  *   Raw when all params are empty
+                                  *   Params when raw is provided by containerRequest properties but workflow mount is absent for preview
+                                  */}
+                                {!loading && (hasRaw || hasParams) && (
+                                    <>
+                                        <Tabs
+                                            value={mainProcTabState}
+                                            onChange={handleMainProcTabChange}
+                                            variant="fullWidth"
+                                            className={classes.symmetricTabs}
+                                        >
+                                            {/* params will be empty on processes without workflow definitions in mounts, so we only show raw */}
+                                            {hasParams && <Tab label="Parameters" />}
+                                            {!forceShowParams && <Tab label="JSON" />}
+                                            {hasOutputCollecton && <Tab label="Collection" />}
+                                        </Tabs>
+                                        {mainProcTabState === 0 && params && hasParams && (
+                                            <div className={classes.tableWrapper}>
+                                                <ProcessIOPreview
+                                                    data={params}
+                                                    valueLabel={forceShowParams ? "Default value" : "Value"}
+                                                />
+                                            </div>
+                                        )}
+                                        {(mainProcTabState === 1 || !hasParams) && (
+                                            <div className={classes.jsonWrapper}>
+                                                <ProcessIORaw data={raw} />
+                                            </div>
+                                        )}
+                                        {mainProcTabState === 2 && hasOutputCollecton && (
+                                            <>
+                                                {outputUuid && (
+                                                    <Typography className={classes.collectionLink}>
+                                                        Output Collection:{" "}
+                                                        <MuiLink
+                                                            className={classes.keepLink}
+                                                            onClick={() => {
+                                                                navigateTo(outputUuid || "");
+                                                            }}
+                                                        >
+                                                            {outputUuid}
+                                                        </MuiLink>
+                                                    </Typography>
+                                                )}
+                                                <ProcessOutputCollectionFiles
+                                                    isWritable={false}
+                                                    currentItemUuid={outputUuid}
+                                                />
+                                            </>
+                                        )}
+
+                                    </>
+                                )}
+                                {!loading && !hasRaw && !hasParams && (
+                                    <Grid
+                                        container
+                                        item
+                                        alignItems="center"
+                                        justify="center"
+                                    >
+                                        <DefaultView messages={["No parameters found"]} />
+                                    </Grid>
+                                )}
+                            </>
+                        ) : (
+                            // Subprocess
                             <>
-                                <Tabs value={subProcTabState} onChange={handleSubProcTabChange} variant="fullWidth" className={classes.symmetricTabs}>
-                                    {hasInputMounts && <Tab label="Collections" />}
-                                    {hasOutputCollecton && <Tab label="Collection" />}
-                                    <Tab label="JSON" />
-                                </Tabs>
-                                <div className={classes.tableWrapper}>
-                                    {subProcTabState === 0 && hasInputMounts && <ProcessInputMounts mounts={mounts || []} />}
-                                    {subProcTabState === 0 && hasOutputCollecton && <>
-                                        {outputUuid && <Typography className={classes.collectionLink}>
-                                            Output Collection: <MuiLink className={classes.keepLink} onClick={() => { navigateTo(outputUuid || "") }}>
-                                                {outputUuid}
-                                            </MuiLink></Typography>}
-                                        <ProcessOutputCollectionFiles isWritable={false} currentItemUuid={outputUuid} />
-                                    </>}
-                                    {(subProcTabState === 1 || (!hasInputMounts && !hasOutputCollecton)) && <div className={classes.tableWrapper}>
-                                        <ProcessIORaw data={raw} />
-                                    </div>}
-                                </div>
-                            </> :
-                            <Grid container item alignItems='center' justify='center'>
-                                <DefaultView messages={["No data to display"]} />
-                            </Grid>
-                        }
-                    </>)
-                }
-            </CardContent>
-        </Card>;
-    }
-));
+                                {subProcessLoading ? (
+                                    <Grid
+                                        container
+                                        item
+                                        alignItems="center"
+                                        justify="center"
+                                    >
+                                        <CircularProgress />
+                                    </Grid>
+                                ) : !subProcessLoading && (hasInputMounts || hasOutputCollecton || isRawLoaded) ? (
+                                    <>
+                                        <Tabs
+                                            value={subProcTabState}
+                                            onChange={handleSubProcTabChange}
+                                            variant="fullWidth"
+                                            className={classes.symmetricTabs}
+                                        >
+                                            {hasInputMounts && <Tab label="Collections" />}
+                                            {hasOutputCollecton && <Tab label="Collection" />}
+                                            {isRawLoaded && <Tab label="JSON" />}
+                                        </Tabs>
+                                        {subProcTabState === 0 && hasInputMounts && <ProcessInputMounts mounts={mounts || []} />}
+                                        {subProcTabState === 0 && hasOutputCollecton && (
+                                            <div className={classes.tableWrapper}>
+                                                <>
+                                                    {outputUuid && (
+                                                        <Typography className={classes.collectionLink}>
+                                                            Output Collection:{" "}
+                                                            <MuiLink
+                                                                className={classes.keepLink}
+                                                                onClick={() => {
+                                                                    navigateTo(outputUuid || "");
+                                                                }}
+                                                            >
+                                                                {outputUuid}
+                                                            </MuiLink>
+                                                        </Typography>
+                                                    )}
+                                                    <ProcessOutputCollectionFiles
+                                                        isWritable={false}
+                                                        currentItemUuid={outputUuid}
+                                                    />
+                                                </>
+                                            </div>
+                                        )}
+                                        {isRawLoaded && (subProcTabState === 1 || (!hasInputMounts && !hasOutputCollecton)) && (
+                                            <div className={classes.jsonWrapper}>
+                                                <ProcessIORaw data={raw} />
+                                            </div>
+                                        )}
+                                    </>
+                                ) : (
+                                    <Grid
+                                        container
+                                        item
+                                        alignItems="center"
+                                        justify="center"
+                                    >
+                                        <DefaultView messages={["No data to display"]} />
+                                    </Grid>
+                                )}
+                            </>
+                        )}
+                    </CardContent>
+                </Card>
+            );
+        }
+    )
+);
 
 export type ProcessIOValue = {
     display: ReactElement<any, any>;
     imageUrl?: string;
     collection?: ReactElement<any, any>;
     secondary?: boolean;
-}
+};
 
 export type ProcessIOParameter = {
     id: string;
     label: string;
-    value: ProcessIOValue[];
-}
+    value: ProcessIOValue;
+};
 
 interface ProcessIOPreviewDataProps {
     data: ProcessIOParameter[];
-    showImagePreview: boolean;
     valueLabel: string;
 }
 
 type ProcessIOPreviewProps = ProcessIOPreviewDataProps & WithStyles<CssRules>;
 
-const ProcessIOPreview = memo(withStyles(styles)(
-    ({ classes, data, showImagePreview, valueLabel }: ProcessIOPreviewProps) => {
+const ProcessIOPreview = memo(
+    withStyles(styles)(({ classes, data, valueLabel }: ProcessIOPreviewProps) => {
         const showLabel = data.some((param: ProcessIOParameter) => param.label);
-        return <Table className={classes.tableRoot} aria-label="Process IO Preview">
-            <TableHead>
-                <TableRow>
-                    <TableCell>Name</TableCell>
-                    {showLabel && <TableCell className={classes.labelColumn}>Label</TableCell>}
-                    <TableCell>{valueLabel}</TableCell>
-                    <TableCell>Collection</TableCell>
-                </TableRow>
-            </TableHead>
-            <TableBody>
-                {data.map((param: ProcessIOParameter) => {
-                    const firstVal = param.value.length > 0 ? param.value[0] : undefined;
-                    const rest = param.value.slice(1);
-                    const mainRowClasses = {
-                        [classes.noBorderRow]: (rest.length > 0),
-                    };
-
-                    return <React.Fragment key={param.id}>
-                        <TableRow className={classNames(mainRowClasses)} data-cy="process-io-param">
-                            <TableCell>
+
+        const hasMoreValues = (index: number) => (
+            data[index+1] && !isMainRow(data[index+1])
+        );
+
+        const isMainRow = (param: ProcessIOParameter) => (
+            param &&
+            ((param.id || param.label) &&
+            !param.value.secondary)
+        );
+
+        const RenderRow = ({index, style}) => {
+            const param = data[index];
+
+            const rowClasses = {
+                [classes.noBorderRow]: hasMoreValues(index),
+            };
+
+            return <TableRow
+                style={style}
+                className={classNames(rowClasses)}
+                data-cy={isMainRow(param) ? "process-io-param" : ""}>
+                <TableCell>
+                    <Tooltip title={param.id}>
+                        <Typography className={classes.paramTableCellText}>
+                            <span>
                                 {param.id}
-                            </TableCell>
-                            {showLabel && <TableCell >{param.label}</TableCell>}
-                            <TableCell>
-                                {firstVal && <ProcessValuePreview value={firstVal} showImagePreview={showImagePreview} />}
-                            </TableCell>
-                            <TableCell className={firstVal?.imageUrl ? classes.rowWithPreview : undefined}>
-                                <Typography className={classes.paramValue}>
-                                    {firstVal?.collection}
-                                </Typography>
-                            </TableCell>
-                        </TableRow>
-                        {rest.map((val, i) => {
-                            const rowClasses = {
-                                [classes.noBorderRow]: (i < rest.length - 1),
-                                [classes.secondaryRow]: val.secondary,
-                            };
-                            return <TableRow className={classNames(rowClasses)} key={i}>
-                                <TableCell />
-                                {showLabel && <TableCell />}
-                                <TableCell>
-                                    <ProcessValuePreview value={val} showImagePreview={showImagePreview} />
-                                </TableCell>
-                                <TableCell className={firstVal?.imageUrl ? classes.rowWithPreview : undefined}>
-                                    <Typography className={classes.paramValue}>
-                                        {val.collection}
-                                    </Typography>
-                                </TableCell>
-                            </TableRow>
-                        })}
-                    </React.Fragment>;
-                })}
-            </TableBody>
-        </Table >;
-    }));
+                            </span>
+                        </Typography>
+                    </Tooltip>
+                </TableCell>
+                {showLabel && <TableCell>
+                    <Tooltip title={param.label}>
+                        <Typography className={classes.paramTableCellText}>
+                            <span>
+                                {param.label}
+                            </span>
+                        </Typography>
+                    </Tooltip>
+                </TableCell>}
+                <TableCell>
+                    <ProcessValuePreview
+                        value={param.value}
+                    />
+                </TableCell>
+                <TableCell>
+                    <Typography className={classes.paramTableCellText}>
+                        {/** Collection is an anchor so doesn't require wrapper element */}
+                        {param.value.collection}
+                    </Typography>
+                </TableCell>
+            </TableRow>;
+        };
+
+        return (
+            <Table
+                className={classes.paramTableRoot}
+                aria-label="Process IO Preview"
+            >
+                <TableHead>
+                    <TableRow>
+                        <TableCell>Name</TableCell>
+                        {showLabel && <TableCell>Label</TableCell>}
+                        <TableCell>{valueLabel}</TableCell>
+                        <TableCell>Collection</TableCell>
+                    </TableRow>
+                </TableHead>
+                <TableBody>
+                    <AutoSizer>
+                        {({ height, width }) =>
+                            <FixedSizeList
+                                height={height}
+                                itemCount={data.length}
+                                itemSize={40}
+                                width={width}
+                            >
+                                {RenderRow}
+                            </FixedSizeList>
+                        }
+                    </AutoSizer>
+                </TableBody>
+            </Table>
+        );
+    })
+);
 
 interface ProcessValuePreviewProps {
     value: ProcessIOValue;
-    showImagePreview: boolean;
 }
 
-const ProcessValuePreview = withStyles(styles)(
-    ({ value, showImagePreview, classes }: ProcessValuePreviewProps & WithStyles<CssRules>) =>
-        <Typography className={classes.paramValue}>
-            {value.imageUrl && showImagePreview ? <img className={classes.imagePreview} src={value.imageUrl} alt="Inline Preview" /> : ""}
-            {value.imageUrl && !showImagePreview ? <ImagePlaceholder /> : ""}
-            <span className={classNames(classes.valArray, value.secondary && classes.secondaryVal)}>
-                {value.display}
-            </span>
-        </Typography>
-)
+const ProcessValuePreview = withStyles(styles)(({ value, classes }: ProcessValuePreviewProps & WithStyles<CssRules>) => (
+    <Typography className={classNames(classes.paramTableCellText, value.secondary && classes.secondaryVal)}>
+        {value.display}
+    </Typography>
+));
 
 interface ProcessIORawDataProps {
     data: ProcessIOParameter[];
 }
 
-const ProcessIORaw = withStyles(styles)(
-    ({ data }: ProcessIORawDataProps) =>
-        <Paper elevation={0}>
-            <DefaultCodeSnippet lines={[JSON.stringify(data, null, 2)]} linked />
-        </Paper>
-);
+const ProcessIORaw = withStyles(styles)(({ data }: ProcessIORawDataProps) => (
+    <Paper elevation={0} style={{minWidth: "100%", height: "100%"}}>
+        <DefaultVirtualCodeSnippet
+            lines={JSON.stringify(data, null, 2).split('\n')}
+            linked
+        />
+    </Paper>
+));
 
 interface ProcessInputMountsDataProps {
     mounts: InputCollectionMount[];
@@ -477,146 +674,168 @@ interface ProcessInputMountsDataProps {
 
 type ProcessInputMountsProps = ProcessInputMountsDataProps & WithStyles<CssRules>;
 
-const ProcessInputMounts = withStyles(styles)(connect((state: RootState) => ({
-    auth: state.auth,
-}))(({ mounts, classes, auth }: ProcessInputMountsProps & { auth: AuthState }) => (
-    <Table className={classes.tableRoot} aria-label="Process Input Mounts">
-        <TableHead>
-            <TableRow>
-                <TableCell>Path</TableCell>
-                <TableCell>Portable Data Hash</TableCell>
-            </TableRow>
-        </TableHead>
-        <TableBody>
-            {mounts.map(mount => (
-                <TableRow key={mount.path}>
-                    <TableCell><pre>{mount.path}</pre></TableCell>
-                    <TableCell>
-                        <RouterLink to={getNavUrl(mount.pdh, auth)} className={classes.keepLink}>{mount.pdh}</RouterLink>
-                    </TableCell>
+const ProcessInputMounts = withStyles(styles)(
+    connect((state: RootState) => ({
+        auth: state.auth,
+    }))(({ mounts, classes, auth }: ProcessInputMountsProps & { auth: AuthState }) => (
+        <Table
+            className={classes.mountsTableRoot}
+            aria-label="Process Input Mounts"
+        >
+            <TableHead>
+                <TableRow>
+                    <TableCell>Path</TableCell>
+                    <TableCell>Portable Data Hash</TableCell>
                 </TableRow>
-            ))}
-        </TableBody>
-    </Table>
-)));
+            </TableHead>
+            <TableBody>
+                {mounts.map(mount => (
+                    <TableRow key={mount.path}>
+                        <TableCell>
+                            <pre>{mount.path}</pre>
+                        </TableCell>
+                        <TableCell>
+                            <RouterLink
+                                to={getNavUrl(mount.pdh, auth)}
+                                className={classes.keepLink}
+                            >
+                                {mount.pdh}
+                            </RouterLink>
+                        </TableCell>
+                    </TableRow>
+                ))}
+            </TableBody>
+        </Table>
+    ))
+);
 
 type FileWithSecondaryFiles = {
     secondaryFiles: File[];
-}
+};
 
 export const getIOParamDisplayValue = (auth: AuthState, input: CommandInputParameter | CommandOutputParameter, pdh?: string): ProcessIOValue[] => {
     switch (true) {
         case isPrimitiveOfType(input, CWLType.BOOLEAN):
             const boolValue = (input as BooleanCommandInputParameter).value;
-            return boolValue !== undefined &&
-                !(Array.isArray(boolValue) && boolValue.length === 0) ?
-                [{ display: renderPrimitiveValue(boolValue, false) }] :
-                [{ display: <EmptyValue /> }];
+            return boolValue !== undefined && !(Array.isArray(boolValue) && boolValue.length === 0)
+                ? [{ display: <PrimitiveTooltip data={boolValue}>{renderPrimitiveValue(boolValue, false)}</PrimitiveTooltip> }]
+                : [{ display: <EmptyValue /> }];
 
         case isPrimitiveOfType(input, CWLType.INT):
         case isPrimitiveOfType(input, CWLType.LONG):
             const intValue = (input as IntCommandInputParameter).value;
             return intValue !== undefined &&
                 // Missing values are empty array
-                !(Array.isArray(intValue) && intValue.length === 0) ?
-                [{ display: renderPrimitiveValue(intValue, false) }]
+                !(Array.isArray(intValue) && intValue.length === 0)
+                ? [{ display: <PrimitiveTooltip data={intValue}>{renderPrimitiveValue(intValue, false)}</PrimitiveTooltip> }]
                 : [{ display: <EmptyValue /> }];
 
         case isPrimitiveOfType(input, CWLType.FLOAT):
         case isPrimitiveOfType(input, CWLType.DOUBLE):
             const floatValue = (input as FloatCommandInputParameter).value;
-            return floatValue !== undefined &&
-                !(Array.isArray(floatValue) && floatValue.length === 0) ?
-                [{ display: renderPrimitiveValue(floatValue, false) }] :
-                [{ display: <EmptyValue /> }];
+            return floatValue !== undefined && !(Array.isArray(floatValue) && floatValue.length === 0)
+                ? [{ display: <PrimitiveTooltip data={floatValue}>{renderPrimitiveValue(floatValue, false)}</PrimitiveTooltip> }]
+                : [{ display: <EmptyValue /> }];
 
         case isPrimitiveOfType(input, CWLType.STRING):
             const stringValue = (input as StringCommandInputParameter).value || undefined;
-            return stringValue !== undefined &&
-                !(Array.isArray(stringValue) && stringValue.length === 0) ?
-                [{ display: renderPrimitiveValue(stringValue, false) }] :
-                [{ display: <EmptyValue /> }];
+            return stringValue !== undefined && !(Array.isArray(stringValue) && stringValue.length === 0)
+                ? [{ display: <PrimitiveTooltip data={stringValue}>{renderPrimitiveValue(stringValue, false)}</PrimitiveTooltip> }]
+                : [{ display: <EmptyValue /> }];
 
         case isPrimitiveOfType(input, CWLType.FILE):
             const mainFile = (input as FileCommandInputParameter).value;
             // secondaryFiles: File[] is not part of CommandOutputParameter so we cast to access secondaryFiles
-            const secondaryFiles = ((mainFile as unknown) as FileWithSecondaryFiles)?.secondaryFiles || [];
-            const files = [
-                ...(mainFile && !(Array.isArray(mainFile) && mainFile.length === 0) ? [mainFile] : []),
-                ...secondaryFiles
-            ];
+            const secondaryFiles = (mainFile as unknown as FileWithSecondaryFiles)?.secondaryFiles || [];
+            const files = [...(mainFile && !(Array.isArray(mainFile) && mainFile.length === 0) ? [mainFile] : []), ...secondaryFiles];
             const mainFilePdhUrl = mainFile ? getResourcePdhUrl(mainFile, pdh) : "";
-            return files.length ?
-                files.map((file, i) => fileToProcessIOValue(file, (i > 0), auth, pdh, (i > 0 ? mainFilePdhUrl : ""))) :
-                [{ display: <EmptyValue /> }];
+            return files.length
+                ? files.map((file, i) => fileToProcessIOValue(file, i > 0, auth, pdh, i > 0 ? mainFilePdhUrl : ""))
+                [{ display: <EmptyValue /> }];
 
         case isPrimitiveOfType(input, CWLType.DIRECTORY):
             const directory = (input as DirectoryCommandInputParameter).value;
-            return directory !== undefined &&
-                !(Array.isArray(directory) && directory.length === 0) ?
-                [directoryToProcessIOValue(directory, auth, pdh)] :
-                [{ display: <EmptyValue /> }];
+            return directory !== undefined && !(Array.isArray(directory) && directory.length === 0)
+                ? [directoryToProcessIOValue(directory, auth, pdh)]
+                : [{ display: <EmptyValue /> }];
 
         case getEnumType(input) !== null:
             const enumValue = (input as EnumCommandInputParameter).value;
-            return enumValue !== undefined && enumValue ?
-                [{ display: <pre>{enumValue}</pre> }] :
-                [{ display: <EmptyValue /> }];
+            return enumValue !== undefined && enumValue ? [{ display: <PrimitiveTooltip data={enumValue}>{enumValue}</PrimitiveTooltip> }] : [{ display: <EmptyValue /> }];
 
         case isArrayOfType(input, CWLType.STRING):
             const strArray = (input as StringArrayCommandInputParameter).value || [];
-            return strArray.length ?
-                [{ display: <>{strArray.map((val) => renderPrimitiveValue(val, true))}</> }] :
-                [{ display: <EmptyValue /> }];
+            return strArray.length ? [{ display: <PrimitiveArrayTooltip data={strArray}>{strArray.map(val => renderPrimitiveValue(val, true))}</PrimitiveArrayTooltip> }] : [{ display: <EmptyValue /> }];
 
         case isArrayOfType(input, CWLType.INT):
         case isArrayOfType(input, CWLType.LONG):
             const intArray = (input as IntArrayCommandInputParameter).value || [];
-            return intArray.length ?
-                [{ display: <>{intArray.map((val) => renderPrimitiveValue(val, true))}</> }] :
-                [{ display: <EmptyValue /> }];
+            return intArray.length ? [{ display: <PrimitiveArrayTooltip data={intArray}>{intArray.map(val => renderPrimitiveValue(val, true))}</PrimitiveArrayTooltip> }] : [{ display: <EmptyValue /> }];
 
         case isArrayOfType(input, CWLType.FLOAT):
         case isArrayOfType(input, CWLType.DOUBLE):
             const floatArray = (input as FloatArrayCommandInputParameter).value || [];
-            return floatArray.length ?
-                [{ display: <>{floatArray.map((val) => renderPrimitiveValue(val, true))}</> }] :
-                [{ display: <EmptyValue /> }];
+            return floatArray.length ? [{ display: <PrimitiveArrayTooltip data={floatArray}>{floatArray.map(val => renderPrimitiveValue(val, true))}</PrimitiveArrayTooltip> }] : [{ display: <EmptyValue /> }];
 
         case isArrayOfType(input, CWLType.FILE):
-            const fileArrayMainFiles = ((input as FileArrayCommandInputParameter).value || []);
-            const firstMainFilePdh = (fileArrayMainFiles.length > 0 && fileArrayMainFiles[0]) ? getResourcePdhUrl(fileArrayMainFiles[0], pdh) : "";
+            const fileArrayMainFiles = (input as FileArrayCommandInputParameter).value || [];
+            const firstMainFilePdh = fileArrayMainFiles.length > 0 && fileArrayMainFiles[0] ? getResourcePdhUrl(fileArrayMainFiles[0], pdh) : "";
 
             // Convert each main and secondaryFiles into array of ProcessIOValue preserving ordering
             let fileArrayValues: ProcessIOValue[] = [];
             for (let i = 0; i < fileArrayMainFiles.length; i++) {
-                const secondaryFiles = ((fileArrayMainFiles[i] as unknown) as FileWithSecondaryFiles)?.secondaryFiles || [];
+                const secondaryFiles = (fileArrayMainFiles[i] as unknown as FileWithSecondaryFiles)?.secondaryFiles || [];
                 fileArrayValues.push(
                     // Pass firstMainFilePdh to secondary files and every main file besides the first to hide pdh if equal
                     ...(fileArrayMainFiles[i] ? [fileToProcessIOValue(fileArrayMainFiles[i], false, auth, pdh, i > 0 ? firstMainFilePdh : "")] : []),
-                    ...(secondaryFiles.map(file => fileToProcessIOValue(file, true, auth, pdh, firstMainFilePdh)))
+                    ...secondaryFiles.map(file => fileToProcessIOValue(file, true, auth, pdh, firstMainFilePdh))
                 );
             }
 
-            return fileArrayValues.length ?
-                fileArrayValues :
-                [{ display: <EmptyValue /> }];
+            return fileArrayValues.length ? fileArrayValues : [{ display: <EmptyValue /> }];
 
         case isArrayOfType(input, CWLType.DIRECTORY):
             const directories = (input as DirectoryArrayCommandInputParameter).value || [];
-            return directories.length ?
-                directories.map(directory => directoryToProcessIOValue(directory, auth, pdh)) :
-                [{ display: <EmptyValue /> }];
+            return directories.length ? directories.map(directory => directoryToProcessIOValue(directory, auth, pdh)) : [{ display: <EmptyValue /> }];
 
         default:
             return [{ display: <UnsupportedValue /> }];
     }
 };
 
+interface PrimitiveTooltipProps {
+    data: boolean | number | string;
+}
+
+const PrimitiveTooltip = (props: React.PropsWithChildren<PrimitiveTooltipProps>) => (
+    <Tooltip title={typeof props.data !== 'object' ? String(props.data) : ""}>
+        <pre>{props.children}</pre>
+    </Tooltip>
+);
+
+interface PrimitiveArrayTooltipProps {
+    data: string[];
+}
+
+const PrimitiveArrayTooltip = (props: React.PropsWithChildren<PrimitiveArrayTooltipProps>) => (
+    <Tooltip title={props.data.join(', ')}>
+        <span>{props.children}</span>
+    </Tooltip>
+);
+
+
 const renderPrimitiveValue = (value: any, asChip: boolean) => {
-    const isObject = typeof value === 'object';
+    const isObject = typeof value === "object";
     if (!isObject) {
-        return asChip ? <Chip label={String(value)} /> : <pre>{String(value)}</pre>;
+        return asChip ? (
+            <Chip
+                key={value}
+                label={String(value)}
+                style={{marginRight: "10px"}}
+            />
+        ) : (
+            <>{String(value)}</>
+        );
     } else {
         return asChip ? <UnsupportedValueChip /> : <UnsupportedValue />;
     }
@@ -626,11 +845,9 @@ const renderPrimitiveValue = (value: any, asChip: boolean) => {
  * @returns keep url without keep: prefix
  */
 const getKeepUrl = (file: File | Directory, pdh?: string): string => {
-    const isKeepUrl = file.location?.startsWith('keep:') || false;
-    const keepUrl = isKeepUrl ?
-        file.location?.replace('keep:', '') :
-        pdh ? `${pdh}/${file.location}` : file.location;
-    return keepUrl || '';
+    const isKeepUrl = file.location?.startsWith("keep:") || false;
+    const keepUrl = isKeepUrl ? file.location?.replace("keep:", "") : pdh ? `${pdh}/${file.location}` : file.location;
+    return keepUrl || "";
 };
 
 interface KeepUrlProps {
@@ -641,47 +858,73 @@ interface KeepUrlProps {
 
 const getResourcePdhUrl = (res: File | Directory, pdh?: string): string => {
     const keepUrl = getKeepUrl(res, pdh);
-    return keepUrl ? keepUrl.split('/').slice(0, 1)[0] : '';
+    return keepUrl ? keepUrl.split("/").slice(0, 1)[0] : "";
 };
 
 const KeepUrlBase = withStyles(styles)(({ auth, res, pdh, classes }: KeepUrlProps & WithStyles<CssRules>) => {
     const pdhUrl = getResourcePdhUrl(res, pdh);
     // Passing a pdh always returns a relative wb2 collection url
     const pdhWbPath = getNavUrl(pdhUrl, auth);
-    return pdhUrl && pdhWbPath ?
-        <Tooltip title={"View collection in Workbench"}><RouterLink to={pdhWbPath} className={classes.keepLink}>{pdhUrl}</RouterLink></Tooltip> :
-        <></>;
+    return pdhUrl && pdhWbPath ? (
+        <Tooltip title={<>View collection in Workbench<br />{pdhUrl}</>}>
+            <RouterLink
+                to={pdhWbPath}
+                className={classes.keepLink}
+            >
+                {pdhUrl}
+            </RouterLink>
+        </Tooltip>
+    ) : (
+        <></>
+    );
 });
 
 const KeepUrlPath = withStyles(styles)(({ auth, res, pdh, classes }: KeepUrlProps & WithStyles<CssRules>) => {
     const keepUrl = getKeepUrl(res, pdh);
-    const keepUrlParts = keepUrl ? keepUrl.split('/') : [];
-    const keepUrlPath = keepUrlParts.length > 1 ? keepUrlParts.slice(1).join('/') : '';
+    const keepUrlParts = keepUrl ? keepUrl.split("/") : [];
+    const keepUrlPath = keepUrlParts.length > 1 ? keepUrlParts.slice(1).join("/") : "";
 
     const keepUrlPathNav = getKeepNavUrl(auth, res, pdh);
-    return keepUrlPathNav ?
-        <Tooltip title={"View in keep-web"}><a className={classes.keepLink} href={keepUrlPathNav} target="_blank" rel="noopener noreferrer">{keepUrlPath || '/'}</a></Tooltip> :
-        <EmptyValue />;
+    return keepUrlPathNav ? (
+        <Tooltip classes={{tooltip: classes.wrapTooltip}} title={<>View in keep-web<br />{keepUrlPath || "/"}</>}>
+            <a
+                className={classes.keepLink}
+                href={keepUrlPathNav}
+                target="_blank"
+                rel="noopener noreferrer"
+            >
+                {keepUrlPath || "/"}
+            </a>
+        </Tooltip>
+    ) : (
+        <EmptyValue />
+    );
 });
 
 const getKeepNavUrl = (auth: AuthState, file: File | Directory, pdh?: string): string => {
     let keepUrl = getKeepUrl(file, pdh);
-    return (getInlineFileUrl(`${auth.config.keepWebServiceUrl}/c=${keepUrl}?api_token=${auth.apiToken}`, auth.config.keepWebServiceUrl, auth.config.keepWebInlineServiceUrl));
+    return getInlineFileUrl(
+        `${auth.config.keepWebServiceUrl}/c=${keepUrl}?api_token=${auth.apiToken}`,
+        auth.config.keepWebServiceUrl,
+        auth.config.keepWebInlineServiceUrl
+    );
 };
 
 const getImageUrl = (auth: AuthState, file: File, pdh?: string): string => {
     const keepUrl = getKeepUrl(file, pdh);
-    return getInlineFileUrl(`${auth.config.keepWebServiceUrl}/c=${keepUrl}?api_token=${auth.apiToken}`, auth.config.keepWebServiceUrl, auth.config.keepWebInlineServiceUrl);
+    return getInlineFileUrl(
+        `${auth.config.keepWebServiceUrl}/c=${keepUrl}?api_token=${auth.apiToken}`,
+        auth.config.keepWebServiceUrl,
+        auth.config.keepWebInlineServiceUrl
+    );
 };
 
 const isFileImage = (basename?: string): boolean => {
-    return basename ? (mime.getType(basename) || "").startsWith('image/') : false;
+    return basename ? (mime.getType(basename) || "").startsWith("image/") : false;
 };
 
-const isFileUrl = (location?: string): boolean => (
-    !!location && !KEEP_URL_REGEX.exec(location) &&
-    (location.startsWith("http://") || location.startsWith("https://"))
-);
+const isFileUrl = (location?: string): boolean =>
+    !!location && !KEEP_URL_REGEX.exec(location) && (location.startsWith("http://") || location.startsWith("https://"));
 
 const normalizeDirectoryLocation = (directory: Directory): Directory => {
     if (!directory.location) {
@@ -689,55 +932,98 @@ const normalizeDirectoryLocation = (directory: Directory): Directory => {
     }
     return {
         ...directory,
-        location: (directory.location || '').endsWith('/') ? directory.location : directory.location + '/',
+        location: (directory.location || "").endsWith("/") ? directory.location : directory.location + "/",
     };
 };
 
 const directoryToProcessIOValue = (directory: Directory, auth: AuthState, pdh?: string): ProcessIOValue => {
-    if (isExternalValue(directory)) { return { display: <UnsupportedValue /> } }
+    if (isExternalValue(directory)) {
+        return { display: <UnsupportedValue /> };
+    }
 
     const normalizedDirectory = normalizeDirectoryLocation(directory);
     return {
-        display: <KeepUrlPath auth={auth} res={normalizedDirectory} pdh={pdh} />,
-        collection: <KeepUrlBase auth={auth} res={normalizedDirectory} pdh={pdh} />,
+        display: (
+            <KeepUrlPath
+                auth={auth}
+                res={normalizedDirectory}
+                pdh={pdh}
+            />
+        ),
+        collection: (
+            <KeepUrlBase
+                auth={auth}
+                res={normalizedDirectory}
+                pdh={pdh}
+            />
+        ),
     };
 };
 
+type MuiLinkWithTooltipProps = WithStyles<CssRules> & React.PropsWithChildren<LinkProps>;
+
+const MuiLinkWithTooltip = withStyles(styles)((props: MuiLinkWithTooltipProps) => (
+    <Tooltip title={props.title} classes={{tooltip: props.classes.wrapTooltip}}>
+        <MuiLink {...props}>
+            {props.children}
+        </MuiLink>
+    </Tooltip>
+));
+
 const fileToProcessIOValue = (file: File, secondary: boolean, auth: AuthState, pdh: string | undefined, mainFilePdh: string): ProcessIOValue => {
-    if (isExternalValue(file)) { return { display: <UnsupportedValue /> } }
+    if (isExternalValue(file)) {
+        return { display: <UnsupportedValue /> };
+    }
 
     if (isFileUrl(file.location)) {
         return {
-            display: <MuiLink href={file.location} target="_blank">{file.location}</MuiLink>,
+            display: (
+                <MuiLinkWithTooltip
+                    href={file.location}
+                    target="_blank"
+                    rel="noopener"
+                    title={file.location}
+                >
+                    {file.location}
+                </MuiLinkWithTooltip>
+            ),
             secondary,
         };
     }
 
     const resourcePdh = getResourcePdhUrl(file, pdh);
     return {
-        display: <KeepUrlPath auth={auth} res={file} pdh={pdh} />,
+        display: (
+            <KeepUrlPath
+                auth={auth}
+                res={file}
+                pdh={pdh}
+            />
+        ),
         secondary,
         imageUrl: isFileImage(file.basename) ? getImageUrl(auth, file, pdh) : undefined,
-        collection: (resourcePdh !== mainFilePdh) ? <KeepUrlBase auth={auth} res={file} pdh={pdh} /> : <></>,
-    }
+        collection:
+            resourcePdh !== mainFilePdh ? (
+                <KeepUrlBase
+                    auth={auth}
+                    res={file}
+                    pdh={pdh}
+                />
+            ) : (
+                <></>
+            ),
+    };
 };
 
-const isExternalValue = (val: any) =>
-    Object.keys(val).includes('$import') ||
-    Object.keys(val).includes('$include')
+const isExternalValue = (val: any) => Object.keys(val).includes("$import") || Object.keys(val).includes("$include");
 
-export const EmptyValue = withStyles(styles)(
-    ({ classes }: WithStyles<CssRules>) => <span className={classes.emptyValue}>No value</span>
-);
+export const EmptyValue = withStyles(styles)(({ classes }: WithStyles<CssRules>) => <span className={classes.emptyValue}>No value</span>);
 
-const UnsupportedValue = withStyles(styles)(
-    ({ classes }: WithStyles<CssRules>) => <span className={classes.emptyValue}>Cannot display value</span>
-);
-
-const UnsupportedValueChip = withStyles(styles)(
-    ({ classes }: WithStyles<CssRules>) => <Chip icon={<InfoIcon />} label={"Cannot display value"} />
-);
+const UnsupportedValue = withStyles(styles)(({ classes }: WithStyles<CssRules>) => <span className={classes.emptyValue}>Cannot display value</span>);
 
-const ImagePlaceholder = withStyles(styles)(
-    ({ classes }: WithStyles<CssRules>) => <span className={classes.imagePlaceholder}><ImageIcon /></span>
-);
+const UnsupportedValueChip = withStyles(styles)(({ classes }: WithStyles<CssRules>) => (
+    <Chip
+        icon={<InfoIcon />}
+        label={"Cannot display value"}
+    />
+));