From 4f2c9756b8e823e9ce9901c080a5b6a869f0fbc1 Mon Sep 17 00:00:00 2001
From: Marko Djordjevic <marko.homoludens@gmail.com>
Date: Wed, 18 Feb 2026 20:39:07 +0100
Subject: [PATCH] code-review-fix task 12.6: remove manual Escape handler from
 SpanPopover conflicting with Radix Dialog

---
 src/components/SpanPopover.tsx | 11 -----------
 1 file changed, 11 deletions(-)

diff --git a/src/components/SpanPopover.tsx b/src/components/SpanPopover.tsx
index d941d22..89613ed 100644
--- a/src/components/SpanPopover.tsx
+++ b/src/components/SpanPopover.tsx
@@ -97,17 +97,6 @@ export default function SpanPopover({
     onCancel();
   };
 
-  const handleEscape = (e: KeyboardEvent) => {
-    if (e.key === 'Escape' && open) {
-      handleCancel();
-    }
-  };
-
-  useEffect(() => {
-    window.addEventListener('keydown', handleEscape);
-    return () => window.removeEventListener('keydown', handleEscape);
-  }, [open]);
-
   return (
     <Dialog open={open} onOpenChange={onOpenChange}>
       <DialogContent className="sm:max-w-[425px]">