fix(useThrottleFn)!: align with traditional throttle behavior (#4923)

Co-authored-by: IlyaL <[email protected]>

Author: IceMooncake

packages/shared/useThrottleFn/index.test.ts

@@ -36,17 +36,37 @@ describe('useThrottleFn', () => {
   it('should work with leading', async () => {
     const callback = vi.fn()
     const ms = 20
-    const run = useThrottleFn(callback, ms, false, false)
+    const run = useThrottleFn(callback, ms, false, true)
+    run()
+    expect(callback).toHaveBeenCalledTimes(1)
     run()
     run()
     expect(callback).toHaveBeenCalledTimes(1)
     vi.advanceTimersByTime(ms + 10)
     run()
+    expect(callback).toHaveBeenCalledTimes(2)
     run()
     run()
     expect(callback).toHaveBeenCalledTimes(2)
+    vi.advanceTimersByTime(ms + 10)
+    run()
+    expect(callback).toHaveBeenCalledTimes(3)
+  })
+
+  it('should work with not leading and not trailing', async () => {
+    const callback = vi.fn()
+    const ms = 20
+    const run = useThrottleFn(callback, ms, false, false)
+    run()
+    run()
+    expect(callback).toHaveBeenCalledTimes(0)
+    vi.advanceTimersByTime(ms + 10)
+    run()
+    run()
+    run()
+    expect(callback).toHaveBeenCalledTimes(0)
     vi.advanceTimersByTime(ms + 20)
     run()
-    expect(callback).toHaveBeenCalledTimes(2)
+    expect(callback).toHaveBeenCalledTimes(0)
   })
 })

packages/shared/utils/filters.ts

@@ -179,10 +179,10 @@ export function throttleFilter(...args: any[]) {
       lastExec = Date.now()
       return invoke()
     }
-
-    if (elapsed > duration && (leading || !isLeading)) {
+    if (elapsed > duration) {
       lastExec = Date.now()
-      invoke()
+      if (leading || !isLeading)
+        invoke()
     }
     else if (trailing) {
       lastValue = new Promise((resolve, reject) => {