fix: final load jump when top is true (#71)

* Fix a bug in InfiniteLoading.vue that causes the final load to jump to the top of the elements when `top` is `true`.

* Updated InfiniteLoading.vue to make the `complete` function async.

* Moved the scroll position logic to a utility function.

Author: lanmaster53

src/components/InfiniteLoading.vue

@@ -1,7 +1,7 @@
 <script lang="ts" setup>
 import type { Props, Params, State, StateHandler } from "@root/types";
 import { onMounted, ref, toRefs, onUnmounted, watch, nextTick } from "vue";
-import { startObserver, getParentEl, isVisible } from "@root/utils";
+import { startObserver, getParentEl, isVisible, updateScrollPosition } from "@root/utils";
 // @ts-ignore
 import Spinner from "./Spinner.vue";
 
@@ -44,13 +44,12 @@ const stateHandler: StateHandler = {
   },
   async loaded() {
     state.value = "loaded";
-    const parentEl = params.parentEl || document.documentElement;
-    await nextTick();
-    if (top) parentEl.scrollTop = parentEl.scrollHeight - prevHeight;
+    if (top) updateScrollPosition(params, prevHeight);
     if (isVisible(infiniteLoading.value!, params.parentEl)) params.emit();
   },
-  complete() {
+  async complete() {
     state.value = "complete";
+    if (top) updateScrollPosition(params, prevHeight);
     observer?.disconnect();
   },
   error() {

src/utils.ts

@@ -32,4 +32,10 @@ function startObserver(params: Params) {
   return observer;
 }
 
-export { startObserver, isVisible, getParentEl };
+async function updateScrollPosition(params: Params, prevHeight: number) {
+  const parentEl = params.parentEl || document.documentElement;
+  await nextTick();
+  parentEl.scrollTop = parentEl.scrollHeight - prevHeight;
+}
+
+export { startObserver, isVisible, getParentEl, updateScrollPosition };