From 5f60cc826a64b462d9049e2d54e71c5e31ce1355 Mon Sep 17 00:00:00 2001
From: Hiren Kangad <hir.012612@gmail.com>
Date: Thu, 12 Mar 2026 18:50:01 +0530
Subject: [PATCH] fix(review): gate pin visibility on review mode per spec

---
 frontend/src/pages/Editor.tsx | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/frontend/src/pages/Editor.tsx b/frontend/src/pages/Editor.tsx
index 08ae71d..ef2d016 100644
--- a/frontend/src/pages/Editor.tsx
+++ b/frontend/src/pages/Editor.tsx
@@ -201,12 +201,12 @@ export default function Editor({ isPublicView }: EditorProps) {
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [objectCount]);
 
-  // Pin visibility — will be gated on reviewMode in Task 10
+  // Pins visible only in review mode
   useEffect(() => {
     if (!pinOverlay) return;
-    pinOverlay.visible = true;
-    pinOverlay.refresh();
-  }, [pinOverlay]);
+    pinOverlay.visible = reviewMode;
+    if (reviewMode) pinOverlay.refresh();
+  }, [pinOverlay, reviewMode]);
 
   // Sync draft pin to PinOverlay ghost pin
   useEffect(() => {