push custom handling to radix - keep modifier logic

Author: jaffrepaul

src/components/imageLightbox/index.tsx

@@ -77,16 +77,12 @@ export function ImageLightbox({
   };
 
   const handleKeyDown = (e: React.KeyboardEvent) => {
-    if (e.key === 'Enter' || e.key === ' ') {
-      if (e.ctrlKey || e.metaKey) {
-        e.preventDefault();
-        e.stopPropagation();
-        openInNewTab();
-        return;
-      }
-      // Regular Enter/Space should open lightbox
+    // Only handle Ctrl/Cmd+Enter/Space for opening in new tab
+    // Radix UI Dialog.Trigger handles regular Enter/Space automatically
+    if ((e.key === 'Enter' || e.key === ' ') && (e.ctrlKey || e.metaKey)) {
       e.preventDefault();
-      setOpen(true);
+      e.stopPropagation();
+      openInNewTab();
     }
   };
 
@@ -140,8 +136,6 @@ export function ImageLightbox({
           onKeyDown={handleKeyDown}
           className="cursor-pointer border-none bg-transparent p-0 block w-full no-underline"
           aria-label={`View image: ${alt}`}
-          role="button"
-          tabIndex={0}
         >
           {renderImage()}
         </div>