refactor: clarify when to show toolbar

MovieReviewComment · May 4, 2024 · 171799e · 171799e
1 parent 765dba6
commit 171799e
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/ui/src/components/review/client/editor-sticky-toolbar.tsx b/ui/src/components/review/client/editor-sticky-toolbar.tsx
@@ -16,13 +16,11 @@ import { useEditorRef } from '@/context/editor/editor-ref-context';
 import { useStickyIOS } from '@/hooks/common/use-sticky-ios';
 
 export function EditorStickyToolbar() {
-  const { isEditable } = useEditable() ?? {};
+  const { isEditable } = useEditable() ?? { isEditable: null };
 
-  if (isEditable === false) {
-    return null;
-  }
+  const showToolbar = isEditable === null || isEditable === true;
 
-  return <EditorToolbarInner />;
+  return showToolbar ? <EditorToolbarInner /> : null;
 }
 
 function EditorToolbarInner() {