ng-virtual-list 14.3.0 → 14.3.2

package/fesm2020/ng-virtual-list.mjs

@@ -846,14 +846,14 @@ class TrackBox extends CacheMap {
      */
     getItemPosition(id, stickyMap, options) {
         const opt = { fromItemId: id, stickyMap, ...options };
-        const { scrollSize } = this.recalculateMetrics({
+        const { scrollSize, isFromItemIdFound } = this.recalculateMetrics({
             ...opt,
             dynamicSize: this._crudDetected || opt.dynamicSize,
             previousTotalSize: this._previousTotalSize,
             crudDetected: this._crudDetected,
             deletedItemsMap: this._deletedItemsMap,
         });
-        return scrollSize;
+        return isFromItemIdFound ? scrollSize : -1;
     }
     /**
      * Updates the collection of display objects
@@ -962,7 +962,7 @@ class TrackBox extends CacheMap {
         else {
             leftItemsOffset = rightItemsOffset = itemsOffset;
         }
-        let itemsFromStartToScrollEnd = -1, itemsFromDisplayEndToOffsetEnd = 0, itemsFromStartToDisplayEnd = -1, leftItemLength = 0, rightItemLength = 0, leftItemsWeight = 0, rightItemsWeight = 0, leftHiddenItemsWeight = 0, totalItemsToDisplayEndWeight = 0, leftSizeOfAddedItems = 0, leftSizeOfUpdatedItems = 0, leftSizeOfDeletedItems = 0, itemById = undefined, itemByIdPos = 0, targetDisplayItemIndex = -1, isTargetInOverscroll = false, actualScrollSize = itemByIdPos, totalSize = 0, startIndex;
+        let itemsFromStartToScrollEnd = -1, itemsFromDisplayEndToOffsetEnd = 0, itemsFromStartToDisplayEnd = -1, leftItemLength = 0, rightItemLength = 0, leftItemsWeight = 0, rightItemsWeight = 0, leftHiddenItemsWeight = 0, totalItemsToDisplayEndWeight = 0, leftSizeOfAddedItems = 0, leftSizeOfUpdatedItems = 0, leftSizeOfDeletedItems = 0, itemById = undefined, itemByIdPos = 0, targetDisplayItemIndex = -1, isTargetInOverscroll = false, actualScrollSize = itemByIdPos, totalSize = 0, startIndex, isFromItemIdFound = false;
         // If the list is dynamic or there are new elements in the collection, then it switches to the long algorithm.
         if (dynamicSize) {
             let y = 0, stickyCollectionItem = undefined, stickyComponentSize = 0;
@@ -1002,6 +1002,7 @@ class TrackBox extends CacheMap {
                             stickyCollectionItem = collectionItem;
                         }
                         if (id === fromItemId) {
+                            isFromItemIdFound = true;
                             targetDisplayItemIndex = i;
                             if (stickyCollectionItem && stickyMap) {
                                 const { num } = this.getElementNumToEnd(i, collection, map, typicalItemSize, size, isVertical);
@@ -1108,7 +1109,7 @@ class TrackBox extends CacheMap {
                         }
                     }
                     if (deletedItemsMap.hasOwnProperty(i)) {
-                        const bounds = deletedItemsMap[i], size = bounds[sizeProperty] ?? typicalItemSize;
+                        const bounds = deletedItemsMap[i], size = bounds?.[sizeProperty] ?? typicalItemSize;
                         if (y < scrollSize - size) {
                             leftSizeOfDeletedItems += size;
                         }
@@ -1177,6 +1178,7 @@ class TrackBox extends CacheMap {
             totalLength,
             totalSize,
             typicalItemSize,
+            isFromItemIdFound,
         };
         return metrics;
     }
@@ -1199,6 +1201,9 @@ class TrackBox extends CacheMap {
             let pos = startPosition, renderItems = renderItemsLength, stickyItem, nextSticky, stickyItemIndex = -1, stickyItemSize = 0, endStickyItem, nextEndSticky, endStickyItemIndex = -1, endStickyItemSize = 0;
             if (snap) {
                 for (let i = Math.min(itemsFromStartToScrollEnd > 0 ? itemsFromStartToScrollEnd : 0, totalLength - 1); i >= 0; i--) {
+                    if (!items[i]) {
+                        continue;
+                    }
                     const id = items[i].id, sticky = stickyMap[id], size = dynamicSize ? this.get(id)?.[sizeProperty] || typicalItemSize : typicalItemSize;
                     if (sticky === 1) {
                         const measures = {
@@ -1938,6 +1943,10 @@ class NgVirtualListComponent extends DisposableComponent {
                     itemsOffset: this.itemsOffset, scrollSize: (isVertical ? container.nativeElement.scrollTop : container.nativeElement.scrollLeft) + delta,
                     snap: this.snap, fromItemId: id, enabledBufferOptimization: this.enabledBufferOptimization,
                 }, scrollSize = this._trackBox.getItemPosition(id, stickyMap, opts), params = { [isVertical ? TOP_PROP_NAME : LEFT_PROP_NAME]: scrollSize, behavior };
+                if (scrollSize === -1) {
+                    container.nativeElement.addEventListener(SCROLL, this._onScrollHandler);
+                    return;
+                }
                 this._trackBox.clearDelta();
                 if (container) {
                     const { displayItems, totalSize } = this._trackBox.updateCollection(items, stickyMap, {
@@ -1949,6 +1958,10 @@ class NgVirtualListComponent extends DisposableComponent {
                     this.createDisplayComponentsIfNeed(displayItems);
                     this.tracking();
                     const _scrollSize = this._trackBox.getItemPosition(id, stickyMap, { ...opts, scrollSize: actualScrollSize, fromItemId: id });
+                    if (_scrollSize === -1) {
+                        container.nativeElement.addEventListener(SCROLL, this._onScrollHandler);
+                        return;
+                    }
                     const notChanged = actualScrollSize === _scrollSize;
                     if (!notChanged || iteration < MAX_SCROLL_TO_ITERATIONS) {
                         this.clearScrollToRepeatExecutionTimeout();
@@ -1965,9 +1978,12 @@ class NgVirtualListComponent extends DisposableComponent {
                 this._$scrollSize.next(scrollSize);
             }
             else {
-                const index = items.findIndex(item => item.id === id), scrollSize = index * this.itemSize;
-                const params = { [this._isVertical ? TOP_PROP_NAME : LEFT_PROP_NAME]: scrollSize, behavior };
-                container.nativeElement.scrollTo(params);
+                const index = items.findIndex(item => item.id === id);
+                if (index > -1) {
+                    const scrollSize = index * this.itemSize;
+                    const params = { [this._isVertical ? TOP_PROP_NAME : LEFT_PROP_NAME]: scrollSize, behavior };
+                    container.nativeElement.scrollTo(params);
+                }
             }
         }
     }