cat-documents-ng 1.0.15 → 1.0.16

package/fesm2022/cat-documents-ng.mjs

@@ -9280,6 +9280,39 @@ class DocumentListComponent {
      */
     documentListSubscription = new Subscription();
     handleSelectedDocumentInNewTab = new EventEmitter();
+    /**
+     * Callback function to handle opening a document in a new tab when Ctrl+Click is used.
+     * This callback is called directly from the user gesture (preserving the gesture chain),
+     * BEFORE any Angular change detection, so window.open() will work without popup blockers.
+     *
+     * IMPORTANT: Build your URL synchronously and call window.open() immediately. Do NOT:
+     * - Use setTimeout, Promise, async/await
+     * - Use Angular Router.navigate() (use window.open() with URL string instead)
+     * - Trigger any operations that cause change detection before window.open()
+     *
+     * Best practice implementation:
+     *
+     * ```typescript
+     * onOpenInNewTab = (event: { selectedDocument: any, contextId: string, documentList: any[] }) => {
+     *   // Build URL synchronously (no async operations)
+     *   const baseUrl = window.location.origin;
+     *   const params = new URLSearchParams({
+     *     selectedDocument: JSON.stringify(event.selectedDocument),
+     *     contextId: event.contextId,
+     *     documentList: JSON.stringify(event.documentList)
+     *   });
+     *   const url = `${baseUrl}/your-route?${params.toString()}`;
+     *
+     *   // Call window.open() immediately - this preserves the user gesture chain
+     *   const newWindow = window.open(url, '_blank');
+     *   if (newWindow) {
+     *     newWindow.focus(); // Focus the new tab
+     *   }
+     * }
+     * ```
+     *
+     * @param event - The event object containing selectedDocument, contextId, and documentList
+     */
     onOpenInNewTab;
     /**
      * The currently selected document.
@@ -9507,20 +9540,37 @@ class DocumentListComponent {
         }
     }
     handleTableRowCtrlClick(rowData) {
-        this.selectedDocument = this.documentListService.handleTableRowClick(rowData);
-        this.handleSelectedDocument.emit(this.selectedDocument);
+        // CRITICAL: Prepare data synchronously but don't trigger any Angular operations yet
+        // The callback MUST be called before any emit() calls to preserve user gesture chain
+        const selectedDocument = this.documentListService.handleTableRowClick(rowData);
         const event = {
-            selectedDocument: this.selectedDocument,
+            selectedDocument: selectedDocument,
             contextId: this.contextId,
             documentList: this.documentList
         };
-        // Call callback directly from click handler (preserves user gesture chain)
+        // Call callback FIRST, before any Angular operations that might trigger change detection
+        // This preserves the user gesture chain, allowing window.open() to work and focus properly
         if (this.onOpenInNewTab && typeof this.onOpenInNewTab === 'function') {
-            this.onOpenInNewTab(event);
-            // Don't emit event if callback is provided
+            try {
+                // Execute callback immediately while still in the user gesture chain
+                this.onOpenInNewTab(event);
+                // After callback completes, update component state and emit events
+                // These operations happen after window.open() has already been called
+                this.selectedDocument = selectedDocument;
+                this.handleSelectedDocument.emit(this.selectedDocument);
+            }
+            catch (error) {
+                console.error('Error in onOpenInNewTab callback:', error);
+                // Fallback to event emitter if callback fails
+                this.selectedDocument = selectedDocument;
+                this.handleSelectedDocument.emit(this.selectedDocument);
+                this.handleSelectedDocumentInNewTab.emit(event);
+            }
             return;
         }
         // Only emit if no callback provided (backward compatibility)
+        this.selectedDocument = selectedDocument;
+        this.handleSelectedDocument.emit(this.selectedDocument);
         this.handleSelectedDocumentInNewTab.emit(event);
     }
     /**
@@ -10122,6 +10172,39 @@ class DocumentContainerComponent {
      * The currently selected menu item ID for scrolling
      */
     selectedMenuItemId = null;
+    /**
+     * Callback function to handle opening a document in a new tab when Ctrl+Click is used.
+     * This callback is called directly from the user gesture (preserving the gesture chain),
+     * BEFORE any Angular change detection, so window.open() will work without popup blockers.
+     *
+     * IMPORTANT: Build your URL synchronously and call window.open() immediately. Do NOT:
+     * - Use setTimeout, Promise, async/await
+     * - Use Angular Router.navigate() (use window.open() with URL string instead)
+     * - Trigger any operations that cause change detection before window.open()
+     *
+     * Best practice implementation:
+     *
+     * ```typescript
+     * onOpenInNewTab = (event: { selectedDocument: any, contextId: string, documentList: any[] }) => {
+     *   // Build URL synchronously (no async operations)
+     *   const baseUrl = window.location.origin;
+     *   const params = new URLSearchParams({
+     *     selectedDocument: JSON.stringify(event.selectedDocument),
+     *     contextId: event.contextId,
+     *     documentList: JSON.stringify(event.documentList)
+     *   });
+     *   const url = `${baseUrl}/your-route?${params.toString()}`;
+     *
+     *   // Call window.open() immediately - this preserves the user gesture chain
+     *   const newWindow = window.open(url, '_blank');
+     *   if (newWindow) {
+     *     newWindow.focus(); // Focus the new tab
+     *   }
+     * }
+     * ```
+     *
+     * @param event - The event object containing selectedDocument, contextId, and documentList
+     */
     onOpenInNewTab;
     /**
      * Additional navigation information for better scrolling