Skip to content

Commit e0407d4

Browse files
committed
#417 align network items list in control panel
1 parent d2bce44 commit e0407d4

2 files changed

Lines changed: 37 additions & 22 deletions

File tree

webapp/components/general/ControlPanelTreeItem.js

Lines changed: 24 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,14 @@
11
import * as React from 'react';
22
import { makeStyles } from '@material-ui/core/styles';
33
import Box from '@material-ui/core/Box';
4+
import Grid from '@material-ui/core/Grid';
45
import Typography from '@material-ui/core/Typography';
56
import IconButton from '@material-ui/core/IconButton';
67
import TreeItem from '@material-ui/lab/TreeItem';
78
import Visibility from '@material-ui/icons/Visibility';
89
import ColorLens from '@material-ui/icons/ColorLens';
910
import { ChromePicker } from 'react-color';
10-
import { bgInputs } from '../../theme';
11+
import { experimentLabelColor } from '../../theme';
1112

1213
const useStyles = makeStyles((theme) => ({
1314
networkItem: {
@@ -18,7 +19,7 @@ const useStyles = makeStyles((theme) => ({
1819
'& .MuiIconButton-root': {
1920
padding: 0,
2021
marginLeft: '0.5rem',
21-
color: bgInputs,
22+
color: experimentLabelColor,
2223
'&:hover': {
2324
color: 'white',
2425
},
@@ -39,7 +40,6 @@ const ControlPanelTreeItem = (props) => {
3940

4041
const handleColorSelection = (color) => {
4142
setColor(color.hex);
42-
console.log('color changed', color.hex);
4343
};
4444

4545
const {
@@ -56,37 +56,40 @@ const ControlPanelTreeItem = (props) => {
5656
<TreeItem
5757
nodeId={nodeId}
5858
label={(
59-
<Box
59+
<Grid
60+
container
6061
className={classes.networkItem}
6162
onMouseEnter={() => setIsHoveredOver(true)}
6263
onMouseLeave={() => setIsHoveredOver(false)}
6364
display="flex"
6465
flexDirection="row"
6566
justifyContent="space-between"
6667
>
67-
<Typography onClick={(event) => onNodeSelect(event, nodeId)}>{label}</Typography>
68-
<Typography>{type}</Typography>
69-
{
70-
isHoveredOver
68+
<Grid item xs={4}><Typography onClick={(event) => onNodeSelect(event, nodeId)}>{label}</Typography></Grid>
69+
<Grid item xs={4} justifyContent="center"><Typography>{type}</Typography></Grid>
70+
<Grid item xs={4} justifyContent="flex-end" className={classes.controls}>
71+
{isHoveredOver
7172
? (
72-
<Box className={classes.controls}>
73+
<>
74+
7375
<IconButton onClick={(event) => onVisibilityClick(event, nodeId)}><Visibility /></IconButton>
7476
<IconButton onClick={() => setShowColorPicker(!showColorPicker)}><ColorLens /></IconButton>
7577
{
76-
showColorPicker
77-
? (
78-
<ChromePicker
79-
className={classes.colorPicker}
80-
color={color}
81-
onChangeComplete={handleColorSelection}
82-
/>
83-
) : null
78+
showColorPicker
79+
? (
80+
<ChromePicker
81+
className={classes.colorPicker}
82+
color={color}
83+
onChangeComplete={handleColorSelection}
84+
/>
85+
) : null
8486
}
85-
</Box>
87+
88+
</>
8689
)
87-
: null
88-
}
89-
</Box>
90+
: null}
91+
</Grid>
92+
</Grid>
9093
)}
9194
>
9295
{children}

webapp/components/general/ExperimentControlPanel.js

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,21 @@ import TreeItem from '@material-ui/lab/TreeItem';
88
import ExpandMoreIcon from '@material-ui/icons/ExpandMore';
99
import ChevronRightIcon from '@material-ui/icons/ChevronRight';
1010
import ControlPanelTreeItem from './ControlPanelTreeItem';
11+
import { experimentLabelColor } from '../../theme';
1112

1213
import { MODEL_STATE } from '../../constants';
1314

15+
const useStyles = makeStyles(() => ({
16+
header: {
17+
'& .MuiTypography-root': {
18+
color: experimentLabelColor,
19+
fontWeight: 'bold',
20+
},
21+
},
22+
}));
23+
1424
const ExperimentControlPanel = (props) => {
25+
const classes = useStyles();
1526
const [filter, setFilter] = React.useState('');
1627
const onNodeSelect = (event, nodeId) => {
1728
console.log(`Node with id ${nodeId} clicked`);
@@ -50,9 +61,10 @@ const ExperimentControlPanel = (props) => {
5061
? (
5162
<Box display="flex" flexDirection="column" p={1}>
5263
<TextField label="Filter results" variant="outlined" fullWidth onChange={(e) => setFilter(e.target.value)} />
53-
<Box display="flex" justifyContent="space-between" mt={1}>
64+
<Box className={classes.header} display="flex" justifyContent="space-between" mt={1}>
5465
<Typography>Name</Typography>
5566
<Typography>Type(s)</Typography>
67+
<Typography />
5668
</Box>
5769
<TreeView
5870
aria-label="Network data navigator"

0 commit comments

Comments
 (0)