From c1f1cb94fd335080082f541244ea92caee677491 Mon Sep 17 00:00:00 2001
From: Robin <robin@robin.town>
Date: Mon, 18 Sep 2023 16:47:15 -0400
Subject: [PATCH] Fix home page overflowing by a few pixels

Because the height of our header component changed at some point, the hard-coded height values in the CSS were off by a few px and caused the page to overflow slightly.
---
 src/home/RegisteredView.tsx      | 16 ++++++++--------
 src/home/UnauthenticatedView.tsx | 16 ++++++++--------
 src/home/common.module.css       |  6 +-----
 3 files changed, 17 insertions(+), 21 deletions(-)

diff --git a/src/home/RegisteredView.tsx b/src/home/RegisteredView.tsx
index 921ee0da..4e7b151c 100644
--- a/src/home/RegisteredView.tsx
+++ b/src/home/RegisteredView.tsx
@@ -114,15 +114,15 @@ export function RegisteredView({ client }: Props) {
 
   return (
     <>
-      <Header>
-        <LeftNav>
-          <HeaderLogo />
-        </LeftNav>
-        <RightNav>
-          <UserMenuContainer />
-        </RightNav>
-      </Header>
       <div className={commonStyles.container}>
+        <Header>
+          <LeftNav>
+            <HeaderLogo />
+          </LeftNav>
+          <RightNav>
+            <UserMenuContainer />
+          </RightNav>
+        </Header>
         <main className={commonStyles.main}>
           <HeaderLogo className={commonStyles.logo} />
           <Heading size="lg" weight="semibold">
diff --git a/src/home/UnauthenticatedView.tsx b/src/home/UnauthenticatedView.tsx
index 82839ff7..e001c5a8 100644
--- a/src/home/UnauthenticatedView.tsx
+++ b/src/home/UnauthenticatedView.tsx
@@ -148,15 +148,15 @@ export const UnauthenticatedView: FC = () => {
 
   return (
     <>
-      <Header>
-        <LeftNav>
-          <HeaderLogo />
-        </LeftNav>
-        <RightNav hideMobile>
-          <UserMenuContainer />
-        </RightNav>
-      </Header>
       <div className={commonStyles.container}>
+        <Header>
+          <LeftNav>
+            <HeaderLogo />
+          </LeftNav>
+          <RightNav hideMobile>
+            <UserMenuContainer />
+          </RightNav>
+        </Header>
         <main className={commonStyles.main}>
           <HeaderLogo className={commonStyles.logo} />
           <Heading size="lg" weight="semibold">
diff --git a/src/home/common.module.css b/src/home/common.module.css
index 5a7cf350..a3e84eee 100644
--- a/src/home/common.module.css
+++ b/src/home/common.module.css
@@ -16,7 +16,7 @@ limitations under the License.
 
 .container {
   display: flex;
-  min-height: calc(100% - 64px);
+  min-height: 100%;
   flex-direction: column;
   justify-content: space-between;
 }
@@ -42,8 +42,4 @@ limitations under the License.
   .logo {
     display: none;
   }
-
-  .container {
-    min-height: calc(100% - 76px);
-  }
 }