funda-ui 4.7.723 → 4.7.730

package/lib/cjs/Utils/useBoundedDrag.d.ts

@@ -112,6 +112,7 @@ export interface BoundedDragOptions {
     dragHandleSelector?: string;
     onDragStart?: (index: number) => void;
     onDragOver?: (dragIndex: number | null, dropIndex: number | null) => void;
+    onDragUpdate?: (dragIndex: number | null, dropIndex: number | null) => void;
     onDragEnd?: (dragIndex: number | null, dropIndex: number | null) => void;
 }
 export declare const useBoundedDrag: (options?: BoundedDragOptions) => {

package/lib/cjs/Utils/useBoundedDrag.js

@@ -220,6 +220,7 @@ var useBoundedDrag = function useBoundedDrag() {
     dragHandleSelector = _options$dragHandleSe === void 0 ? '.custom-draggable-list__handle' : _options$dragHandleSe,
     onDragStart = options.onDragStart,
     onDragOver = options.onDragOver,
+    onDragUpdate = options.onDragUpdate,
     onDragEnd = options.onDragEnd;
   var _useState = (0,react__WEBPACK_IMPORTED_MODULE_0__.useState)(false),
     _useState2 = _slicedToArray(_useState, 2),
@@ -228,11 +229,35 @@ var useBoundedDrag = function useBoundedDrag() {
   var dragItem = (0,react__WEBPACK_IMPORTED_MODULE_0__.useRef)(null);
   var dragOverItem = (0,react__WEBPACK_IMPORTED_MODULE_0__.useRef)(null);
   var dragNode = (0,react__WEBPACK_IMPORTED_MODULE_0__.useRef)(null);
+  var draggedElement = (0,react__WEBPACK_IMPORTED_MODULE_0__.useRef)(null);
+  var boundaryElement = (0,react__WEBPACK_IMPORTED_MODULE_0__.useRef)(null);
   var touchOffset = (0,react__WEBPACK_IMPORTED_MODULE_0__.useRef)({
     x: 0,
     y: 0
   });
   var currentHoverItem = (0,react__WEBPACK_IMPORTED_MODULE_0__.useRef)(null);
+  var rafId = (0,react__WEBPACK_IMPORTED_MODULE_0__.useRef)(null);
+  var lastUpdateDragIndex = (0,react__WEBPACK_IMPORTED_MODULE_0__.useRef)(null);
+  var lastUpdateDropIndex = (0,react__WEBPACK_IMPORTED_MODULE_0__.useRef)(null);
+
+  /**
+   * Performance Note:
+   *
+   * Drag-over events can fire at a very high frequency, especially on touch devices
+   * or when dragging quickly. Directly performing DOM read/write operations in the
+   * event handler (e.g. `getBoundingClientRect`, `classList` changes, style updates)
+   * can easily cause layout thrashing and frame drops when there are many items.
+   *
+   * To mitigate this, we:
+   * - Collect the pointer coordinates synchronously in the event handler.
+   * - Schedule all DOM-intensive work inside `requestAnimationFrame`, so the browser
+   *   batches these operations before the next paint.
+   * - Cancel any pending frame (`cancelAnimationFrame`) before scheduling a new one,
+   *   ensuring there is at most one pending DOM update per frame.
+   *
+   * This keeps drag interactions smooth even with large lists.
+   */
+
   var handleDragStart = function handleDragStart(e, position) {
     var isTouch = ('touches' in e);
     var target = e.target;
@@ -285,69 +310,127 @@ var useBoundedDrag = function useBoundedDrag() {
         opacity: '0.9'
       });
       document.body.appendChild(dragNode.current);
+
+      // Keep track of the original element (acts as a placeholder inside the list)
+      draggedElement.current = listItem;
+      boundaryElement.current = boundary;
       setIsDragging(true);
       listItem.classList.add('dragging-placeholder');
     } else {
-      // ... desktop drag logic remains the same ...
+      // Desktop: use native drag image, but still record dragged element / boundary
+      draggedElement.current = listItem;
+      boundaryElement.current = boundary;
+      setIsDragging(true);
+      var dragEvent = e;
+      if (dragEvent.dataTransfer) {
+        dragEvent.dataTransfer.effectAllowed = 'move';
+        // Optional: customize drag preview if needed
+        dragEvent.dataTransfer.setData('text/plain', '');
+      }
+      listItem.classList.add('dragging-placeholder');
     }
   };
   var handleDragOver = function handleDragOver(e) {
+    // Always prevent default synchronously
     e.preventDefault();
     var isTouch = ('touches' in e);
     if (!isTouch) {
       e.dataTransfer.dropEffect = 'move';
     }
 
-    // Get the current pointer/touch position
-    var point = isTouch ? e.touches[0] : {
-      clientX: e.clientX,
-      clientY: e.clientY
-    };
+    // Extract primitive coordinates synchronously to avoid using pooled events in async callbacks
+    var clientX;
+    var clientY;
+    if (isTouch) {
+      var touch = e.touches[0];
+      clientX = touch.clientX;
+      clientY = touch.clientY;
+    } else {
+      clientX = e.clientX;
+      clientY = e.clientY;
+    }
 
-    // Update dragged element position for touch events
-    if (isTouch && isDragging && dragNode.current) {
-      dragNode.current.style.left = "".concat(point.clientX - touchOffset.current.x, "px");
-      dragNode.current.style.top = "".concat(point.clientY - touchOffset.current.y, "px");
+    // Cancel any pending frame to avoid stacking DOM operations
+    if (rafId.current !== null) {
+      cancelAnimationFrame(rafId.current);
     }
+    rafId.current = requestAnimationFrame(function () {
+      // Update dragged element position for touch events
+      if (isTouch && isDragging && dragNode.current) {
+        dragNode.current.style.left = "".concat(clientX - touchOffset.current.x, "px");
+        dragNode.current.style.top = "".concat(clientY - touchOffset.current.y, "px");
+      }
 
-    // Find the element below the pointer/touch
-    var elemBelow = document.elementFromPoint(point.clientX, point.clientY);
-    if (!elemBelow) return;
+      // Find the element below the pointer/touch
+      var elemBelow = document.elementFromPoint(clientX, clientY);
+      if (!elemBelow) return;
 
-    // Find the closest list item
-    var listItem = elemBelow.closest(itemSelector);
-    if (!listItem || listItem === currentHoverItem.current) return;
+      // Find the closest list item
+      var listItem = elemBelow.closest(itemSelector);
+      if (!listItem) return;
 
-    // Check boundary
-    var boundary = listItem.closest(boundarySelector);
-    if (!boundary) return;
+      // Check boundary
+      var boundary = boundaryElement.current || listItem.closest(boundarySelector);
+      if (!boundary) return;
 
-    // Update hover states
-    if (currentHoverItem.current) {
-      currentHoverItem.current.classList.remove('drag-over', 'drag-over-top', 'drag-over-bottom');
-    }
-    currentHoverItem.current = listItem;
-    listItem.classList.add('drag-over');
-
-    // Calculate position in list
-    var position = Array.from(listItem.parentNode.children).indexOf(listItem);
-    dragOverItem.current = position;
-
-    // Determine drop position (top/bottom)
-    var rect = listItem.getBoundingClientRect();
-    var middleY = rect.top + rect.height / 2;
-    if (point.clientY < middleY) {
-      listItem.classList.add('drag-over-top');
-    } else {
-      listItem.classList.add('drag-over-bottom');
-    }
-    onDragOver === null || onDragOver === void 0 ? void 0 : onDragOver(dragItem.current, dragOverItem.current);
+      // Update hover states
+      if (currentHoverItem.current && currentHoverItem.current !== listItem) {
+        currentHoverItem.current.classList.remove('drag-over', 'drag-over-top', 'drag-over-bottom');
+      }
+      currentHoverItem.current = listItem;
+      listItem.classList.add('drag-over');
+      var dragEl = draggedElement.current;
+      if (!dragEl || !dragEl.parentNode) return;
+      var container = boundary;
+
+      // Collect current ordered items in the container
+      var children = Array.from(container.querySelectorAll(itemSelector));
+      var currentIndex = children.indexOf(dragEl);
+      var targetIndex = children.indexOf(listItem);
+      if (currentIndex === -1 || targetIndex === -1) return;
+
+      // Determine drop position (top/bottom)
+      var rect = listItem.getBoundingClientRect();
+      var middleY = rect.top + rect.height / 2;
+      listItem.classList.remove('drag-over-top', 'drag-over-bottom');
+      var insertBefore = clientY < middleY ? listItem : listItem.nextElementSibling;
+      if (clientY < middleY) {
+        listItem.classList.add('drag-over-top');
+      } else {
+        listItem.classList.add('drag-over-bottom');
+      }
+
+      // Only move in DOM when the effective position changes
+      if (insertBefore !== dragEl && container.contains(dragEl)) {
+        container.insertBefore(dragEl, insertBefore);
+      }
+
+      // Recompute index after DOM move
+      var reorderedChildren = Array.from(container.querySelectorAll(itemSelector));
+      var newIndex = reorderedChildren.indexOf(dragEl);
+      dragOverItem.current = newIndex;
+      onDragOver === null || onDragOver === void 0 ? void 0 : onDragOver(dragItem.current, dragOverItem.current);
+
+      // Only fire onDragUpdate when the (dragIndex, dropIndex) pair actually changes.
+      if (onDragUpdate && (dragItem.current !== lastUpdateDragIndex.current || dragOverItem.current !== lastUpdateDropIndex.current)) {
+        lastUpdateDragIndex.current = dragItem.current;
+        lastUpdateDropIndex.current = dragOverItem.current;
+        onDragUpdate(dragItem.current, dragOverItem.current);
+      }
+      rafId.current = null;
+    });
   };
   var handleDragEnd = function handleDragEnd(e) {
     var isTouch = ('touches' in e);
     if (isTouch && !isDragging) return;
     onDragEnd === null || onDragEnd === void 0 ? void 0 : onDragEnd(dragItem.current, dragOverItem.current);
 
+    // Cancel any pending animation frame
+    if (rafId.current !== null) {
+      cancelAnimationFrame(rafId.current);
+      rafId.current = null;
+    }
+
     // Cleanup
     if (dragNode.current) {
       dragNode.current.remove();
@@ -361,6 +444,8 @@ var useBoundedDrag = function useBoundedDrag() {
     currentHoverItem.current = null;
     dragItem.current = null;
     dragOverItem.current = null;
+    draggedElement.current = null;
+    boundaryElement.current = null;
   };
   return {
     isDragging: isDragging,

package/lib/css/Table/index.css

@@ -7,7 +7,7 @@
   --syntable-row-active-bg: #f0f8ff;
   --syntable-scrollbar-color: rgba(0, 0, 0, 0.2);
   --syntable-scrollbar-track: rgba(0, 0, 0, 0);
-  --syntable-scrollbar-h: 3px;
+  --syntable-scrollbar-h: 10px;
   --syntable-scrollbar-w: 10px;
   --syntable-padding-x: 1rem;
   --syntable-padding-y: 0.5rem;
@@ -414,10 +414,13 @@
 }
 .syntable__wrapper.allow-dragdrop tbody .row-obj-clonelast {
   height: 0 !important;
+  border-color: transparent !important;
+  padding: 0 !important;
 }
 .syntable__wrapper.allow-dragdrop tbody .row-obj-clonelast td {
   border: none;
   box-shadow: none;
+  padding: 0 !important;
 }
 .syntable__wrapper.allow-dragdrop tbody td,
 .syntable__wrapper.allow-dragdrop tbody th {
@@ -428,6 +431,7 @@
 .syntable__wrapper.allow-dragdrop tbody.drag-trigger-mousedown th {
   pointer-events: auto;
 }
+.syntable__wrapper.allow-dragdrop .row-obj-lastplaceholder,
 .syntable__wrapper.allow-dragdrop .row-placeholder {
   border: 2px dotted #b5ba91;
   background-color: #e4e9c3;

package/lib/esm/DragDropList/index.tsx

@@ -8,7 +8,6 @@ import { clsWrite, combinedCls } from 'funda-utils/dist/cjs/cls';
 import useBoundedDrag from 'funda-utils/dist/cjs/useBoundedDrag';
 
 
-
 export interface ListItem {
     id: number;
     parentId?: number;
@@ -19,6 +18,7 @@ export interface ListItem {
     depth?: number;
     children?: ListItem[];
     disabled?: boolean;
+    itemDraggable?: boolean;
     appendControl?: React.ReactNode;
     [key: string]: any;
 }
@@ -222,6 +222,10 @@ const DragDropList = forwardRef((props: DragDropListProps, externalRef: any) =>
         onDragOver: (dragIndex: number | null, dropIndex: number | null) => {
             // Additional drag over logic if needed
         },
+        onDragUpdate: (dragIndex: number | null, dropIndex: number | null) => {
+            // console.log(dragIndex, dropIndex);
+          
+        },
         onDragEnd: (dragIndex: number | null, dropIndex: number | null) => {
             if (dragIndex !== null && dropIndex !== null && dragIndex !== dropIndex) {
                 // Handle item movement
@@ -251,12 +255,12 @@ const DragDropList = forwardRef((props: DragDropListProps, externalRef: any) =>
                 });
 
                 // Calculate new insert position
-                let insertIndex = dropIndex;
-                if (dropIndex > dragIndex) {
-                    insertIndex -= itemsToMove.length;
-                }
+                // Directly use dropIndex as the insertion position to avoid items snapping back
+                // when dragging an item from above to directly below its neighbor.
+                const insertIndex = dropIndex;
 
-                // Insert all items
+                // Insert all items (remove first, then insert at the target index;
+                // JavaScript's splice will handle index shifting automatically).
                 newItems.splice(insertIndex, 0, ...itemsBeingMoved);
 
                 // Rebuild tree structure
@@ -397,6 +401,9 @@ const DragDropList = forwardRef((props: DragDropListProps, externalRef: any) =>
                 // If the item should be hidden, the rendering is skipped
                 if (!shouldShowItem(item)) return null;
 
+                // Item level draggable control, default true when not specified
+                const isItemDraggable = draggable && item.itemDraggable !== false;
+
                 // collapse
                 const hasChildItems = hasChildren(item.id);
                 const isCollapsed = collapsedItems.has(item.id);
@@ -415,7 +422,7 @@ const DragDropList = forwardRef((props: DragDropListProps, externalRef: any) =>
                         clsWrite(dragMode, 'handle'),
                         {
                             'disabled': item.disabled,
-                            'draggable': draggable,
+                            'draggable': isItemDraggable,
                             'editing': editingItem === item.id,
 
                             // collapse
@@ -423,13 +430,13 @@ const DragDropList = forwardRef((props: DragDropListProps, externalRef: any) =>
                             'collapsed': isCollapsed
                         }
                     )}
-                    draggable={!draggable ? undefined : editingItem !== item.id && "true"}
-                    onDragStart={!draggable ? undefined : (e) => dragHandlers.handleDragStart(e, index)}
-                    onDragOver={!draggable ? undefined : dragHandlers.handleDragOver}
-                    onDragEnd={!draggable ? undefined : dragHandlers.handleDragEnd}
-                    onTouchStart={!draggable ? undefined : (e) => dragHandlers.handleDragStart(e, index)}
-                    onTouchMove={!draggable ? undefined : dragHandlers.handleDragOver}
-                    onTouchEnd={!draggable ? undefined : dragHandlers.handleDragEnd}
+                    draggable={!isItemDraggable ? undefined : editingItem !== item.id && "true"}
+                    onDragStart={!isItemDraggable ? undefined : (e) => dragHandlers.handleDragStart(e, index)}
+                    onDragOver={!isItemDraggable ? undefined : dragHandlers.handleDragOver}
+                    onDragEnd={!isItemDraggable ? undefined : dragHandlers.handleDragEnd}
+                    onTouchStart={!isItemDraggable ? undefined : (e) => dragHandlers.handleDragStart(e, index)}
+                    onTouchMove={!isItemDraggable ? undefined : dragHandlers.handleDragOver}
+                    onTouchEnd={!isItemDraggable ? undefined : dragHandlers.handleDragEnd}
                     style={itemStyle}
                     onDoubleClick={() => handleDoubleClick(item)}
                 >
@@ -437,14 +444,14 @@ const DragDropList = forwardRef((props: DragDropListProps, externalRef: any) =>
                         {renderOption ? (
                             renderOption(
                                 item,
-                                `${prefix}-draggable-list__handle`,
+                                isItemDraggable ? `${prefix}-draggable-list__handle` : '',
                                 index
                             )
                         ) : (
                             <>
                                 {/** DRAG HANDLE */}
                                 {/* Fix the problem that mobile terminals cannot be touched, DO NOT USE "<svg>" */}
-                                {draggable && !handleHide ? <span className={`${prefix}-draggable-list__handle ${handlePos ?? 'left'}`} draggable={dragMode === 'handle'} dangerouslySetInnerHTML={{
+                                {isItemDraggable && !handleHide ? <span className={`${prefix}-draggable-list__handle ${handlePos ?? 'left'}`} draggable={dragMode === 'handle'} dangerouslySetInnerHTML={{
                                     __html: `${handleIcon}`
                                 }}></span> : null}
                                 {/** /DRAG HANDLE */}

package/lib/esm/Table/Table.tsx

@@ -5,6 +5,7 @@ import useComId from 'funda-utils/dist/cjs/useComId';
 import { clsWrite, combinedCls } from 'funda-utils/dist/cjs/cls';
 
 
+
 import { TableProvider } from './TableContext';
 import useTableResponsive from './utils/hooks/useTableResponsive';
 import useTableDraggable from './utils/hooks/useTableDraggable';
@@ -148,10 +149,10 @@ const Table = forwardRef<HTMLDivElement, TableProps>((
 
     // initialize drag & drop data
     const {
-        handleDragStart,
-        handleDragEnd,
-        handledragOver,
-        handleTbodyEnter
+        isDragging,
+        dragHandlers,
+        handleTbodyEnter,
+        handleTbodyLeave
     } = useTableDraggable({
         enabled: rowDraggable && rootRef.current,
         data: data,
@@ -268,9 +269,10 @@ const Table = forwardRef<HTMLDivElement, TableProps>((
 
                 // drag & drop
                 rowDraggable,
-                handleDragStart,
-                handleDragEnd,
-                handledragOver,
+                isRowDragging: isDragging,
+                handleDragStart: dragHandlers.handleDragStart,
+                handleDragEnd: dragHandlers.handleDragEnd,
+                handledragOver: dragHandlers.handleDragOver,
                 handleTbodyEnter,
                 
 

package/lib/esm/Table/TableRow.tsx

@@ -44,9 +44,15 @@ const TableRow = forwardRef<HTMLTableRowElement, TableRowProps>((
     // selection
     // ================================================================
     const _res = convertMapToArr(selectedItems);
+    // Performance optimization: stringify itemData only once instead of N times
+    const itemDataStr = itemData ? JSON.stringify(itemData) : '';
     const filteredSelectedItems = _res.map((v: any) => Number(v)).map((rowNum: number) => {
-        if (JSON.stringify(itemData) === JSON.stringify(originData[rowNum])) {
-            return originData[rowNum];
+        const originItem = originData?.[rowNum];
+        // Fast path: reference equality
+        if (itemData === originItem) return originItem;
+        // Fallback: JSON comparison (itemDataStr is cached)
+        if (itemDataStr && itemDataStr === JSON.stringify(originItem)) {
+            return originItem;
         }
     }).filter(Boolean);
     const selectedClassName = activeClassName || 'active';
@@ -64,7 +70,7 @@ const TableRow = forwardRef<HTMLTableRowElement, TableRowProps>((
                 onDragEnd={rowDraggable ? handleDragEnd : () => void(0)} 
                 data-row-data={`${itemData && typeof itemData === 'object' ? JSON.stringify(itemData) : itemData}`}
                 
-                className={`row-obj ${className || ''} ${active ? selectedClassName : ''} ${itemData && originData ? (filteredData.every((item: any) => filterFieldsData.some((s: string) => !item[s]?.toLowerCase().includes(itemData[s]?.toLowerCase())) ) ? 'd-none' : '') : ''} ${itemData && originData ? (filteredSelectedItems.some((item: any) => JSON.stringify(itemData) === JSON.stringify(item) ) ? selectedClassName : '') : ''}`}
+                className={`row-obj ${className || ''} ${active ? selectedClassName : ''} ${itemData && originData ? (filteredData.every((item: any) => filterFieldsData.some((s: string) => !item[s]?.toLowerCase().includes(itemData[s]?.toLowerCase())) ) ? 'd-none' : '') : ''} ${itemData && originData && filteredSelectedItems.length > 0 ? selectedClassName : ''}`}
             >
                 {children}
             </tr>

package/lib/esm/Table/index.scss

@@ -1,3 +1,5 @@
+
+
 /* ======================================================
     <!-- Table (Synthetic) -->
 /* ====================================================== */
@@ -7,7 +9,7 @@
     --syntable-row-active-bg: #f0f8ff;
     --syntable-scrollbar-color: rgba(0, 0, 0, 0.2);
     --syntable-scrollbar-track: rgba(0, 0, 0, 0);
-    --syntable-scrollbar-h: 3px;
+    --syntable-scrollbar-h: 10px;
     --syntable-scrollbar-w: 10px;
     --syntable-padding-x: 1rem;
     --syntable-padding-y: 0.5rem;
@@ -165,11 +167,13 @@
 
             .row-obj-clonelast {
                 height: 0 !important;
-
+                border-color: transparent !important;
+                padding: 0 !important;
              
                 td {
                     border: none;
                     box-shadow: none;
+                    padding: 0 !important;
                 }
             }
 
@@ -190,11 +194,13 @@
         }
 
         /* placeholder */
+        .row-obj-lastplaceholder,
         .row-placeholder {
             border: 2px dotted #b5ba91;
             background-color: #e4e9c3;
         }
         
+        
         /* trigger */
         .drag-trigger {
             display: inline-block;

package/lib/esm/Table/utils/DragHandleSprite.tsx

@@ -15,7 +15,8 @@ const DragHandleSprite = forwardRef((props: DragHandleSpriteProps, externalRef:
 
     const {
         rowDraggable,
-        handleTbodyEnter
+        handleTbodyEnter,
+        handleTbodyLeave
     } = useContext(TableContext);
 
     return (
@@ -23,7 +24,10 @@ const DragHandleSprite = forwardRef((props: DragHandleSpriteProps, externalRef:
             {rowDraggable ? <span
                 ref={externalRef}
                 className={className || 'drag-trigger'}
-                onMouseEnter={handleTbodyEnter}
+                // Only when mousedown happens on this handle will we allow row dragging.
+                onMouseDown={handleTbodyEnter}
+                onMouseUp={handleTbodyLeave}
+                onMouseLeave={handleTbodyLeave}
             >
                 {icon || <svg width="1em" height="1em" viewBox="0 0 24 24" fill="none">
                     <g>

package/lib/esm/Table/utils/func.ts

@@ -63,6 +63,14 @@ export function initOrderProps(rootElem: any) {
 
 export function initRowColProps(rootElem: any) {
     if (rootElem === null) return;
+
+    // !!! Important, performance optimization for large data renderings
+    // With this protection, it is only performed once
+    if (typeof rootElem.dataset.rowColPropsInit !== 'undefined') return;
+    rootElem.dataset.rowColPropsInit = '1';
+
+
+    //
     const _allRows = allRows(rootElem);
     const _allHeadRows = allHeadRows(rootElem);
 
@@ -167,7 +175,10 @@ export function cellMark(row: number | string | undefined, col: number | string
 
 export function removeCellFocusClassName(root: any) {
     if (root) {
-        [].slice.call(root.querySelectorAll('td, th')).forEach((el: HTMLTableElement) => {
+        // !!! Important, performance optimization for large data renderings
+        // Only query elements with cell-focus classes
+        const focusedCells = root.querySelectorAll('td.cell-focus, th.cell-focus');
+        focusedCells.forEach((el: HTMLElement) => {
             el.classList.remove('cell-focus');
         });
     }