From bd886d85e8bae59838c3def800c830a8898e3ef2 Mon Sep 17 00:00:00 2001
From: 07akioni <07akioni2@gmail.com>
Date: Fri, 6 Dec 2019 15:04:08 +0800
Subject: [PATCH] refactor(anchor): make last anchor inactive when it leaves
 screen

---
 .../components/anchor/enUS/placeholder.md     |  2 +-
 .../components/anchor/enUS/theCaretaker.md    |  1 -
 .../components/anchor/enUS/theColonel.md      |  1 -
 .../components/anchor/enUS/theGatekeeper.md   |  1 -
 .../components/anchor/enUS/theLibrarian.md    |  1 -
 .../components/anchor/enUS/theNarrator.md     |  1 -
 .../anchor/enUS/theNarratorSShadow.md         |  1 -
 packages/common/Anchor/src/AnchorAdapter.vue  |  8 ++--
 .../Anchor/src/{Anchor.vue => BaseAnchor.vue} | 44 +++++++++----------
 packages/common/Anchor/src/Link.vue           |  1 -
 10 files changed, 26 insertions(+), 35 deletions(-)
 rename packages/common/Anchor/src/{Anchor.vue => BaseAnchor.vue} (84%)

diff --git a/demo/documentation/components/anchor/enUS/placeholder.md b/demo/documentation/components/anchor/enUS/placeholder.md
index 620f2c67b..ef22c21ba 100644
--- a/demo/documentation/components/anchor/enUS/placeholder.md
+++ b/demo/documentation/components/anchor/enUS/placeholder.md
@@ -1,4 +1,4 @@
 # Placeholder
 ```html
-<div style="height: 600px;"></div>
+<div style="height: 1200px;"></div>
 ```
\ No newline at end of file
diff --git a/demo/documentation/components/anchor/enUS/theCaretaker.md b/demo/documentation/components/anchor/enUS/theCaretaker.md
index ac1c2dcd3..50737cc6d 100644
--- a/demo/documentation/components/anchor/enUS/theCaretaker.md
+++ b/demo/documentation/components/anchor/enUS/theCaretaker.md
@@ -1,4 +1,3 @@
 # The Caretaker
 ```html
-<n-tag id="the-caretaker">The Caretaker</n-tag>
 ```
\ No newline at end of file
diff --git a/demo/documentation/components/anchor/enUS/theColonel.md b/demo/documentation/components/anchor/enUS/theColonel.md
index 9a8610534..6ff2ce123 100644
--- a/demo/documentation/components/anchor/enUS/theColonel.md
+++ b/demo/documentation/components/anchor/enUS/theColonel.md
@@ -1,4 +1,3 @@
 # The Colonel
 ```html
-<n-tag id="the-colonel">The Colonel</n-tag>
 ```
\ No newline at end of file
diff --git a/demo/documentation/components/anchor/enUS/theGatekeeper.md b/demo/documentation/components/anchor/enUS/theGatekeeper.md
index de2b67c88..fdaa8fef7 100644
--- a/demo/documentation/components/anchor/enUS/theGatekeeper.md
+++ b/demo/documentation/components/anchor/enUS/theGatekeeper.md
@@ -1,4 +1,3 @@
 # The Gatekeeper
 ```html
-<n-tag id="the-gatekeeper">The Gatekeeper</n-tag>
 ```
\ No newline at end of file
diff --git a/demo/documentation/components/anchor/enUS/theLibrarian.md b/demo/documentation/components/anchor/enUS/theLibrarian.md
index faeb55da7..03990462d 100644
--- a/demo/documentation/components/anchor/enUS/theLibrarian.md
+++ b/demo/documentation/components/anchor/enUS/theLibrarian.md
@@ -1,4 +1,3 @@
 # The Librarian
 ```html
-<n-tag id="the-librarian">The Librarian</n-tag>
 ```
\ No newline at end of file
diff --git a/demo/documentation/components/anchor/enUS/theNarrator.md b/demo/documentation/components/anchor/enUS/theNarrator.md
index 1fdb55dc4..7bcc2fff8 100644
--- a/demo/documentation/components/anchor/enUS/theNarrator.md
+++ b/demo/documentation/components/anchor/enUS/theNarrator.md
@@ -1,4 +1,3 @@
 # The Narrator
 ```html
-<n-tag id="the-narrator">The Narrator</n-tag>
 ```
\ No newline at end of file
diff --git a/demo/documentation/components/anchor/enUS/theNarratorSShadow.md b/demo/documentation/components/anchor/enUS/theNarratorSShadow.md
index f44c27d92..0a46ced32 100644
--- a/demo/documentation/components/anchor/enUS/theNarratorSShadow.md
+++ b/demo/documentation/components/anchor/enUS/theNarratorSShadow.md
@@ -1,4 +1,3 @@
 # The Narrators's Shadow
 ```html
-<n-tag id="the-narrator-s-shadow">The Narrators's Shadow</n-tag>
 ```
\ No newline at end of file
diff --git a/packages/common/Anchor/src/AnchorAdapter.vue b/packages/common/Anchor/src/AnchorAdapter.vue
index 1f70a4545..842e4cbf2 100644
--- a/packages/common/Anchor/src/AnchorAdapter.vue
+++ b/packages/common/Anchor/src/AnchorAdapter.vue
@@ -14,22 +14,22 @@
     :offset-bottom="offsetBottom"
     :position="position"
   >
-    <n-anchor
+    <n-base-anchor
       :target="target"
     >
       <slot />
-    </n-anchor>
+    </n-base-anchor>
   </n-affix>
 </template>
 
 <script>
-import NAnchor from './Anchor'
+import NBaseAnchor from './BaseAnchor'
 import NAffix from '../../Affix'
 
 export default {
   name: 'NAnchor',
   components: {
-    NAnchor,
+    NBaseAnchor,
     NAffix
   },
   props: {
diff --git a/packages/common/Anchor/src/Anchor.vue b/packages/common/Anchor/src/BaseAnchor.vue
similarity index 84%
rename from packages/common/Anchor/src/Anchor.vue
rename to packages/common/Anchor/src/BaseAnchor.vue
index c470d6437..d74d8e0a2 100644
--- a/packages/common/Anchor/src/Anchor.vue
+++ b/packages/common/Anchor/src/BaseAnchor.vue
@@ -4,6 +4,7 @@
     :class="{
       [`n-${synthesizedTheme}-theme`]: synthesizedTheme
     }"
+    :style="synthesizedStyle"
   >
     <div
       ref="slot"
@@ -28,19 +29,22 @@ import withapp from '../../../mixins/withapp'
 import themeable from '../../../mixins/themeable'
 import fontawarable from '../../../mixins/fontawarable'
 
-function getOffsetTop (el, container) {
+function getOffset (el, container) {
   const {
-    top: elTop
+    top: elTop,
+    height
   } = el.getBoundingClientRect()
   const {
     top: containerTop
   } = container.getBoundingClientRect()
-  // console.log('elTop', elTop, 'containerTop', containerTop)
-  return elTop - containerTop
+  return {
+    top: elTop - containerTop,
+    height
+  }
 }
 
 export default {
-  name: 'NAnchor',
+  name: 'NBaseAnchor',
   mixins: [withapp, themeable, fontawarable],
   props: {
     target: {
@@ -68,16 +72,10 @@ export default {
         window.setTimeout(() => {
           slotEl.style.top = null
           barEl.style.top = null
-          // slotEl.style.top = null
         }, 150)
       }
     }
   },
-  mounted () {
-    // this.init()
-    // this.setActiveHref(window.location)
-    // this.blockTransitionOneTick()
-  },
   provide () {
     return {
       NAnchor: this
@@ -85,13 +83,11 @@ export default {
   },
   methods: {
     handleFontReady () {
-      console.log('font-ready')
       this.init()
       this.setActiveHref(window.location)
       this.handleScroll(false)
     },
     blockTransitionOneTick () {
-      console.log('block transition')
       const barEl = this.$refs.bar
       const slotEl = this.$refs.slot
       barEl.style.transition = 'none'
@@ -99,7 +95,6 @@ export default {
       barEl.getBoundingClientRect()
       slotEl.getBoundingClientRect()
       this.$nextTick().then(() => {
-        console.log('unblock transition')
         barEl.style.transition = null
         slotEl.style.transition = null
       })
@@ -111,7 +106,6 @@ export default {
         barEl.style.transition = 'none'
         slotEl.style.transition = 'none'
       }
-      // console.log('updateBarPosition')
       const {
         offsetHeight,
         offsetWidth
@@ -144,7 +138,7 @@ export default {
         const linkEl = document.getElementById(idMatchResult[1])
         if (linkEl) {
           this.activeHref = href
-          const top = getOffsetTop(linkEl, this.container) + (this.container.scrollTop || 0)
+          const top = getOffset(linkEl, this.container).top + (this.container.scrollTop || 0)
           this.container.scrollTo({
             top: top - this.bound
           })
@@ -156,28 +150,32 @@ export default {
       }
     },
     handleScroll (transition = true) {
-      // console.log('handleScroll')
       const links = []
       this.collectedLinkHrefs.forEach(href => {
         const idMatchResult = /#([^#]+)$/.exec(href)
         if (idMatchResult) {
           const linkEl = document.getElementById(idMatchResult[1])
           if (linkEl) {
+            const {
+              top,
+              height
+            } = getOffset(linkEl, this.container)
             links.push({
-              top: getOffsetTop(linkEl, this.container),
+              top,
+              height,
               href
             })
           }
         }
       })
       links.sort((a, b) => {
-        return a.top > b.top
+        return a.top > b.top || a.top === b.top ? a.height < b.height : false
       })
-      // const containerScrollTop = this.container.scrollTop
-      // console.log(links)
       const currentActiveHref = this.activeHref
-      const activeLink = links.reduce((prevLink, link) => {
-        // console.log(link.top)
+      const activeLink = links.reduce((prevLink, link, index) => {
+        if (index === links.length - 1 && link.top + link.height < 0) {
+          return null
+        }
         if (link.top <= this.bound) {
           if (prevLink === null) {
             return link
diff --git a/packages/common/Anchor/src/Link.vue b/packages/common/Anchor/src/Link.vue
index dabc174d5..e0b166a66 100644
--- a/packages/common/Anchor/src/Link.vue
+++ b/packages/common/Anchor/src/Link.vue
@@ -55,7 +55,6 @@ export default {
       }
     },
     handleClick (e) {
-      // e.preventDefault()
       this.NAnchor.setActiveHref(this.href)
     }
   }