From 6555487b2effbef5f509e19d5a886f5d9b1239ed Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jonas=20B=C3=A4ck?= <jonas.c.back@gmail.com>
Date: Mon, 4 Sep 2023 13:15:21 +0300
Subject: [PATCH 1/3] Update table of contents styling

---
 components/TableOfContents.tsx | 29 +++++++++++++++++------------
 1 file changed, 17 insertions(+), 12 deletions(-)

diff --git a/components/TableOfContents.tsx b/components/TableOfContents.tsx
index b75a895..f542491 100644
--- a/components/TableOfContents.tsx
+++ b/components/TableOfContents.tsx
@@ -4,18 +4,23 @@ import { titleToAnchor } from '@utils/helpers'
 
 const TableOfContents = ({ sections }: { sections: Section[] }) => {
   return (
-    <ul>
-      {sections.map((section, i) => (
-        <li key={i}>
-          <Link
-            className="no-underline hover:underline"
-            href={`#${titleToAnchor(section.attributes.title)}`}
-          >
-            {section.attributes.title}
-          </Link>
-        </li>
-      ))}
-    </ul>
+    <>
+      <h2>Innehållsförteckning</h2>
+      <table className="table-auto border-separate text-sm">
+        <tbody>
+          {sections.map((section, i) => (
+            <tr key={i}>
+              <Link
+                className="no-underline hover:underline"
+                href={`#${titleToAnchor(section.attributes.title)}`}
+              >
+                {section.attributes.title}
+              </Link>
+            </tr>
+          ))}
+        </tbody>
+      </table>
+    </>
   )
 }
 

From 2395d072719feff6eb8d9f9b4b8c0b330f4cea72 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jonas=20B=C3=A4ck?= <jonas.c.back@gmail.com>
Date: Mon, 4 Sep 2023 13:40:39 +0300
Subject: [PATCH 2/3] Improve content page anchor links

Trim anchor tags, add scroll margin to section headers
---
 components/PageSection.tsx | 4 +++-
 utils/helpers.ts           | 1 +
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/components/PageSection.tsx b/components/PageSection.tsx
index 0fe1485..74ab747 100644
--- a/components/PageSection.tsx
+++ b/components/PageSection.tsx
@@ -18,7 +18,9 @@ const PageSection = ({
   isPrivate,
 }: PageSectionProps) => (
   <div>
-    <h2 id={titleToAnchor(title)}>{title}</h2>
+    <h2 id={titleToAnchor(title)} className="scroll-mt-24">
+      {title}
+    </h2>
     <div
       dangerouslySetInnerHTML={{
         __html: marked.parse(content ?? ''),
diff --git a/utils/helpers.ts b/utils/helpers.ts
index 6a5fd30..b8b2c01 100644
--- a/utils/helpers.ts
+++ b/utils/helpers.ts
@@ -35,6 +35,7 @@ export const getLayoutProps = async (): Promise<LayoutProps> => {
 
 export const titleToAnchor = (title: string) => {
   return title
+    .trim()
     .replace(/ /g, '-')
     .replace(/[\/\\^$*+?.()|\[\]{}<>:;"'~,=@`#!%&]/g, '')
     .toLowerCase()

From 0d4e341e0dd82aa7ddb6baf779566895afc7b5aa Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jonas=20B=C3=A4ck?= <jonas.c.back@gmail.com>
Date: Mon, 4 Sep 2023 14:03:23 +0300
Subject: [PATCH 3/3] Show page instead of redirecting unauthorized users

---
 components/Page.tsx            | 54 +++++++++++++++++++++-------------
 components/Unauthorized.tsx    | 18 ++++++++++++
 pages/medlem/[privatePage].tsx | 14 ++-------
 3 files changed, 55 insertions(+), 31 deletions(-)
 create mode 100644 components/Unauthorized.tsx

diff --git a/components/Page.tsx b/components/Page.tsx
index 91602fc..de548a5 100644
--- a/components/Page.tsx
+++ b/components/Page.tsx
@@ -7,6 +7,7 @@ import { NationLogo } from '@components/footer/Logos'
 import Footer from '@components/footer'
 import TableOfContents from '@components/TableOfContents'
 import PageSection from '@components/PageSection'
+import Unauthorized from '@components/Unauthorized'
 
 const renderer: marked.RendererObject = {
   image(href: string | null): string {
@@ -24,34 +25,47 @@ type PageProps = {
   logos: NationLogo[]
   navbarLinks: NavbarLink[]
   isPrivate: boolean
+  unauthorized?: boolean
 }
 
-const Page: NextPage<PageProps> = ({ page, navbarLinks, logos, isPrivate }) => {
+const Page: NextPage<PageProps> = ({
+  page,
+  navbarLinks,
+  logos,
+  isPrivate,
+  unauthorized = false,
+}) => {
   return (
     <div className="flex min-h-screen flex-col bg-white">
       <Header navbarLinks={navbarLinks} />
       <div className="flex flex-grow justify-center">
         <div className="prose prose-sm mx-4 mb-12 mt-6 flex flex-col sm:mx-8 md:mx-16 md:mt-12">
-          <h1>{page?.title}</h1>
-          {page?.content && (
-            <div
-              dangerouslySetInnerHTML={{
-                __html: marked.parse(page.content ?? ''),
-              }}
-            />
+          {unauthorized ? (
+            <Unauthorized />
+          ) : (
+            <>
+              <h1>{page?.title}</h1>
+              {page?.content && (
+                <div
+                  dangerouslySetInnerHTML={{
+                    __html: marked.parse(page.content ?? ''),
+                  }}
+                />
+              )}
+              {page?.showTableOfContents && (
+                <TableOfContents sections={page.sections.data} />
+              )}
+              {page?.sections?.data.map((section, i) => (
+                <PageSection
+                  key={i}
+                  title={section.attributes.title}
+                  content={section.attributes.content}
+                  file_folders={section.attributes.file_folders.data}
+                  isPrivate={isPrivate}
+                />
+              ))}
+            </>
           )}
-          {page?.showTableOfContents && (
-            <TableOfContents sections={page.sections.data} />
-          )}
-          {page?.sections?.data.map((section, i) => (
-            <PageSection
-              key={i}
-              title={section.attributes.title}
-              content={section.attributes.content}
-              file_folders={section.attributes.file_folders.data}
-              isPrivate={isPrivate}
-            />
-          ))}
         </div>
       </div>
       <Footer logos={logos} />
diff --git a/components/Unauthorized.tsx b/components/Unauthorized.tsx
new file mode 100644
index 0000000..28efcb0
--- /dev/null
+++ b/components/Unauthorized.tsx
@@ -0,0 +1,18 @@
+import { signIn } from 'next-auth/react'
+
+const Unauthorized = () => (
+  <>
+    <p>Denna sida kräver inloggning</p>
+    <button
+      onClick={(e) => {
+        e.preventDefault()
+        signIn('keycloak')
+      }}
+      className="mx-3 rounded-lg border p-2 hover:font-bold"
+    >
+      Logga in
+    </button>
+  </>
+)
+
+export default Unauthorized
diff --git a/pages/medlem/[privatePage].tsx b/pages/medlem/[privatePage].tsx
index db73dd5..a6bda28 100644
--- a/pages/medlem/[privatePage].tsx
+++ b/pages/medlem/[privatePage].tsx
@@ -1,7 +1,5 @@
 import { GetServerSideProps, NextPage } from 'next'
 import { PageType } from '@models/page'
-import { signIn } from 'next-auth/react'
-import { useEffect } from 'react'
 import { Session } from 'next-auth'
 import { getSession } from 'next-auth/react'
 import { fetchPrivatePage } from '@lib/api/privatepage'
@@ -20,15 +18,9 @@ type PrivatePageProps = {
 const PrivatePage: NextPage<PrivatePageProps> = ({
   session,
   ...props
-}: PrivatePageProps) => {
-  useEffect(() => {
-    if (!session) {
-      signIn('keycloak')
-    }
-  }, [session])
-
-  return <Page {...props} isPrivate={true} />
-}
+}: PrivatePageProps) => (
+  <Page {...props} isPrivate={true} unauthorized={!session || !props.page} />
+)
 
 export const getServerSideProps: GetServerSideProps<{
   session: Session | null