@linzjs/step-ag-grid 17.4.3 → 17.4.5

package/dist/step-ag-grid.esm.js

@@ -2749,7 +2749,7 @@ const Grid = ({ "data-testid": dataTestId, rowSelection = "multiple", suppressCo
     }, []);
     const onRowDragEnd = useCallback(async (event) => {
         const clientSideRowModel = event.api.getModel();
-        if (event.node.rowIndex) {
+        if (event.node.rowIndex != null) {
             const lastHighlightedRowNode = clientSideRowModel.getLastHighlightedRowNode();
             const isBelow = lastHighlightedRowNode && lastHighlightedRowNode.highlighted === RowHighlightPosition.Below;
             let targetIndex = event.overIndex;
@@ -2762,7 +2762,7 @@ const Grid = ({ "data-testid": dataTestId, rowSelection = "multiple", suppressCo
             const moved = event.node.data;
             const target = event.overNode?.data;
             moved.id != target.id && //moved over a different row
-                moved.rowIndex != targetIndex && //moved to a different index
+                event.node.rowIndex != targetIndex && //moved to a different index
                 params.onRowDragEnd &&
                 (await params.onRowDragEnd(moved, target, targetIndex));
         }
@@ -3569,7 +3569,7 @@ const GridFormDropDown = (props) => {
     }, [filter, filteredValues, options, props, selectItemHandler, selectedItem, selectedRows, subSelectedValue]);
     const { popoverWrapper } = useGridPopoverHook({
         className: props.className,
-        invalid: () => !!(selectedItem && !subComponentIsValid.current),
+        invalid: () => !options || !!(selectedItem && !subComponentIsValid.current),
         save,
         dontSaveOnExternalClick: true,
     });