From 11890ef3c1df0ace917ed3ec2b1f3b313589793d Mon Sep 17 00:00:00 2001
From: chenjiahan <chenjiahan@youzan.com>
Date: Sat, 26 Sep 2020 11:11:30 +0800
Subject: [PATCH] chore(Tabs): use relation

---
 src/tab/index.js  |  8 ++-----
 src/tabs/index.js | 55 ++++++++++++++++++++++++-----------------------
 2 files changed, 30 insertions(+), 33 deletions(-)

diff --git a/src/tab/index.js b/src/tab/index.js
index 7b793a79f..87de18f9a 100644
--- a/src/tab/index.js
+++ b/src/tab/index.js
@@ -4,7 +4,7 @@ import { TABS_KEY } from '../tabs';
 
 // Composition
 import { routeProps } from '../composition/use-route';
-import { useParent } from '../composition/use-parent';
+import { useParent } from '../composition/use-relation';
 
 const [createComponent, bem] = createNamespace('tab');
 
@@ -22,11 +22,7 @@ export default createComponent({
   setup(props, { slots }) {
     const root = ref();
     const inited = ref(false);
-    const { parent, index } = useParent(TABS_KEY, {
-      getRoot: () => root.value,
-      props,
-      slots,
-    });
+    const { parent, index } = useParent(TABS_KEY);
 
     if (!parent) {
       throw new Error('[Vant] Tabs: <van-tab> must be used inside <van-tabs>');
diff --git a/src/tabs/index.js b/src/tabs/index.js
index f427f6733..30a620ddb 100644
--- a/src/tabs/index.js
+++ b/src/tabs/index.js
@@ -1,7 +1,6 @@
 import {
   ref,
   watch,
-  provide,
   computed,
   reactive,
   nextTick,
@@ -25,9 +24,10 @@ import {
 } from '../utils/dom/scroll';
 
 // Composition
+import { useWindowSize, useScrollParent, useEventListener } from '@vant/use';
 import { useRefs } from '../composition/use-refs';
 import { useExpose } from '../composition/use-expose';
-import { useWindowSize, useScrollParent, useEventListener } from '@vant/use';
+import { useChildren } from '../composition/use-relation';
 
 // Components
 import Sticky from '../sticky';
@@ -97,8 +97,8 @@ export default createComponent({
     const windowSize = useWindowSize();
     const scroller = useScrollParent(root);
     const [titleRefs, setTitleRefs] = useRefs();
+    const { children, linkChildren } = useChildren(TABS_KEY);
 
-    const children = reactive([]);
     const state = reactive({
       position: '',
       currentIndex: -1,
@@ -117,7 +117,7 @@ export default createComponent({
       background: props.background,
     }));
 
-    const getTabName = (tab, index) => tab.props.name ?? index;
+    const getTabName = (tab, index) => tab.name ?? index;
 
     const currentName = computed(() => {
       const activeTab = children[state.currentIndex];
@@ -203,7 +203,7 @@ export default createComponent({
       const diff = index < state.currentIndex ? -1 : 1;
 
       while (index >= 0 && index < children.length) {
-        if (!children[index].props.disabled) {
+        if (!children[index].disabled) {
           return index;
         }
 
@@ -220,7 +220,7 @@ export default createComponent({
         emit('update:active', currentName.value);
 
         if (shouldEmitChange) {
-          emit('change', currentName.value, children[currentIndex].props.title);
+          emit('change', currentName.value, children[currentIndex].title);
         }
       }
     };
@@ -237,11 +237,10 @@ export default createComponent({
 
     const scrollToCurrentContent = (immediate = false) => {
       if (props.scrollspy) {
-        const target = children[state.currentIndex];
-        const el = target?.getRoot();
+        const target = children[state.currentIndex].$el;
 
-        if (el) {
-          const to = getElementTop(el, scroller.value) - scrollOffset.value;
+        if (target) {
+          const to = getElementTop(target, scroller.value) - scrollOffset.value;
 
           lockScroll = true;
           scrollTopTo(
@@ -258,7 +257,7 @@ export default createComponent({
 
     // emit event when clicked
     const onClick = (item, index) => {
-      const { title, disabled } = children[index].props;
+      const { title, disabled } = children[index];
       const name = getTabName(children[index], index);
 
       if (disabled) {
@@ -292,7 +291,7 @@ export default createComponent({
 
     const getCurrentIndexOnScroll = () => {
       for (let index = 0; index < children.length; index++) {
-        const top = getVisibleTop(children[index].getRoot());
+        const top = getVisibleTop(children[index].$el);
 
         if (top > scrollOffset.value) {
           return index === 0 ? 0 : index - 1;
@@ -313,16 +312,16 @@ export default createComponent({
       children.map((item, index) => (
         <TabsTitle
           ref={setTitleRefs(index)}
-          dot={item.props.dot}
+          dot={item.dot}
           type={props.type}
-          badge={item.props.badge}
-          title={item.props.title}
+          badge={item.badge}
+          title={item.title}
           color={props.color}
-          style={item.props.titleStyle}
+          style={item.titleStyle}
           isActive={index === state.currentIndex}
-          disabled={item.props.disabled}
+          disabled={item.disabled}
           scrollable={scrollable.value}
-          renderTitle={item.slots.title}
+          renderTitle={item.$slots.title}
           activeColor={props.titleActiveColor}
           inactiveColor={props.titleInactiveColor}
           onClick={() => {
@@ -369,13 +368,16 @@ export default createComponent({
       }
     );
 
-    watch(children, () => {
-      setCurrentIndexByName(currentName.value || props.active);
-      setLine();
-      nextTick(() => {
-        scrollIntoView(true);
-      });
-    });
+    watch(
+      () => children.length,
+      () => {
+        setCurrentIndexByName(currentName.value || props.active);
+        setLine();
+        nextTick(() => {
+          scrollIntoView(true);
+        });
+      }
+    );
 
     watch(
       () => state.currentIndex,
@@ -406,11 +408,10 @@ export default createComponent({
 
     useEventListener('scroll', onScroll, { target: scroller.value });
 
-    provide(TABS_KEY, {
+    linkChildren({
       emit,
       props,
       setLine,
-      children,
       currentName,
     });