df-ae-forms-package 1.0.82 → 1.0.84

package/dist/index.esm.js

@@ -7592,8 +7592,10 @@ const TableCellComponent = ({ cell, mode, onComponentSelect, onComponentDelete,
 };
 const DfFormTable = ({ id, properties, mode = 'edit', formData = {}, validationErrors = {}, touchedFields = {}, formSubmitted = false, onValueChange, onSelect, isSelected = false, className = '', onTableSelect, onComponentSelect, onComponentDelete, onComponentEdit, selectedComponent, renderFormComponent, formTemplateId, onThresholdActionCompletion, onThresholdIssueRaised, onNotesChange, onAttachmentChange, workOrderNumber, assetNumber, user, onCreateIssue, onUpdateIssue, shouldShowComponent }) => {
     const [isCollapsed, setIsCollapsed] = useState(false);
-    // Cells may be 2D (row arrays) or 1D (flat cells) depending on API format.
-    // The useMemo in DfFormPreview ensures IDs, but structure may still vary.
+    // NOTE: Unlike the main website, the package does NOT have a cell initialization useEffect.
+    // In the main website, onValueChange handles component structure updates (updating cells).
+    // In the package, onValueChange maps to onFormValueChange which only handles simple form values.
+    // The cells are already properly structured from the API and processed by the useMemo in DfFormPreview.
     // Check if table has any components in any cells
     const hasAnyComponents = properties.cells?.some((row) => {
         if (Array.isArray(row)) {
@@ -7658,11 +7660,10 @@ const DfFormTable = ({ id, properties, mode = 'edit', formData = {}, validationE
     const cellsWithIds = properties.cells && properties.cells.length > 0
         ? ensureTableCellComponentsHaveIds(properties.cells)
         : [];
-    // CRITICAL FIX: Update the parent component with the cells that have proper IDs
-    // Skip this in package to prevent flickering - cells are managed by parent
-    // useEffect(() => {
-    //   // Disabled to prevent flickering when user types
-    // }, [])
+    // NOTE: Unlike the main website, we do NOT sync cellsWithIds back via onValueChange.
+    // In the package, onValueChange = onFormValueChange which handles simple form values,
+    // not component structure updates. Syncing would corrupt the state.
+    // cellsWithIds is used only for rendering.
     // CRITICAL: Create a stable renderComponent that doesn't recreate on every render
     // Match main app implementation - directly access formData from closure
     const renderComponent = renderFormComponent || useCallback((field) => {
@@ -7770,22 +7771,22 @@ const DfFormTable = ({ id, properties, mode = 'edit', formData = {}, validationE
     if (!hasAnyComponents && mode === 'preview') {
         return null;
     }
-    return (jsxs("div", { className: `form-group df-form-table ${isSelected ? 'selected' : ''} ${mode === 'preview' ? 'preview-mode' : mode === 'test' ? 'test-mode' : ''} ${className}`, onClick: handleTableClick, style: tableStyle, children: [jsx("div", { className: "table-header", children: jsx("div", { className: "table-title", onClick: toggleCollapse, children: jsxs("div", { className: "title-content", children: [isCollapsed ? jsx(ChevronRight, { size: 16 }) : jsx(ChevronDown, { size: 16 }), jsx(Table, { size: 16 }), jsx("span", { className: "table-label", children: properties.basic?.label || 'Table' }), properties.validation?.required && (jsx("span", { className: "required-indicator", children: "*" }))] }) }) }), properties.basic?.description && !isCollapsed && (jsx("div", { className: "table-description", children: properties.basic.description })), !isCollapsed && (jsxs("div", { className: "table-content", children: [(() => {
-                        return null;
-                    })(), jsxs("table", { style: tableElementStyle, children: [properties.table?.displayAsTable && (mode === 'edit' || properties.table?.showColumns) && (jsx("thead", { children: jsx("tr", { className: "table-header-row", children: Array.from({ length: properties.table?.columns || properties.basic?.columns || 3 }, (_, colIndex) => {
-                                        // Parse column names from the comma-separated string
-                                        const columnNames = properties.table?.columnNames?.split(',').map(name => name.trim()) || [];
-                                        const columnName = columnNames[colIndex] || `Column ${colIndex + 1}`;
-                                        return (jsx("th", { className: "table-header-cell", style: {
-                                                backgroundColor: properties.styles.headerBackgroundColor || (mode === 'preview' || mode === 'test') ? 'var(--df-color-fb-container)' : 'var(--df-color-fb-container)',
-                                                color: properties.styles.headerTextColor || 'var(--df-color-text-dark)',
-                                                padding: '12px',
-                                                border: (mode === 'preview' || mode === 'test') ? '1px solid var(--df-color-fb-border)' : '1px solid var(--df-color-fb-border)',
-                                                fontWeight: '600',
-                                                fontSize: '14px',
-                                                textAlign: 'center'
-                                            }, children: columnName }, `header-${colIndex}`));
-                                    }) }) })), jsx("tbody", { children: cellsWithIds.filter((row) => Array.isArray(row) && row.length > 0).map((row, rowIndex) => (jsx("tr", { className: "table-row", children: row.filter((cell) => cell != null).map((cell) => (jsx(TableCellComponent, { cell: cell, mode: mode, onComponentSelect: onComponentSelect || (() => { }), onComponentDelete: handleComponentDelete, onComponentEdit: onComponentEdit, selectedComponent: selectedComponent || null, renderFormComponent: renderComponent, formData: formData, formTemplateId: formTemplateId, onThresholdActionCompletion: onThresholdActionCompletion, onThresholdIssueRaised: onThresholdIssueRaised, tableId: id, onNotesChange: onNotesChange, onAttachmentChange: onAttachmentChange, workOrderNumber: workOrderNumber, assetNumber: assetNumber, user: user, onCreateIssue: onCreateIssue, onUpdateIssue: onUpdateIssue, shouldShowComponent: shouldShowComponent }, cell.id))) }, rowIndex))) })] })] }))] }));
+    return (jsxs("div", { className: `form-group df-form-table ${isSelected ? 'selected' : ''} ${mode === 'preview' ? 'preview-mode' : mode === 'test' ? 'test-mode' : ''} ${className}`, onClick: handleTableClick, style: tableStyle, children: [jsx("div", { className: "table-header", children: jsx("div", { className: "table-title", onClick: toggleCollapse, children: jsxs("div", { className: "title-content", children: [isCollapsed ? jsx(ChevronRight, { size: 16 }) : jsx(ChevronDown, { size: 16 }), jsx(Table, { size: 16 }), jsx("span", { className: "table-label", children: properties.basic?.label || 'Table' }), properties.validation?.required && (jsx("span", { className: "required-indicator", children: "*" }))] }) }) }), properties.basic?.description && !isCollapsed && (jsx("div", { className: "table-description", children: properties.basic.description })), !isCollapsed && (jsx("div", { className: "table-content", children: jsxs("table", { style: tableElementStyle, children: [properties.table?.displayAsTable && (mode === 'edit' || properties.table?.showColumns) && (jsx("thead", { children: jsx("tr", { className: "table-header-row", children: Array.from({ length: properties.table?.columns || properties.basic?.columns || 3 }, (_, colIndex) => {
+                                    const columnNames = properties.table?.columnNames?.split(',').map((name) => name.trim()) || [];
+                                    const columnName = columnNames[colIndex] || `Column ${colIndex + 1}`;
+                                    return (jsx("th", { className: "table-header-cell", style: {
+                                            backgroundColor: 'var(--df-color-fb-container)',
+                                            color: properties.styles.headerTextColor || 'var(--df-color-text-dark)',
+                                            padding: '12px',
+                                            border: '1px solid var(--df-color-fb-border)',
+                                            fontWeight: '600',
+                                            fontSize: '14px',
+                                            textAlign: 'center'
+                                        }, children: columnName }, `header-${colIndex}`));
+                                }) }) })), jsx("tbody", { children: cellsWithIds.map((row, rowIndex) => {
+                                const rowCells = Array.isArray(row) ? row : [row];
+                                return (jsx("tr", { className: "table-row", children: rowCells.map((cell) => (jsx(TableCellComponent, { cell: cell, mode: mode, onComponentSelect: onComponentSelect || (() => { }), onComponentDelete: handleComponentDelete, onComponentEdit: onComponentEdit, selectedComponent: selectedComponent || null, renderFormComponent: renderComponent, formData: formData, formTemplateId: formTemplateId, onThresholdActionCompletion: onThresholdActionCompletion, onThresholdIssueRaised: onThresholdIssueRaised, tableId: id, onNotesChange: onNotesChange, onAttachmentChange: onAttachmentChange, workOrderNumber: workOrderNumber, assetNumber: assetNumber, user: user, onCreateIssue: onCreateIssue, onUpdateIssue: onUpdateIssue, shouldShowComponent: shouldShowComponent }, cell.id))) }, rowIndex));
+                            }) })] }) }))] }));
 };
 
 var dfFormTable = /*#__PURE__*/Object.freeze({