Fix markdown editor not updating resource #969

atomicdata-dev · Oct 7, 2024 · 69f4b5c · 69f4b5c
1 parent 94f2ede
commit 69f4b5c
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 2 deletions.
diff --git a/browser/CHANGELOG.md b/browser/CHANGELOG.md
@@ -9,6 +9,8 @@ This changelog covers all five packages, as they are (for now) updated as a whol
 - [#952](https://github.com/atomicdata-dev/atomic-server/issues/952) Add templates containing pre made ontologies and resources.
 - [#970](https://github.com/atomicdata-dev/atomic-server/issues/970) Add "show commit" button in History
 - [#968](https://github.com/atomicdata-dev/atomic-server/issues/968) Allow users to pick files by entering a subject into the file picker search bar.
+- [#969](https://github.com/atomicdata-dev/atomic-server/issues/969) Fix markdown editor sometimes doesn't update the value after saving.
+- Fix markdown editor closing the edit page when a button is clicked.
 
 ### @tomic/lib
 

diff --git a/browser/data-browser/src/chunks/MarkdownEditor/AsyncMarkdownEditor.tsx b/browser/data-browser/src/chunks/MarkdownEditor/AsyncMarkdownEditor.tsx
@@ -110,7 +110,7 @@ export default function AsyncMarkdownEditor({
           />
         )}
         <EditorContent key='rich-editor' editor={editor}>
-          <FloatingMenu editor={editor ?? undefined}>
+          <FloatingMenu editor={editor ?? null}>
             <FloatingMenuText>Type &apos;/&apos; for options</FloatingMenuText>
           </FloatingMenu>
           <BubbleMenu />

diff --git a/browser/data-browser/src/chunks/MarkdownEditor/BubbleMenu.tsx b/browser/data-browser/src/chunks/MarkdownEditor/BubbleMenu.tsx
@@ -36,6 +36,7 @@ export function BubbleMenu(): React.JSX.Element {
           active={!!editor.isActive('bold')}
           onClick={() => editor.chain().focus().toggleBold().run()}
           disabled={!editor.can().chain().focus().toggleBold().run()}
+          type='button'
         >
           <FaBold />
         </ToggleButton>
@@ -44,6 +45,7 @@ export function BubbleMenu(): React.JSX.Element {
           active={!!editor.isActive('italic')}
           onClick={() => editor.chain().focus().toggleItalic().run()}
           disabled={!editor.can().chain().focus().toggleItalic().run()}
+          type='button'
         >
           <FaItalic />
         </ToggleButton>
@@ -52,6 +54,7 @@ export function BubbleMenu(): React.JSX.Element {
           active={!!editor.isActive('strike')}
           onClick={() => editor.chain().focus().toggleStrike().run()}
           disabled={!editor.can().chain().focus().toggleStrike().run()}
+          type='button'
         >
           <FaStrikethrough />
         </ToggleButton>
@@ -60,6 +63,7 @@ export function BubbleMenu(): React.JSX.Element {
           active={!!editor.isActive('blockquote')}
           onClick={() => editor.chain().focus().toggleBlockquote().run()}
           disabled={!editor.can().chain().focus().toggleBlockquote().run()}
+          type='button'
         >
           <FaQuoteLeft />
         </ToggleButton>
@@ -68,6 +72,7 @@ export function BubbleMenu(): React.JSX.Element {
           active={!!editor.isActive('code')}
           onClick={() => editor.chain().focus().toggleCode().run()}
           disabled={!editor.can().chain().focus().toggleCode().run()}
+          type='button'
         >
           <FaCode />
         </ToggleButton>
@@ -80,6 +85,7 @@ export function BubbleMenu(): React.JSX.Element {
               as={RadixPopover.Trigger}
               active={!!editor.isActive('link')}
               disabled={!editor.can().chain().focus().toggleCode().run()}
+              type='button'
             >
               <FaLink />
             </ToggleButton>

diff --git a/browser/data-browser/src/chunks/MarkdownEditor/EditorEvents.tsx b/browser/data-browser/src/chunks/MarkdownEditor/EditorEvents.tsx
@@ -24,7 +24,7 @@ export function EditorEvents({ onChange }: EditorEventsProps): null {
         editor.off('update', callback);
       }
     };
-  }, [editor]);
+  }, [editor, onChange]);
 
   return null;
 }