Merge pull request #1548 from thehyve/ui-improvements

Add improvements following the UI 2.0 feedback document.
thehyve · Aug 21, 2024 · 952178a · 952178a
2 parents cda0534 + eaba481
commit 952178a
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 15 deletions.
diff --git a/projects/mercury/src/dashboard/DashboardPage.js b/projects/mercury/src/dashboard/DashboardPage.js
@@ -85,7 +85,7 @@ const DashboardPage = props => {
                             <Typography className={classes.header} variant="h4" paragraph align="center">
                                 About
                             </Typography>
-                            <Typography variant="body1" paragraph>
+                            <Typography variant="body1" paragraph align="center">
                                 {APPLICATION_NAME} version: {process.env.REACT_APP_VERSION}
                             </Typography>
                             <Typography variant="body2" paragraph className={classes.footer} align="center">

diff --git a/projects/mercury/src/dashboard/DomainInfo.styles.js b/projects/mercury/src/dashboard/DomainInfo.styles.js
@@ -14,7 +14,8 @@ const styles = theme => ({
     },
     icon: {
         marginLeft: 10,
-        marginTop: 10
+        marginTop: 10,
+        color: theme.palette.primary.contrastText
     },
     imageIconRoot: {
         marginTop: 10,

diff --git a/projects/mercury/src/file/FileVersionsList.js b/projects/mercury/src/file/FileVersionsList.js
@@ -176,12 +176,7 @@ const FileVersionsList = ({selectedFile, onRevertVersion, isWritingEnabled, clas
     );
 
     const renderHeader = ({label}) => (
-        <TableCell
-            component="div"
-            className={`${classes.tableCell} ${classes.tableHeaderRow}`}
-            variant="head"
-            align="left"
-        >
+        <TableCell component="div" className={classes.tableCell} variant="head" align="left">
             {label}
         </TableCell>
     );

diff --git a/projects/mercury/src/layout/Layout.styles.js b/projects/mercury/src/layout/Layout.styles.js
@@ -3,7 +3,7 @@ const styles = theme => ({
         backgroundColor: theme.palette.background.default,
         paddingTop: 14,
         paddingBottom: 14,
-        paddingRight: 0,
+        paddingRight: 8,
         minWidth: 0, // So the Typography noWrap works
         overflowY: 'scroll',
         overflowX: 'hidden',

diff --git a/projects/mercury/src/metadata/views/MetadataViewTable.js b/projects/mercury/src/metadata/views/MetadataViewTable.js
@@ -31,11 +31,6 @@ type MetadataViewTableProperties = {
 };
 
 const useStyles = makeStyles(() => ({
-    headerRow: {
-        '& .MuiTableCell-root': {
-            borderRadius: 20
-        }
-    },
     headerCellContents: {
         verticalAlign: 'top',
         whiteSpace: 'nowrap'

diff --git a/projects/mercury/src/metadata/views/MetadataViewTableContainer.js b/projects/mercury/src/metadata/views/MetadataViewTableContainer.js
@@ -80,7 +80,8 @@ const styles = theme => ({
         overflowX: 'auto',
         '& .MuiTableCell-stickyHeader': {
             backgroundColor: theme.palette.background.default
-        }
+        },
+        borderRadius: theme.shape.borderRadius
     },
     tableFooter: {
         flex: 1