Skip to content

Commit 9d02a37

Browse files
authored
Condense footer (#2555)
Since the footer is included in every page, we should try extra hard to use little vertical space. Also, I add an icon to the version text for balance.
1 parent 06ea7c8 commit 9d02a37

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

site/src/components/Footer/Footer.tsx

+8-5
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import Link from "@material-ui/core/Link"
22
import { makeStyles } from "@material-ui/core/styles"
3+
import AccountTreeIcon from "@material-ui/icons/AccountTree"
34
import AssistantIcon from "@material-ui/icons/Assistant"
45
import * as TypesGen from "../../api/typesGenerated"
56

@@ -22,14 +23,15 @@ export const Footer: React.FC<FooterProps> = ({ buildInfo }) => {
2223

2324
return (
2425
<div className={styles.root}>
25-
<Link className={styles.link} variant="caption" target="_blank" href={githubUrl}>
26-
<AssistantIcon className={styles.assistantIcon} /> {Language.reportBugLink}
27-
</Link>
2826
<div className={styles.copyRight}>{Language.copyrightText}</div>
2927
{buildInfo && (
3028
<div className={styles.buildInfo}>
3129
<Link className={styles.link} variant="caption" target="_blank" href={buildInfo.external_url}>
32-
{Language.buildInfoText(buildInfo)}
30+
<AccountTreeIcon className={styles.icon} /> {Language.buildInfoText(buildInfo)}
31+
</Link>
32+
&nbsp;|&nbsp;
33+
<Link className={styles.link} variant="caption" target="_blank" href={githubUrl}>
34+
<AssistantIcon className={styles.icon} /> {Language.reportBugLink}
3335
</Link>
3436
</div>
3537
)}
@@ -51,6 +53,7 @@ const useFooterStyles = makeStyles((theme) => ({
5153
},
5254
buildInfo: {
5355
margin: theme.spacing(0.25),
56+
display: "inline-flex",
5457
},
5558
link: {
5659
color: theme.palette.text.secondary,
@@ -59,7 +62,7 @@ const useFooterStyles = makeStyles((theme) => ({
5962
alignItems: "center",
6063
justifyContent: "center",
6164
},
62-
assistantIcon: {
65+
icon: {
6366
fontSize: 12,
6467
color: theme.palette.secondary.dark,
6568
marginRight: theme.spacing(0.5),

0 commit comments

Comments
 (0)