From 377da36432c8fb8ca7dd59627ca51aa8b1c138ce Mon Sep 17 00:00:00 2001
From: Billyyyyy3320 <newsbielt703@gmail.com>
Date: Fri, 11 Oct 2019 11:43:20 +0800
Subject: [PATCH] feat($theme-default): support nav config for attr of internal
 links

---
 .../theme-default/components/NavLink.vue          | 15 +++++++++------
 packages/docs/docs/theme/default-theme-config.md  |  1 +
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/packages/@vuepress/theme-default/components/NavLink.vue b/packages/@vuepress/theme-default/components/NavLink.vue
index 73872880a2..b8517b1eaa 100644
--- a/packages/@vuepress/theme-default/components/NavLink.vue
+++ b/packages/@vuepress/theme-default/components/NavLink.vue
@@ -3,7 +3,7 @@
     class="nav-link"
     :to="link"
     @focusout.native="focusoutAction"
-    v-if="!isExternal(link)"
+    v-if="isInternal"
     :exact="exact"
   >{{ item.text }}</router-link>
   <a
@@ -42,7 +42,8 @@ export default {
     },
 
     target () {
-      return isMailto(this.link) || isTel(this.link) ? null : this.item.target || '_blank'
+      return isMailto(this.link) || isTel(this.link) ? null : this.item.target
+      || (isExternal(this.link) ? '_blank' : '')
     },
 
     isTargetBlank () {
@@ -50,14 +51,16 @@ export default {
     },
 
     rel () {
-      return isMailto(this.link) || isTel(this.link) ? null : this.item.rel || 'noopener noreferrer'
+      return isMailto(this.link) || isTel(this.link) ? null : this.item.rel
+      || (this.isTargetBlank ? 'noopener noreferrer' : '')
+    },
+
+    isInternal () {
+      return !isExternal(this.link) && !this.isTargetBlank
     }
   },
 
   methods: {
-    isExternal,
-    isMailto,
-    isTel,
     focusoutAction () {
       this.$emit('focusout')
     }
diff --git a/packages/docs/docs/theme/default-theme-config.md b/packages/docs/docs/theme/default-theme-config.md
index 04d0bcd48c..0ecc1a4437 100644
--- a/packages/docs/docs/theme/default-theme-config.md
+++ b/packages/docs/docs/theme/default-theme-config.md
@@ -62,6 +62,7 @@ module.exports = {
   themeConfig: {
     nav: [
       { text: 'External', link: 'https://google.com', target:'_self', rel:'' },
+      { text: 'Guide', link: '/guide/', target:'_blank' }
     ]
   }
 }