Added top margin to nav without session (#5655)

keystonejs · May 10, 2021 · fc9c3d5 · fc9c3d5 · vercel · May 10, 2021
1 parent ffd3543
commit fc9c3d5
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/.changeset/mighty-impalas-appear.md b/.changeset/mighty-impalas-appear.md
@@ -0,0 +1,5 @@
+---
+"@keystone-next/admin-ui": patch
+---
+
+Fixed navigation margin when session isn't configured
diff --git a/packages-next/admin-ui/src/components/Navigation.tsx b/packages-next/admin-ui/src/components/Navigation.tsx
@@ -65,7 +65,8 @@ const AuthenticatedItem = ({ item }: { item: { id: string; label: string } }) =>
         display: 'flex',
         flexDirection: 'row',
         justifyContent: 'space-between',
-        padding: spacing.xlarge,
+        margin: spacing.xlarge,
+        marginBottom: 0,
       }}
     >
       <div css={{ fontSize: typography.fontSize.small }}>
@@ -127,6 +128,7 @@ export const Navigation = () => {
     authenticatedItem,
     visibleLists,
   } = useKeystone();
+  const { spacing } = useTheme();
 
   return (
     <div
@@ -139,7 +141,7 @@ export const Navigation = () => {
       {authenticatedItem.state === 'authenticated' && (
         <AuthenticatedItem item={authenticatedItem} />
       )}
-      <nav>
+      <nav css={{ marginTop: spacing.xlarge }}>
         <NavItem href="/">Dashboard</NavItem>
         {(() => {
           if (visibleLists.state === 'loading') return null;