From 84ac24e173451dbdf90947915fdcd2f321420edf Mon Sep 17 00:00:00 2001
From: olaven <olav@sundfoer.com>
Date: Fri, 10 Mar 2023 13:57:50 +0100
Subject: [PATCH] Small UX improvements and basic mobile UI

- cards have full width on mobile
- input has full width on mobile
- copy-on-click for the feed URLs
---
 components/Button.tsx     | 12 ------------
 components/Search.tsx     |  8 ++++----
 components/SeriesCard.tsx | 19 +++++++++++++------
 fresh.gen.ts              |  5 ++++-
 islands/CopyButton.tsx    | 28 ++++++++++++++++++++++++++++
 routes/index.tsx          |  9 ++++++---
 6 files changed, 55 insertions(+), 26 deletions(-)
 delete mode 100644 components/Button.tsx
 create mode 100644 islands/CopyButton.tsx

diff --git a/components/Button.tsx b/components/Button.tsx
deleted file mode 100644
index 909d380..0000000
--- a/components/Button.tsx
+++ /dev/null
@@ -1,12 +0,0 @@
-import { JSX } from "preact";
-import { IS_BROWSER } from "$fresh/runtime.ts";
-
-export function Button(props: JSX.HTMLAttributes<HTMLButtonElement>) {
-  return (
-    <button
-      {...props}
-      disabled={!IS_BROWSER || props.disabled}
-      class="px-2 py-1 border(gray-100 2) hover:bg-gray-200"
-    />
-  );
-}
diff --git a/components/Search.tsx b/components/Search.tsx
index a6ce800..0fa606b 100644
--- a/components/Search.tsx
+++ b/components/Search.tsx
@@ -1,9 +1,9 @@
-export default function Search(props: {defaultValue: string}) {
+export default function Search(props: { defaultValue: string }) {
     return (
-        <form className="flex flex-col w-1/2 mx-auto my-4">
+        <form className="flex flex-col w-full lg:w-2/3 mx-auto my-4">
             <label className="sr-only" htmlFor="query">Program</label>
-            <input placeholder="NRK-podcast" className="border-2" name="query" id="query" defaultValue={props.defaultValue} />
-            <button type="submit" className="my-2 bg-gray-100">Søk</button>
+            <input placeholder="NRK-podcast" className="border-2 rounded-md px-2" name="query" id="query" defaultValue={props.defaultValue} />
+            <button type="submit" className="my-2 bg-gray-100 rounded-md">Søk</button>
         </form>
     );
 }
diff --git a/components/SeriesCard.tsx b/components/SeriesCard.tsx
index d74ce01..0d9701a 100644
--- a/components/SeriesCard.tsx
+++ b/components/SeriesCard.tsx
@@ -1,14 +1,21 @@
 import { Serie } from "../lib/nrk.ts"
+import CopyButton from "../islands/CopyButton.tsx";
 
 export function SerieCard(props: { serie: Serie, origin: string }) {
     const feedUrl = new URL(`/api/feeds/${props.serie.seriesId}`, props.origin);
 
+    console.log(feedUrl.toString())
+
     return (
-        <div className="border-2 my-2 mx-4">
-            <h2 className="text-4xl">{props.serie.title}</h2>
-            <p className="text-2xl">{props.serie.description}</p>
-            <img src={props.serie.images[0].uri} />
-            <p>Feed: {feedUrl.toString()}</p>
-        </div>
+        <div className="border-2 my-2 p-2">
+            <div className="mx-auto h-full w-full">
+                <h2 className="text-xl">{props.serie.title}</h2>
+                <p className="text-md">{props.serie.description}</p>
+                <img src={props.serie.images[0].uri} />
+                <CopyButton text={feedUrl.toString()}>
+                    Kopier URL
+                </CopyButton>
+            </div>
+        </div >
     );
 }
diff --git a/fresh.gen.ts b/fresh.gen.ts
index 756abf4..2b0a226 100644
--- a/fresh.gen.ts
+++ b/fresh.gen.ts
@@ -6,6 +6,7 @@ import config from "./deno.json" assert { type: "json" };
 import * as $0 from "./routes/api/feeds/[seriesId].ts";
 import * as $1 from "./routes/api/search.ts";
 import * as $2 from "./routes/index.tsx";
+import * as $$0 from "./islands/CopyButton.tsx";
 
 const manifest = {
   routes: {
@@ -13,7 +14,9 @@ const manifest = {
     "./routes/api/search.ts": $1,
     "./routes/index.tsx": $2,
   },
-  islands: {},
+  islands: {
+    "./islands/CopyButton.tsx": $$0,
+  },
   baseUrl: import.meta.url,
   config,
 };
diff --git a/islands/CopyButton.tsx b/islands/CopyButton.tsx
new file mode 100644
index 0000000..82a32da
--- /dev/null
+++ b/islands/CopyButton.tsx
@@ -0,0 +1,28 @@
+import Preact from "preact";
+import { useState } from "preact/hooks"
+import { IS_BROWSER } from "$fresh/runtime.ts";
+
+export default function CopyButton(props: { textBefore: string, textAfter: string } & Preact.PropsWithChildren) {
+
+    const [clicked, setClicked] = useState(false);
+
+    const startReset = () => {
+        setTimeout(() => {
+            setClicked(false);
+        }, 2_000);
+    }
+
+    return (
+        <button
+            onClick={() => {
+                navigator.clipboard.writeText(props.text.toString());
+                setClicked(true);
+                startReset();
+            }}
+            disabled={!IS_BROWSER || props.disabled}
+            class="px-2 py-1 border(gray-100 2) hover:bg-gray-200 mx-auto"
+        >
+            {clicked ? "Kopiert!" : props.children}
+        </button>
+    );
+}
diff --git a/routes/index.tsx b/routes/index.tsx
index 9e5c81c..4599bdc 100644
--- a/routes/index.tsx
+++ b/routes/index.tsx
@@ -20,7 +20,7 @@ export const handler: Handlers<HandlerData> = {
         }
         return ctx.render({ query: query || "", result, origin: url.origin });
     },
-  };
+};
 
 export default function Home({ data }: PageProps<HandlerData>) {
     return (
@@ -29,10 +29,13 @@ export default function Home({ data }: PageProps<HandlerData>) {
                 <title>NRSS</title>
             </Head>
             <div className="p-4 mx-auto max-w-screen-md bg-blue-400">
-                <h1 className="text-4xl text-center">NRSS - NRK-podcast som RSS</h1>
+                <div className="text-center">
+                    <h1 className="text-2xl lg:text-3xl">NRSS</h1>
+                    <h2 className="lg:text-xl">NRK-podcast som RSS</h2>
+                </div>
                 <Search defaultValue={data.query} />
                 {data.result ? (
-                    <div className="w-1/2 mx-auto my-4">   
+                    <div className="w-full mx-auto my-4">
                         {data.result.map(result => <SerieCard serie={result} origin={data.origin} />)}
                     </div>
                 ) : null}