From f3c52237fa6fdd5a137250144894995306b3df98 Mon Sep 17 00:00:00 2001
From: reya <reya@lume.nu>
Date: Wed, 17 Apr 2024 13:50:50 +0700
Subject: [PATCH] fix: privacy setting is not working

---
 apps/desktop2/src/components/repost.tsx | 85 +++++++++----------------
 packages/ui/src/note/content.tsx        |  6 +-
 2 files changed, 33 insertions(+), 58 deletions(-)

diff --git a/apps/desktop2/src/components/repost.tsx b/apps/desktop2/src/components/repost.tsx
index 5ee96d5a..a6bd28ca 100644
--- a/apps/desktop2/src/components/repost.tsx
+++ b/apps/desktop2/src/components/repost.tsx
@@ -3,7 +3,7 @@ import { Event } from "@lume/types";
 import { cn } from "@lume/utils";
 import { useQuery } from "@tanstack/react-query";
 import { useTranslation } from "react-i18next";
-import { Note, User } from "@lume/ui";
+import { Note, Spinner, User } from "@lume/ui";
 import { useRouteContext } from "@tanstack/react-router";
 
 export function RepostNote({
@@ -27,51 +27,16 @@ export function RepostNote({
           const embed: Event = JSON.parse(event.content);
           return embed;
         }
-        const id = event.tags.find((el) => el[0] === "e")[1];
-        return await ark.get_event(id);
-      } catch {
-        throw new Error("Failed to get repost event");
+        const id = event.tags.find((el) => el[0] === "e")?.[1];
+        if (id) return await ark.get_event(id);
+      } catch (e) {
+        throw new Error(e);
       }
     },
     refetchOnWindowFocus: false,
     refetchOnMount: false,
   });
 
-  if (isLoading) {
-    return <div className="w-full px-3 pb-3">Loading...</div>;
-  }
-
-  if (isError || !repostEvent) {
-    return (
-      <Note.Root
-        className={cn(
-          "flex flex-col gap-2 border-b border-neutral-100 px-3 py-5 dark:border-neutral-900",
-          className,
-        )}
-      >
-        <User.Provider pubkey={event.pubkey}>
-          <User.Root className="flex h-14 gap-2 px-3">
-            <div className="inline-flex w-10 shrink-0 items-center justify-center">
-              <RepostIcon className="h-5 w-5 text-blue-500" />
-            </div>
-            <div className="inline-flex items-center gap-2">
-              <User.Avatar className="size-6 shrink-0 rounded object-cover" />
-              <div className="inline-flex items-baseline gap-1">
-                <User.Name className="font-medium text-neutral-900 dark:text-neutral-100" />
-                <span className="text-blue-500">{t("note.reposted")}</span>
-              </div>
-            </div>
-          </User.Root>
-        </User.Provider>
-        <div className="mb-3 select-text px-3">
-          <div className="flex flex-col items-start justify-start rounded-lg bg-red-100 px-3 py-3 dark:bg-red-900">
-            <p className="text-red-500">Failed to get event</p>
-          </div>
-        </div>
-      </Note.Root>
-    );
-  }
-
   return (
     <Note.Root
       className={cn(
@@ -93,26 +58,34 @@ export function RepostNote({
           </div>
         </User.Root>
       </User.Provider>
-      <Note.Provider event={repostEvent}>
-        <div className="flex flex-col gap-2">
-          <Note.User />
-          <div className="flex gap-3">
-            <div className="size-11 shrink-0" />
-            <div className="min-w-0 flex-1">
-              <Note.Content />
-              <div className="mt-4 flex items-center justify-between">
-                <div className="-ml-1 inline-flex items-center gap-4">
-                  <Note.Reply />
-                  <Note.Repost />
-                  <Note.Pin />
-                  {settings.zap ? <Note.Zap /> : null}
+      {isLoading ? (
+        <Spinner />
+      ) : isError ? (
+        <div className="w-full h-16 flex items-center px-3 border border-neutral-100 dark:border-neutral-900">
+          <p>Event not found</p>
+        </div>
+      ) : (
+        <Note.Provider event={repostEvent}>
+          <div className="flex flex-col gap-2">
+            <Note.User />
+            <div className="flex gap-3">
+              <div className="size-11 shrink-0" />
+              <div className="min-w-0 flex-1">
+                <Note.Content />
+                <div className="mt-4 flex items-center justify-between">
+                  <div className="-ml-1 inline-flex items-center gap-4">
+                    <Note.Reply />
+                    <Note.Repost />
+                    <Note.Pin />
+                    {settings.zap ? <Note.Zap /> : null}
+                  </div>
+                  <Note.Menu />
                 </div>
-                <Note.Menu />
               </div>
             </div>
           </div>
-        </div>
-      </Note.Provider>
+        </Note.Provider>
+      )}
     </Note.Root>
   );
 }
diff --git a/packages/ui/src/note/content.tsx b/packages/ui/src/note/content.tsx
index 9d3f50d4..ae08c8f7 100644
--- a/packages/ui/src/note/content.tsx
+++ b/packages/ui/src/note/content.tsx
@@ -25,7 +25,9 @@ export function NoteContent({
   compact?: boolean;
   className?: string;
 }) {
-  const settings: Settings = useRouteContext({ strict: false });
+  const { settings }: { settings: Settings } = useRouteContext({
+    strict: false,
+  });
   const event = useNoteContext();
   const content = useMemo(() => {
     const text = event.content.trim();
@@ -124,7 +126,7 @@ export function NoteContent({
       );
 
       if (compact) {
-        parsedContent = reactStringReplace(parsedContent, /\n|\r/g, () => (
+        parsedContent = reactStringReplace(parsedContent, /[\r\n]{2,}/g, () => (
           <div key={nanoid()} className="h-1.5" />
         ));
       }