X-Git-Url: https://git.arvados.org/arvados-workbench2.git/blobdiff_plain/9cd0c2757db479fb75e150d52d4871d9a88af364..3f7e1a8afad27920adf8f03ce82eeb1ae58aa84f:/src/views/process-panel/process-log-code-snippet.tsx diff --git a/src/views/process-panel/process-log-code-snippet.tsx b/src/views/process-panel/process-log-code-snippet.tsx index 6ea628e6..2b7391c2 100644 --- a/src/views/process-panel/process-log-code-snippet.tsx +++ b/src/views/process-panel/process-log-code-snippet.tsx @@ -13,10 +13,12 @@ import { import grey from '@material-ui/core/colors/grey'; import { ArvadosTheme } from 'common/custom-theme'; import { Link, Typography } from '@material-ui/core'; -import { navigateTo } from 'store/navigation/navigation-action'; +import { navigationNotAvailable } from 'store/navigation/navigation-action'; import { Dispatch } from 'redux'; import { connect, DispatchProp } from 'react-redux'; import classNames from 'classnames'; +import { FederationConfig, getNavUrl } from 'routes/routes'; +import { RootState } from 'store/store'; type CssRules = 'root' | 'wordWrap' | 'logText'; @@ -26,6 +28,9 @@ const styles: StyleRulesCallback = (theme: ArvadosTheme) => ({ overflow: 'auto', backgroundColor: '#000', height: `calc(100% - ${theme.spacing.unit * 4}px)`, // so that horizontal scollbar is visible + "& a": { + color: theme.palette.primary.main, + }, }, logText: { padding: theme.spacing.unit * 0.5, @@ -55,7 +60,11 @@ interface ProcessLogCodeSnippetProps { wordWrap?: boolean; } -const renderLinks = (fontSize: number, dispatch: Dispatch) => (text: string) => { +interface ProcessLogCodeSnippetAuthProps { + auth: FederationConfig; +} + +const renderLinks = (fontSize: number, auth: FederationConfig, dispatch: Dispatch) => (text: string) => { // Matches UUIDs & PDHs const REGEX = /[a-z0-9]{5}-[a-z0-9]{5}-[a-z0-9]{15}|[0-9a-f]{32}\+\d+/g; const links = text.match(REGEX); @@ -67,7 +76,14 @@ const renderLinks = (fontSize: number, dispatch: Dispatch) => (text: string) => {part} {links[index] && - dispatch(navigateTo(links[index]))} + { + const url = getNavUrl(links[index], auth) + if (url) { + window.open(`${window.location.origin}${url}`, '_blank'); + } else { + dispatch(navigationNotAvailable(links[index])); + } + }} style={ {cursor: 'pointer'} }> {links[index]} } @@ -76,9 +92,13 @@ const renderLinks = (fontSize: number, dispatch: Dispatch) => (text: string) => ; }; -export const ProcessLogCodeSnippet = withStyles(styles)(connect()( - ({classes, lines, fontSize, dispatch, wordWrap}: ProcessLogCodeSnippetProps & WithStyles & DispatchProp) => { - const [followMode, setFollowMode] = useState(false); +const mapStateToProps = (state: RootState): ProcessLogCodeSnippetAuthProps => ({ + auth: state.auth, +}); + +export const ProcessLogCodeSnippet = withStyles(styles)(connect(mapStateToProps)( + ({classes, lines, fontSize, auth, dispatch, wordWrap}: ProcessLogCodeSnippetProps & WithStyles & ProcessLogCodeSnippetAuthProps & DispatchProp) => { + const [followMode, setFollowMode] = useState(true); const scrollRef = useRef(null); useEffect(() => { @@ -92,7 +112,7 @@ export const ProcessLogCodeSnippet = withStyles(styles)(connect()(
{ const elem = e.target as HTMLDivElement; - if (elem.scrollTop + elem.clientHeight >= elem.scrollHeight) { + if (elem.scrollTop + (elem.clientHeight*1.1) >= elem.scrollHeight) { setFollowMode(true); } else { setFollowMode(false); @@ -101,7 +121,7 @@ export const ProcessLogCodeSnippet = withStyles(styles)(connect()( { lines.map((line: string, index: number) => - {renderLinks(fontSize, dispatch)(line)} + {renderLinks(fontSize, auth, dispatch)(line)} ) }