From 10444b5571c4fc633acffa8f800beaebdd676a73 Mon Sep 17 00:00:00 2001
From: Nemo Shen <nemoshen94@gmail.com>
Date: Tue, 30 Jan 2024 09:17:44 +0800
Subject: [PATCH] fix(Sticky): initial sticky position (#12601)

---
 packages/vant/src/sticky/Sticky.tsx       |  3 +-
 packages/vant/src/tab/test/index.spec.tsx | 45 -----------------------
 2 files changed, 1 insertion(+), 47 deletions(-)

diff --git a/packages/vant/src/sticky/Sticky.tsx b/packages/vant/src/sticky/Sticky.tsx
index 3b80e6300..4a04d6087 100644
--- a/packages/vant/src/sticky/Sticky.tsx
+++ b/packages/vant/src/sticky/Sticky.tsx
@@ -119,8 +119,7 @@ export default defineComponent({
         if (container) {
           const containerRect = useRect(container);
           const difference = containerRect.bottom - offset.value - state.height;
-          state.fixed =
-            offset.value >= rootRect.top && containerRect.bottom > 0;
+          state.fixed = offset.value > rootRect.top && containerRect.bottom > 0;
           state.transform = difference < 0 ? difference : 0;
         } else {
           state.fixed = offset.value > rootRect.top;
diff --git a/packages/vant/src/tab/test/index.spec.tsx b/packages/vant/src/tab/test/index.spec.tsx
index 0c5903a89..7597cdb53 100644
--- a/packages/vant/src/tab/test/index.spec.tsx
+++ b/packages/vant/src/tab/test/index.spec.tsx
@@ -437,51 +437,6 @@ test('should not render header when showHeader is false', async () => {
   expect(tabs.length).toEqual(0);
 });
 
-test('should fixed when sticky offset is equal top', async () => {
-  const wrapper = mount({
-    render() {
-      return (
-        <Tabs sticky>
-          <Tab title="title1">Text</Tab>
-          <Tab title="title2">Text</Tab>
-        </Tabs>
-      );
-    },
-  });
-
-  const mockContainerRect = vi
-    .spyOn(wrapper.element, 'getBoundingClientRect')
-    .mockReturnValue({
-      bottom: 10,
-    } as DOMRect);
-  const container = wrapper.element.children[0];
-  const mockStickyRect = vi
-    .spyOn(container, 'getBoundingClientRect')
-    .mockReturnValue({
-      top: -10,
-    } as DOMRect);
-
-  expect(wrapper.find('.van-sticky').classes()).toStrictEqual(['van-sticky']);
-
-  await mockScrollTop(100);
-  expect(wrapper.find('.van-sticky').classes()).toStrictEqual([
-    'van-sticky',
-    'van-sticky--fixed',
-  ]);
-
-  mockStickyRect.mockReturnValue({
-    top: 0,
-  } as DOMRect);
-  await mockScrollTop(100);
-  expect(wrapper.find('.van-sticky').classes()).toStrictEqual([
-    'van-sticky',
-    'van-sticky--fixed',
-  ]);
-
-  mockStickyRect.mockRestore();
-  mockContainerRect.mockRestore();
-});
-
 test('should call before-change prop before changing', async () => {
   const onChange = vi.fn();
   const beforeChange = (name: number) => {