From 4dd85113215412891790090e20a07a6dc58171c7 Mon Sep 17 00:00:00 2001
From: Aarni Koskela <akx@iki.fi>
Date: Thu, 30 May 2024 17:59:29 +0300
Subject: [PATCH] Gently tweak new layout (#105)

* fix: give top nav links a bit more hit area and breathing room

* fix: disable inadvertent selection on COC checkbox
---
 components/Form.tsx |  2 +-
 components/Nav.tsx  | 35 ++++++++++-------------------------
 2 files changed, 11 insertions(+), 26 deletions(-)

diff --git a/components/Form.tsx b/components/Form.tsx
index 287d29b..a3f2f9c 100644
--- a/components/Form.tsx
+++ b/components/Form.tsx
@@ -85,7 +85,7 @@ export default function Form() {
             </button>
           </div>
 
-          <label className="flex flex-wrap items-center justify-center gap-2 font-mono text-xxs sm:text-xs">
+          <label className="flex select-none flex-wrap items-center justify-center gap-2 font-mono text-xxs sm:text-xs">
             <div className="relative h-5 w-5">
               <input
                 type="checkbox"
diff --git a/components/Nav.tsx b/components/Nav.tsx
index 622c4cf..c52eff8 100644
--- a/components/Nav.tsx
+++ b/components/Nav.tsx
@@ -1,7 +1,7 @@
 'use client';
 
 import Image from 'next/image';
-import Link from 'next/link';
+import Link, { LinkProps } from 'next/link';
 import Wrapper from './Wrapper';
 import { useEffect, useRef, useState } from 'react';
 
@@ -47,30 +47,13 @@ export default function Nav() {
               </svg>
             </button>
             <div
-              className={`${navOpen ? 'flex' : 'hidden'} text-shadow absolute right-6 top-16 flex-col gap-5 rounded-lg bg-black/80 p-5 text-xs uppercase tracking-widest text-pink-100 backdrop-blur-md lg:static lg:flex lg:flex-row lg:bg-transparent lg:backdrop-blur-none`}
+              className={`${navOpen ? 'flex' : 'hidden'} text-shadow absolute right-10 top-16 flex-col divide-white/20 rounded-lg bg-black/80 p-5 text-xs uppercase tracking-widest text-pink-100 backdrop-blur-md lg:static lg:flex lg:flex-row lg:divide-x lg:bg-transparent lg:backdrop-blur-none`}
             >
-              <Link className="underline-offset-4 hover:underline" href="https://github.com/koodiklinikka">
-                GitHub
-              </Link>
-              <NavSeparator />
-              <Link className="underline-offset-4 hover:underline" href="https://koodiklinikka.slack.com">
-                Slack
-              </Link>
-              <NavSeparator />
-              <Link className="underline-offset-4 hover:underline" href="https://resources.koodiklinikka.fi">
-                Resources
-              </Link>
-              <NavSeparator />
-              <Link className="underline-offset-4 hover:underline" href="https://koodiklinikka.myspreadshop.fi/">
-                Shop
-              </Link>
-              <NavSeparator />
-              <Link
-                className="underline-offset-4 hover:underline"
-                href="https://github.com/koodiklinikka/code-of-conduct"
-              >
-                Code of Conduct
-              </Link>
+              <NavLink href="https://github.com/koodiklinikka">GitHub</NavLink>
+              <NavLink href="https://koodiklinikka.slack.com">Slack</NavLink>
+              <NavLink href="https://resources.koodiklinikka.fi">Resources</NavLink>
+              <NavLink href="https://koodiklinikka.myspreadshop.fi/">Shop</NavLink>
+              <NavLink href="https://github.com/koodiklinikka/code-of-conduct">Code of Conduct</NavLink>
             </div>
           </div>
         </div>
@@ -79,4 +62,6 @@ export default function Nav() {
   );
 }
 
-const NavSeparator = () => <div className="hidden opacity-20 lg:block">|</div>;
+const NavLink = (props: React.PropsWithChildren<LinkProps>) => (
+  <Link className="py-1 underline-offset-4 hover:underline lg:px-6" {...props} />
+);