Fix note detail scrolling

README.md

@@ -1,10 +1,6 @@
 Bugs
 
-- [ ] Fix scrolling on note detail
-- [ ] Be sure to deduplicate all events if needed
-- [ ] Format text with line breaks - pre, or split/br
 - [ ] Remove dexie, or use it instead of localstorage for cached data
-- [ ] rename /user to /users
 - [ ] Memoize room list, currently every time the user switches chat rooms it pulls the full list
 - [ ] Fix toast, it gets in the way. Make it smaller and dismissable.
 

src/App.svelte

@@ -53,7 +53,7 @@
 
 <Router {url}>
   <div use:links class="h-full">
-    <div class="pt-16 text-white h-full">
+    <div class="pt-16 text-white h-full" class:overflow-hidden={$modal}>
       <Route path="/notes" component={Notes} />
       <Route path="/notes/new" component={NoteCreate} />
       <Route path="/chat" component={Chat} />
@@ -138,7 +138,7 @@
         transition:fade
         on:click={e => modal.set(null)} />
       <div class="absolute inset-0 mt-20 sm:mt-32 modal-content" transition:fly={{y: 1000, opacity: 1}}>
-        <dialog open class="bg-dark border-t border-solid border-medium h-full w-full">
+        <dialog open class="bg-dark border-t border-solid border-medium h-full w-full overflow-auto">
           {#if $modal.note}
             {#key $modal.note.id}
             <NoteDetail note={$modal.note} />

src/state/app.js

@@ -163,7 +163,7 @@ export const findNotesAndWatchModal = (filters, cb) => {
   })
 
   return () => {
-    stop()
+    stop && stop()
     unsub()
   }
 }

src/state/nostr.js

@@ -16,7 +16,7 @@ const channel = name => {
   const _chan = {
     sub: params => {
       if (active) {
-        throw new Error(`Channel ${name} is already active.`)
+        console.error(`Channel ${name} is already active.`)
       }
 
       active = true