From 6c12dcde231f8a6d419a4e154d6906bce944963c Mon Sep 17 00:00:00 2001
From: Michal Klobukowski <michal.klobukowski@contractors.roche.com>
Date: Mon, 10 Dec 2018 09:23:42 +0100
Subject: [PATCH] Improve WorkflowPresetSelect layout

Feature #14490

Arvados-DCO-1.1-Signed-off-by: Michal Klobukowski <michal.klobukowski@contractors.roche.com>
---
 .../workflow-preset-select.tsx                | 92 +++++++++----------
 1 file changed, 45 insertions(+), 47 deletions(-)

diff --git a/src/views/run-process-panel/workflow-preset-select.tsx b/src/views/run-process-panel/workflow-preset-select.tsx
index 46bdecdf..a69be951 100644
--- a/src/views/run-process-panel/workflow-preset-select.tsx
+++ b/src/views/run-process-panel/workflow-preset-select.tsx
@@ -3,7 +3,7 @@
 // SPDX-License-Identifier: AGPL-3.0
 
 import * as React from 'react';
-import { Select, FormControl, InputLabel, MenuItem, Tooltip, Grid, withStyles, WithStyles } from '@material-ui/core';
+import { Select, FormControl, InputLabel, MenuItem, Tooltip, withStyles, WithStyles } from '@material-ui/core';
 import { WorkflowResource } from '~/models/workflow';
 import { DetailsIcon } from '~/components/icon/icon';
 
@@ -14,57 +14,55 @@ export interface WorkflowPresetSelectProps {
     onChange: (preset: WorkflowResource) => void;
 }
 
-export class WorkflowPresetSelect extends React.Component<WorkflowPresetSelectProps> {
+type CssRules = 'root' | 'icon';
 
-    render() {
+export const WorkflowPresetSelect = withStyles<CssRules>(theme => ({
+    root: {
+        display: 'flex',
+    },
+    icon: {
+        color: theme.palette.text.hint,
+        marginTop: 18,
+        marginLeft: 8,
+    },
+}))(
+    class extends React.Component<WorkflowPresetSelectProps & WithStyles<CssRules>> {
 
-        const { selectedPreset, workflow, presets } = this.props;
+        render() {
 
-        return (
-            <Grid container wrap='nowrap' spacing={32}>
-                <Grid item xs container>
-                    <Grid item xs>
-                        <FormControl fullWidth>
-                            <InputLabel htmlFor="age-simple">Preset</InputLabel>
-                            <Select
-                                value={selectedPreset.uuid}
-                                onChange={this.handleChange}>
-                                <MenuItem value={workflow.uuid}>
-                                    <em>Default</em>
-                                </MenuItem>
-                                {presets.map(
-                                    ({ uuid, name }) => <MenuItem key={uuid} value={uuid}>{name}</MenuItem>
-                                )}
-                            </Select>
-                        </FormControl>
-                    </Grid>
-                    <WorkflowPresetSelectInfo />
-                </Grid>
-            </Grid>
-        );
-    }
+            const { selectedPreset, workflow, presets, classes } = this.props;
 
-    handleChange = ({ target }: React.ChangeEvent<HTMLSelectElement>) => {
+            return (
+                <div className={classes.root}>
+                    <FormControl fullWidth>
+                        <InputLabel htmlFor="age-simple">Preset</InputLabel>
+                        <Select
+                            value={selectedPreset.uuid}
+                            onChange={this.handleChange}>
+                            <MenuItem value={workflow.uuid}>
+                                <em>Default</em>
+                            </MenuItem>
+                            {presets.map(
+                                ({ uuid, name }) => <MenuItem key={uuid} value={uuid}>{name}</MenuItem>
+                            )}
+                        </Select>
+                    </FormControl>
+                    <Tooltip title='List of already defined set of inputs to run a workflow'>
+                        <DetailsIcon className={classes.icon} />
+                    </Tooltip>
+                </div >
+            );
+        }
 
-        const { workflow, presets, onChange } = this.props;
+        handleChange = ({ target }: React.ChangeEvent<HTMLSelectElement>) => {
 
-        const selectedPreset = [workflow, ...presets]
-            .find(({ uuid }) => uuid === target.value);
+            const { workflow, presets, onChange } = this.props;
 
-        if (selectedPreset) {
-            onChange(selectedPreset);
-        }
-    }
-}
+            const selectedPreset = [workflow, ...presets]
+                .find(({ uuid }) => uuid === target.value);
 
-const WorkflowPresetSelectInfo = withStyles<'icon'>(theme => ({
-    icon: {
-        marginTop: 18,
-        marginLeft: 8,
-    },
-}))(
-    ({ classes }: WithStyles<'icon'>) =>
-        <Tooltip title='List of already defined set of inputs to run a workflow'>
-            <DetailsIcon className={classes.icon} />
-        </Tooltip>
-);
+            if (selectedPreset) {
+                onChange(selectedPreset);
+            }
+        }
+    });
-- 
2.30.2