May 20, 2022 · May 20, 2022 · May 20, 2022 · May 20, 2022 · May 20, 2022 · May 20, 2022
diff --git a/site/src/components/Footer/Footer.tsx b/site/src/components/Footer/Footer.tsx
    flex: "0",
    paddingTop: theme.spacing(2),
    paddingBottom: theme.spacing(2),
    marginTop: theme.spacing(3),
  },
  copyRight: {
    margin: theme.spacing(0.25),
diff --git a/site/src/pages/LoginPage/LoginPage.tsx b/site/src/pages/LoginPage/LoginPage.tsx
  layout: {
    display: "flex",
    flexDirection: "column",
    gap: theme.spacing(2),
    alignItems: "center",
  },
  container: {
diff --git a/site/src/pages/TemplatesPage/TemplatesPageView.tsx b/site/src/pages/TemplatesPage/TemplatesPageView.tsx
              </TableRow>
            )}
            {props.templates?.map((template) => (
              <TableRow key={template.id} className={styles.templateRow}>
              <TableRow key={template.id}>
                <TableCell>
                  <Box alignItems="center" display="flex">
                    <Avatar variant="square" className={styles.templateAvatar}>
      lineHeight: `${theme.spacing(3)}px`,
    },
  },
  templateRow: {
    "& > td": {
      paddingTop: theme.spacing(2),
      paddingBottom: theme.spacing(2),
    },
  },
  templateAvatar: {
    borderRadius: 2,
    marginRight: theme.spacing(1),
diff --git a/site/src/pages/WorkspaceBuildPage/WorkspaceBuildPage.tsx b/site/src/pages/WorkspaceBuildPage/WorkspaceBuildPage.tsx

 const useStyles = makeStyles((theme) => ({
  title: {
    marginTop: theme.spacing(5),
    paddingTop: theme.spacing(5),
    paddingBottom: theme.spacing(2),
  },
 }))
diff --git a/site/src/pages/WorkspacesPage/WorkspacesPageView.tsx b/site/src/pages/WorkspacesPage/WorkspacesPageView.tsx
            {props.workspaces?.map((workspace) => {
              const status = getDisplayStatus(theme, workspace.latest_build)
              return (
                <TableRow key={workspace.id} className={styles.workspaceRow}>
                <TableRow key={workspace.id}>
                  <TableCell>
                    <div className={styles.workspaceName}>
                      <Avatar variant="square" className={styles.workspaceAvatar}>
      lineHeight: `${theme.spacing(3)}px`,
    },
  },
  workspaceRow: {
    "& > td": {
      paddingTop: theme.spacing(2),
      paddingBottom: theme.spacing(2),
    },
  },
  workspaceAvatar: {
    borderRadius: 2,
    marginRight: theme.spacing(1),
diff --git a/site/src/theme/overrides.ts b/site/src/theme/overrides.ts
        // Gives the appearance of a border!
        borderRadius: 2,
        border: `1px solid ${palette.divider}`,

        "& td": {
          paddingTop: 16,
          paddingBottom: 16,
        },
      },
    },
    MuiTableCell: {
Original file line number	Diff line number	Diff line change
Expand Up		@@ -42,6 +42,7 @@ const useFooterStyles = makeStyles((theme) => ({
		flex: "0",
		paddingTop: theme.spacing(2),
		paddingBottom: theme.spacing(2),
		marginTop: theme.spacing(3),
Copy link CollaboratorAuthor BrunoQuaresmaMay 20, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. The footer was very close to the content. Added some space.
		},
		copyRight: {
		margin: theme.spacing(0.25),
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -17,7 +17,6 @@ export const useStyles = makeStyles((theme) => ({
		layout: {
		display: "flex",
		flexDirection: "column",
		gap: theme.spacing(2),
		alignItems: "center",
		},
		container: {
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -77,7 +77,7 @@ export const TemplatesPageView: React.FC<TemplatesPageViewProps> = (props) => {
		</TableRow>
		)}
		{props.templates?.map((template) => (
		<TableRow key={template.id} className={styles.templateRow}>
		<TableRow key={template.id}>
		<TableCell>
		<Box alignItems="center" display="flex">
		<Avatar variant="square" className={styles.templateAvatar}>
Expand DownExpand Up		@@ -127,12 +127,6 @@ const useStyles = makeStyles((theme) => ({
		lineHeight: `${theme.spacing(3)}px`,
		},
		},
		templateRow: {
		"& > td": {
		paddingTop: theme.spacing(2),
		paddingBottom: theme.spacing(2),
		},
		},
		templateAvatar: {
		borderRadius: 2,
		marginRight: theme.spacing(1),
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -48,6 +48,7 @@ export const WorkspaceBuildPage: React.FC = () => {

		const useStyles = makeStyles((theme) => ({
		title: {
		marginTop: theme.spacing(5),
		paddingTop: theme.spacing(5),
		paddingBottom: theme.spacing(2),
Copy link CollaboratorAuthor BrunoQuaresmaMay 20, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Add more spacing to the bottom of the Logs title.
		},
		}))
Original file line number	Diff line number	Diff line change
Expand Up		@@ -71,7 +71,7 @@ export const WorkspacesPageView: React.FC<WorkspacesPageViewProps> = (props) =>
		{props.workspaces?.map((workspace) => {
		const status = getDisplayStatus(theme, workspace.latest_build)
		return (
		<TableRow key={workspace.id} className={styles.workspaceRow}>
		<TableRow key={workspace.id}>
		<TableCell>
		<div className={styles.workspaceName}>
		<Avatar variant="square" className={styles.workspaceAvatar}>
Expand DownExpand Up		@@ -134,12 +134,6 @@ const useStyles = makeStyles((theme) => ({
		lineHeight: `${theme.spacing(3)}px`,
		},
		},
		workspaceRow: {
		"& > td": {
		paddingTop: theme.spacing(2),
		paddingBottom: theme.spacing(2),
		},
		},
		workspaceAvatar: {
		borderRadius: 2,
		marginRight: theme.spacing(1),
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -44,6 +44,11 @@ export const getOverrides = (palette: PaletteOptions) => {
		// Gives the appearance of a border!
		borderRadius: 2,
		border: `1px solid ${palette.divider}`,

		"& td": {
		paddingTop: 16,
		paddingBottom: 16,
		},
Copy link CollaboratorAuthor BrunoQuaresmaMay 20, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Other tables were using that "manually" so I extracted them to be the standard.
		},
		},
		MuiTableCell: {
Expand Down