19045: Improves padding on several places. 19045-runtime-status-details-fix
authorLucas Di Pentima <lucas.dipentima@curii.com>
Wed, 20 Apr 2022 14:29:21 +0000 (11:29 -0300)
committerLucas Di Pentima <lucas.dipentima@curii.com>
Wed, 20 Apr 2022 14:29:21 +0000 (11:29 -0300)
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas.dipentima@curii.com>

src/views-components/process-runtime-status/process-runtime-status.tsx
src/views/process-panel/process-details-card.tsx

index a2cd455c512fbcc62ebbda1b111450e9662df606..3858e49e2372be08549f81816327138ffb09c046 100644 (file)
@@ -17,7 +17,8 @@ import { RuntimeStatus } from "models/runtime-status";
 import { ArvadosTheme } from 'common/custom-theme';
 import classNames from 'classnames';
 
-type CssRules = 'heading'
+type CssRules = 'root'
+    | 'heading'
     | 'summary'
     | 'summaryText'
     | 'details'
@@ -28,6 +29,9 @@ type CssRules = 'heading'
     | 'warningColor';
 
 const styles: StyleRulesCallback<CssRules> = (theme: ArvadosTheme) => ({
+    root: {
+        marginBottom: theme.spacing.unit * 1,
+    },
     heading: {
         fontSize: '1rem',
     },
@@ -70,7 +74,7 @@ type ProcessRuntimeStatusProps = ProcessRuntimeStatusDataProps & WithStyles<CssR
 
 export const ProcessRuntimeStatus = withStyles(styles)(
     ({ runtimeStatus, classes }: ProcessRuntimeStatusProps) => {
-    return <>
+    return <div className={classes.root}>
         { runtimeStatus?.error &&
         <div data-cy='process-runtime-status-error'><ExpansionPanel className={classes.error} elevation={0}>
             <ExpansionPanelSummary className={classNames(classes.summary, classes.detailsText)} expandIcon={<ExpandMoreIcon />}>
@@ -99,5 +103,5 @@ export const ProcessRuntimeStatus = withStyles(styles)(
             </ExpansionPanelDetails>
         </ExpansionPanel></div>
         }
-    </>
+    </div>
 });
\ No newline at end of file
index 21563c16aa01ecdbb735d922b0a5f9feab46f99d..da6438a1b83fab34d76e096b8a18a93ddfc0d274 100644 (file)
@@ -41,8 +41,10 @@ const styles: StyleRulesCallback<CssRules> = (theme: ArvadosTheme) => ({
         paddingTop: theme.spacing.unit * 0.5
     },
     content: {
+        padding: theme.spacing.unit * 1.0,
+        paddingTop: theme.spacing.unit * 0.5,
         '&:last-child': {
-            paddingBottom: theme.spacing.unit * 2,
+            paddingBottom: theme.spacing.unit * 1,
         }
     },
     title: {