cleanup

Author: jaffrepaul

src/components/docImage.tsx

@@ -19,19 +19,17 @@ export default function DocImage({
   // Handle external images early - pass through without processing
   if (src.startsWith('http')) {
     // Use provided props or defaults for external images
-    const width =
-      typeof propsWidth === 'number'
-        ? propsWidth
-        : typeof propsWidth === 'string'
-          ? parseInt(propsWidth, 10) || 800
-          : 800;
-    const height =
-      typeof propsHeight === 'number'
-        ? propsHeight
-        : typeof propsHeight === 'string'
-          ? parseInt(propsHeight, 10) || 600
-          : 600;
-
+    const width = typeof propsWidth === 'number'
+      ? propsWidth
+      : typeof propsWidth === 'string'
+        ? parseInt(propsWidth, 10) || 800
+        : 800;
+    const height = typeof propsHeight === 'number'
+      ? propsHeight
+      : typeof propsHeight === 'string'
+        ? parseInt(propsHeight, 10) || 800
+        : 800;
+        
     return (
       <DocImageClient
         src={src}

src/components/docImageClient.tsx

@@ -28,8 +28,8 @@ export function DocImageClient({
   // Check if dimensions are valid (not NaN) for Next.js Image
   const isValidDimensions = !isNaN(width) && !isNaN(height) && width > 0 && height > 0;
 
-  // For external images or invalid dimensions, fall back to regular img tag
-  if (src.startsWith('http') || !isValidDimensions) {
+  // For images with invalid dimensions, fall back to regular img tag
+  if (!isValidDimensions) {
     const handleClick = () => {
       // Always open image in new tab
       const url = src.startsWith('http') ? src : imgPath;

src/components/imageLightbox.tsx

@@ -53,7 +53,7 @@ export function ImageLightbox({
 
         <Dialog.Content className="image-lightbox-content fixed left-[50%] top-[50%] z-50 max-h-[90vh] max-w-[90vw] translate-x-[-50%] translate-y-[-50%]">
           {/* Close button */}
-          <Dialog.Close className="absolute right-4 top-4 z-10 rounded-sm bg-black/50 p-2 text-white opacity-70 ring-offset-background transition-opacity hover:opacity-100 focus:outline-none focus:ring-2 focus:ring-ring focus:ring-offset-2">
+          <Dialog.Close className="absolute right-4 top-4 z-10 rounded-sm bg-black/50 p-2 text-white opacity-70 transition-opacity hover:opacity-100 focus:outline-none focus:ring-2 focus:ring-white focus:ring-offset-2 focus:ring-offset-black/50">
             <X className="h-4 w-4" />
             <span className="sr-only">Close</span>
           </Dialog.Close>