cleanup

frontend/src/documents/DocumentThumbnailImage.tsx

@@ -73,12 +73,12 @@ const getPageCount = (doc?: Document | null): number | null => {
 
 type Asset = AssetManagerAsset;
 type EnsureAssetUrl = AssetManagerEnsureAssetUrl;
-type GetDocumentAsset = AssetManagerGetAsset;
+type GetAsset = AssetManagerGetAsset;
 
 interface DocumentThumbnailImageProps {
   document?: Document | null;
   ensureAssetUrl?: EnsureAssetUrl;
-  getDocumentAsset?: GetDocumentAsset;
+  getAsset?: GetAsset;
   alt?: string;
   maxSize?: number;
   scrollRootRef?: MutableRefObject<Element | null> | null;
@@ -87,7 +87,7 @@ interface DocumentThumbnailImageProps {
 const DocumentThumbnailImage = ({
   document,
   ensureAssetUrl,
-  getDocumentAsset,
+  getAsset,
   alt = '',
   maxSize = DEFAULT_THUMBNAIL_SIZE,
   scrollRootRef = null,
@@ -127,16 +127,16 @@ const DocumentThumbnailImage = ({
     }
     const options: {
       ensureAssetUrl?: EnsureAssetUrl;
-      getAsset?: GetDocumentAsset;
+      getAsset?: GetAsset;
     } = {};
     if (ensureAssetUrl) {
       options.ensureAssetUrl = ensureAssetUrl;
     }
-    if (getDocumentAsset) {
-      options.getAsset = getDocumentAsset;
+    if (getAsset) {
+      options.getAsset = getAsset;
     }
     return resolveDocumentAssetUrl(document, 'thumbnail', options) || resolveAssetUrl(thumbnailAsset);
-  }, [document, ensureAssetUrl, getDocumentAsset, isVisible, thumbnailAsset]);
+  }, [document, ensureAssetUrl, getAsset, isVisible, thumbnailAsset]);
 
   const pageCount = getPageCount(document);
   const showMultiPageBadge = pageCount !== null && pageCount > 1;

frontend/src/documents/components/DocumentsGridCard.tsx

@@ -80,7 +80,7 @@ const DocumentsGridCard: React.FC<DocumentsGridCardProps> = (props) => {
                     <DocumentThumbnailImage
                         document={doc}
                         ensureAssetUrl={props.ensureAssetUrl}
-                        getDocumentAsset={props.getDocumentAsset}
+                        getAsset={props.getDocumentAsset}
                         alt={`Thumbnail for ${doc.title}`}
                         maxSize={iconSize}
                         scrollRootRef={props.scrollRef}

frontend/src/documents/components/DocumentsListRow.tsx

@@ -90,7 +90,7 @@ const DocumentsListRow: React.FC<DocumentsListRowProps> = (props) => {
                         <DocumentThumbnailImage
                             document={doc}
                             ensureAssetUrl={props.ensureAssetUrl}
-                            getDocumentAsset={props.getDocumentAsset}
+                            getAsset={props.getDocumentAsset}
                             alt={`Thumbnail for ${doc.title}`}
                             scrollRootRef={props.scrollRef}
                             maxSize={props.iconSize}

frontend/src/documents/data/useDocumentsWorkspace.ts

@@ -1082,7 +1082,7 @@ const useDocumentsWorkspace = ({
     handleDocumentTagAdd,
     handleTagRemove,
     ensureAssetUrl,
-    getDocumentAsset,
+    getAsset: getDocumentAsset,
     correspondents,
     handleCorrespondentAdd,
     handleCorrespondentRemove,

frontend/src/documents/documentActions.ts

@@ -1,13 +1,5 @@
-import { openTextContentInNewTab } from '../utils/ocr';
-import type {
-  EnsureAssetUrl,
-  EnsurePreviewData,
-  GetDocumentAsset,
-} from '../utils/ocr';
 import type { Document } from '../types/documents';
 
-const asyncFalse = async () => false;
-
 export const resolveDocumentDownloadHref = (document?: Document | null): string | null => {
   if (!document) {
     return null;
@@ -21,65 +13,3 @@ export const resolveDocumentDownloadHref = (document?: Document | null): string
   }
   return download.url;
 };
-
-const hasDocumentTextContentAsset = (document?: Document | null, getDocumentAsset?: GetDocumentAsset | null): boolean => {
-  if (!document || !getDocumentAsset) {
-    return false;
-  }
-  return Boolean(getDocumentAsset(document, 'text-content'));
-};
-
-interface CreateDocumentActionStateArgs {
-  document: Document | null;
-  ensurePreviewData: EnsurePreviewData;
-  ensureAssetUrl: EnsureAssetUrl;
-  getDocumentAsset?: GetDocumentAsset | null;
-  notifyApiError?: (error: unknown, message: string) => void;
-  ocrErrorMessage?: string;
-}
-
-export const createDocumentActionState = ({
-  document,
-  ensurePreviewData,
-  ensureAssetUrl,
-  getDocumentAsset,
-  notifyApiError,
-  ocrErrorMessage = 'Unable to open text content.',
-}: CreateDocumentActionStateArgs) => {
-  if (!document) {
-    return {
-      downloadHref: null,
-      hasOcr: false,
-      openOcr: asyncFalse,
-    };
-  }
-
-  const downloadHref = resolveDocumentDownloadHref(document);
-  const hasOcr = hasDocumentTextContentAsset(document, getDocumentAsset);
-
-  const openOcr = hasOcr
-    ? async () => {
-      try {
-        const success = await openTextContentInNewTab({
-          document,
-          ensurePreviewData,
-          getDocumentAsset,
-          ensureAssetUrl,
-        });
-        if (!success) {
-          notifyApiError?.(new Error('Text content URL unavailable.'), ocrErrorMessage);
-        }
-        return success;
-      } catch (error) {
-        notifyApiError?.(error, ocrErrorMessage);
-        throw error;
-      }
-    }
-    : asyncFalse;
-
-  return {
-    downloadHref,
-    hasOcr,
-    openOcr,
-  };
-};

frontend/src/utils/ocr.ts

@@ -1,133 +0,0 @@
-import { resolveDocumentAssetUrl, resolveAssetUrl } from '../lib/assets/AssetManager';
-import type {
-  GetAsset as AssetManagerGetAsset,
-} from '../lib/assets/AssetManager';
-import type {
-  Document,
-} from '../types/documents';
-import type { Asset } from '../types/assets';
-
-export type EnsurePreviewData = (id: string) => Promise<Document | null>;
-export type EnsureAssetUrl = (
-  id: string,
-  asset: Asset,
-  options?: { force?: boolean },
-) => Promise<Asset | null>;
-export type GetDocumentAsset = AssetManagerGetAsset;
-
-interface ResolveTextContentUrlOptions {
-  document: Document | null;
-  ensurePreviewData?: EnsurePreviewData;
-  getDocumentAsset?: GetDocumentAsset;
-  ensureAssetUrl?: EnsureAssetUrl;
-}
-
-const pickAsset = (doc?: Document | null, getDocumentAsset?: GetDocumentAsset): Asset | null => {
-  if (!doc || !getDocumentAsset) {
-    return null;
-  }
-  return getDocumentAsset(doc, 'text-content') || null;
-};
-
-async function resolveTextContentUrl({
-  document,
-  ensurePreviewData,
-  getDocumentAsset,
-  ensureAssetUrl,
-}: ResolveTextContentUrlOptions): Promise<string | null> {
-  if (!document?.id) {
-    return null;
-  }
-
-  let docRef = document;
-  let asset = pickAsset(docRef, getDocumentAsset);
-
-  if (!asset && ensurePreviewData) {
-    const refreshed = await ensurePreviewData(docRef.id);
-    if (refreshed) {
-      docRef = refreshed;
-    }
-    asset = pickAsset(docRef, getDocumentAsset);
-  }
-
-  if (!asset) {
-    return null;
-  }
-
-  const baseUrl = resolveAssetUrl(asset);
-  const hasUrl = Boolean(baseUrl);
-
-  let entry: Asset = asset;
-  if (ensureAssetUrl) {
-    const ensureOptions = { force: !hasUrl };
-    const ensured = await ensureAssetUrl(docRef.id!, asset, ensureOptions);
-    if (ensured) {
-      entry = ensured;
-    }
-  }
-
-  const directUrl = resolveAssetUrl(entry);
-  if (directUrl) {
-    return directUrl;
-  }
-
-  return (
-    resolveDocumentAssetUrl(docRef, 'text-content', {
-      ensureAssetUrl,
-      getAsset: getDocumentAsset,
-      ensureOptions: { force: true },
-    }) || null
-  );
-}
-
-export async function openTextContentInNewTab(options: ResolveTextContentUrlOptions): Promise<boolean> {
-  const popup = window.open('', '_blank');
-  const popupAvailable = Boolean(popup);
-
-  if (popupAvailable) {
-    try {
-      popup.opener = null;
-    } catch {
-      /* ignore */
-    }
-  }
-
-  try {
-    const url = await resolveTextContentUrl(options);
-    if (!url) {
-      if (popupAvailable) {
-        popup.close();
-      }
-      return false;
-    }
-
-    if (popupAvailable) {
-      try {
-        popup.location.replace(url);
-      } catch {
-        try {
-          popup.location.href = url;
-        } catch (secondError) {
-          popup.close();
-          throw secondError;
-        }
-      }
-    } else {
-      const finalWindow = window.open(url, '_blank');
-      if (finalWindow) {
-        try {
-          finalWindow.opener = null;
-        } catch {
-          /* ignore */
-        }
-      }
-    }
-
-    return true;
-  } catch (error) {
-    if (popupAvailable) {
-      popup.close();
-    }
-    throw error;
-  }
-}

frontend/src/viewer/PdfViewer.tsx

@@ -379,14 +379,17 @@ const PdfViewer = ({ src, title, className, viewportRef }: PdfViewerProps): JSX.
   const stackStyle = useMemo<CSSProperties>(() => ({
     '--pdf-viewer-viewport-width': `${viewportWidth}px`,
     '--pdf-viewer-viewport-height': `${viewportHeight}px`,
-    cursor: viewMode === 'fit-width' ? 'zoom-out' : 'zoom-in',
-  }), [viewportHeight, viewportWidth, viewMode]);
+  }), [viewportHeight, viewportWidth]);
 
   const toggleViewMode = useCallback(() => {
     setViewMode((prev) => (prev === 'fit-width' ? 'contain' : 'fit-width'));
   }, []);
 
   const handlePageClick = useCallback((event: React.MouseEvent<HTMLDivElement>) => {
+    if (!(event.target as HTMLElement).closest('.pdf-viewer__page-wrapper')) {
+      return;
+    }
+
     const stack = containerRef.current;
     const viewportElement = viewportRef?.current || stack?.closest('.document-viewer__viewport');
     if (stack && viewportElement) {

frontend/src/viewer/UnifiedDocumentViewer.tsx

@@ -45,6 +45,7 @@ const UnifiedDocumentViewer: React.FC<UnifiedDocumentViewerProps> = ({
                 mimeType={normalizedMimeType}
                 filename={normalizedFilename}
                 alt={document.title}
+                onClick={(e) => e.stopPropagation()}
             />
         );
     }, [

frontend/src/viewer/components/PreviewZoomOverlay.tsx

@@ -24,6 +24,7 @@ const PreviewZoomOverlay: React.FC<PreviewZoomOverlayProps> = ({
   const [isBackdropVisible, setBackdropVisible] = useState(false);
   const lastDocumentRef = useRef<Document | null>(null);
   const scrollRef = useRef<HTMLDivElement | null>(null);
+  const stageRef = useRef<HTMLDivElement | null>(null);
   const timerRef = useRef<number | null>(null);
 
   if (inputDocument) {
@@ -104,45 +105,43 @@ const PreviewZoomOverlay: React.FC<PreviewZoomOverlayProps> = ({
         onClick={onClose}
         onKeyDown={handleKeyDown}
       >
-        <PanelHeader
-          className="panel-header--dark"
-          title={documentTitle}
-          leading={
-            <>
-              <button
-                onClick={onClose}
-                className="icon-button"
-                aria-label="Close preview"
-                type="button"
-              >
-                <IconX />
-              </button>
-              {onMaximize && (
+        <div onClick={(e) => e.stopPropagation()}>
+          <PanelHeader
+            className="panel-header--dark"
+            title={documentTitle}
+            leading={
+              <>
                 <button
-                  onClick={onMaximize}
+                  onClick={onClose}
                   className="icon-button"
-                  aria-label="Open document info"
-                  title="Open document info"
+                  aria-label="Close preview"
                   type="button"
                 >
-                  <FileInfoIcon />
+                  <IconX />
                 </button>
-              )}
-            </>
-          }
-          actions={
-            <>
-              <DocumentDownloadLink document={lastDocumentRef.current} />
-            </>
-          }
-        />
+                {onMaximize && (
+                  <button
+                    onClick={onMaximize}
+                    className="icon-button"
+                    aria-label="Open document info"
+                    title="Open document info"
+                    type="button"
+                  >
+                    <FileInfoIcon />
+                  </button>
+                )}
+              </>
+            }
+            actions={
+              <>
+                <DocumentDownloadLink document={lastDocumentRef.current} />
+              </>
+            }
+          />
+        </div>
         <div
           className={stageClassName}
-          onClick={(event) => {
-            if (event.target === event.currentTarget) {
-              onClose();
-            }
-          }}
+          ref={stageRef}
         >
           <div
             className={containerClassName}

frontend/src/viewer/logic/useDetailWorkspace.ts

@@ -4,7 +4,7 @@ import { resolveDocumentAssetUrl } from '../../lib/assets/AssetManager';
 import { useDetailPanel } from '../../app/useDetailPanel';
 import { DEFAULT_FOLDER_NAME } from '../../app/workspaceUtils';
 import type { DocumentInfoPanelProps } from '../components/DocumentInfoPanel';
-import type { EnsureAssetUrl, GetDocumentAsset } from '../../utils/ocr';
+import type { EnsureAssetUrl, GetAsset } from '../../lib/assets/AssetManager';
 import type { Identifier } from '../../types/identifiers';
 import type { Document } from '../../types/documents';
 
@@ -29,7 +29,7 @@ interface UseDetailWorkspaceArgs {
   handleDocumentTagAdd?: (doc: Document, value: string, context?: { option?: unknown }) => void;
   handleTagRemove?: (...args: unknown[]) => void;
   ensureAssetUrl?: EnsureAssetUrl;
-  getDocumentAsset?: GetDocumentAsset;
+  getAsset?: GetAsset;
   correspondents?: unknown[];
   handleCorrespondentAdd?: (...args: unknown[]) => void;
   handleCorrespondentRemove?: (...args: unknown[]) => void;
@@ -66,7 +66,7 @@ const useDetailWorkspace = ({
   handleDocumentTagAdd,
   handleTagRemove,
   ensureAssetUrl,
-  getDocumentAsset,
+  getAsset,
   correspondents,
   handleCorrespondentAdd,
   handleCorrespondentRemove,
@@ -191,9 +191,9 @@ const useDetailWorkspace = ({
     (doc) =>
       resolveDocumentAssetUrl(doc, 'thumbnail', {
         ensureAssetUrl,
-        getAsset: getDocumentAsset,
+        getAsset,
       }),
-    [ensureAssetUrl, getDocumentAsset],
+    [ensureAssetUrl, getAsset],
   );
 
   const inspectDocument = useCallback(
@@ -222,7 +222,7 @@ const useDetailWorkspace = ({
       onUpdateTitle: handleDocumentTitleUpdate,
       onUpdateIssued: handleDocumentIssuedUpdate,
       ensureAssetUrl,
-      getDocumentAsset,
+      getDocumentAsset: getAsset,
       correspondents,
       onCorrespondentAdd: handleCorrespondentAdd,
       onCorrespondentRemove: handleCorrespondentRemove,
@@ -237,7 +237,7 @@ const useDetailWorkspace = ({
       correspondents,
       detailPanelDocument,
       ensureAssetUrl,
-      getDocumentAsset,
+      getAsset,
       handleCorrespondentAdd,
       handleCorrespondentRemove,
       handleDetailPanelClose,

frontend/src/viewer/styles/viewer.css

@@ -631,4 +631,12 @@
 .pdf-viewer__password-error {
   color: var(--danger);
   font-size: 0.9rem;
+}
+
+.pdf-viewer__canvas-stack--fit-width .pdf-viewer__page-wrapper {
+  cursor: zoom-out;
+}
+
+.pdf-viewer__canvas-stack--contain .pdf-viewer__page-wrapper {
+  cursor: zoom-in;
 }