🧑‍💻 open TIL items in notion with shortcut

Author: juliuxu

apps/nye-julianjark.no/app/components/dette-kan-jeg.tsx

@@ -5,6 +5,7 @@ import { classNames, shuffled } from "~/utils/misc";
 import { HoverCard, HoverCardContent, HoverCardTrigger } from "./hover-card";
 import { RichTextListRender } from "@julianjark/notion-render";
 import { Popover, PopoverContent, PopoverTrigger } from "~/components/popover";
+import { RegisterEditNotionPage } from "~/routes/($prefix).$notionPage/use-edit-notion-page";
 
 export const getDetteKanJegData = async () => {
   const detteKanJeg = await getAllDetteKanJeg();
@@ -75,6 +76,7 @@ function KnowledgeList({ knowledge }: { knowledge: DetteKanJeg[] }) {
                   sideOffset={14}
                   className="p-6 w-96 flex space-x-6"
                 >
+                  <RegisterEditNotionPage pageId={item.id} />
                   {item.logo && (
                     <span className="relative flex h-12 w-12 shrink-0 overflow-hidden">
                       <img

apps/nye-julianjark.no/app/routes/($prefix).$notionPage/use-edit-notion-page.tsx

@@ -1,7 +1,6 @@
 import { useEffect } from "react";
 
 // This lets the notion editing be triggered from anywhere in the app
-// Although not not ideal, it works
 let currentNotionPageId: string | undefined;
 export function openCurrentNotionPage() {
   if (!currentNotionPageId) return;
@@ -19,6 +18,20 @@ export function openCurrentNotionPage() {
  */
 export function useEditNotionPage({ pageId }: { pageId: string }) {
   useEffect(() => {
+    const previousNotionPageId = currentNotionPageId;
     currentNotionPageId = pageId;
+
+    // Use the previous pageId when this unmounts
+    // This allows us to use this in dialogs and other places temporary components
+    return () => {
+      if (currentNotionPageId === pageId) {
+        currentNotionPageId = previousNotionPageId;
+      }
+    };
   }, [pageId]);
 }
+
+export function RegisterEditNotionPage({ pageId }: { pageId: string }) {
+  useEditNotionPage({ pageId });
+  return null;
+}