From 2e17370baba890a6309b7bf1baf952693543ba6c Mon Sep 17 00:00:00 2001
From: Jason Yu <me@ycmjason.com>
Date: Sat, 12 May 2018 09:08:44 +0100
Subject: [PATCH 1/4] added outboundlink icon to nav external link

---
 lib/default-theme/NavLink.vue | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/lib/default-theme/NavLink.vue b/lib/default-theme/NavLink.vue
index c172e37520..46b027c13d 100644
--- a/lib/default-theme/NavLink.vue
+++ b/lib/default-theme/NavLink.vue
@@ -1,23 +1,24 @@
 <template>
-  <router-link
-    class="nav-link"
+  <router-link class="nav-link"
     :to="link"
     v-if="!isExternal(link)"
-    :exact="link === '/'"
-  >{{ item.text }}</router-link>
-  <a
-    v-else
+    :exact="link === '/'">{{ item.text }}</router-link>
+  <a v-else
     :href="link"
     class="nav-link"
     :target="isMailto(link) || isTel(link) ? null : '_blank'"
-    :rel="isMailto(link) || isTel(link) ? null : 'noopener noreferrer'"
-  >{{ item.text }}</a>
+    :rel="isMailto(link) || isTel(link) ? null : 'noopener noreferrer'">
+    {{ item.text }}
+    <OutboundLink/>
+  </a>
 </template>
 
 <script>
 import { isExternal, isMailto, isTel, ensureExt } from './util'
+import OutboundLink from './OutboundLink.vue'
 
 export default {
+  components: { OutboundLink },
   props: {
     item: {
       required: true

From 14e2d425a320d7a16aab081c1c857d42b9eb2e84 Mon Sep 17 00:00:00 2001
From: Jason Yu <me@ycmjason.com>
Date: Sat, 12 May 2018 10:09:58 +0100
Subject: [PATCH 2/4] include outboundlink when link is external

---
 lib/app/app.js       |  2 ++
 lib/markdown/link.js | 17 +++++++----------
 2 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/lib/app/app.js b/lib/app/app.js
index 8b78a28733..ceaa7b905b 100644
--- a/lib/app/app.js
+++ b/lib/app/app.js
@@ -1,6 +1,7 @@
 import Vue from 'vue'
 import Router from 'vue-router'
 import Content from './Content'
+import OutboundLink from '../default-theme/OutboundLink.vue'
 import ClientOnly from './ClientOnly'
 import dataMixin from './dataMixin'
 import store from './store'
@@ -28,6 +29,7 @@ Vue.use(Router)
 Vue.mixin(dataMixin)
 // component for rendering markdown content and setting title etc.
 Vue.component('Content', Content)
+Vue.component('OutboundLink', OutboundLink)
 // component for client-only content
 Vue.component('ClientOnly', ClientOnly)
 
diff --git a/lib/markdown/link.js b/lib/markdown/link.js
index 2ea7dfb920..79c23928be 100644
--- a/lib/markdown/link.js
+++ b/lib/markdown/link.js
@@ -15,8 +15,14 @@ module.exports = (md, externalAttrs) => {
       const isSourceLink = /(\/|\.md|\.html)(#.*)?$/.test(href)
       if (isExternal) {
         Object.entries(externalAttrs).forEach(([key, val]) => {
-          addAttr(token, key, val)
+          token.attrSet(key, val)
         })
+
+        if (/_blank/i.test(externalAttrs['target'])) {
+          // add OutBoundLink to content if it opens in _blank
+          tokens[idx + 1].type = 'html_block'
+          tokens[idx + 1].content += '<OutboundLink/>'
+        }
       } else if (isSourceLink) {
         hasOpenRouterLink = true
         tokens[idx] = toRouterLink(token, link)
@@ -56,12 +62,3 @@ module.exports = (md, externalAttrs) => {
     return self.renderToken(tokens, idx, options)
   }
 }
-
-function addAttr (token, name, val) {
-  const targetIndex = token.attrIndex(name)
-  if (targetIndex < 0) {
-    token.attrPush([name, val])
-  } else {
-    token.attrs[targetIndex][1] = val
-  }
-}

From 59681d881374185eac49c7148e643f7621c5c3c0 Mon Sep 17 00:00:00 2001
From: Jason Yu <me@ycmjason.com>
Date: Sat, 12 May 2018 10:28:48 +0100
Subject: [PATCH 3/4] remove external link hover styling

---
 lib/default-theme/NavLink.vue  | 2 +-
 lib/default-theme/NavLinks.vue | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/default-theme/NavLink.vue b/lib/default-theme/NavLink.vue
index 46b027c13d..f97c5da05e 100644
--- a/lib/default-theme/NavLink.vue
+++ b/lib/default-theme/NavLink.vue
@@ -5,7 +5,7 @@
     :exact="link === '/'">{{ item.text }}</router-link>
   <a v-else
     :href="link"
-    class="nav-link"
+    class="nav-link external"
     :target="isMailto(link) || isTel(link) ? null : '_blank'"
     :rel="isMailto(link) || isTel(link) ? null : 'noopener noreferrer'">
     {{ item.text }}
diff --git a/lib/default-theme/NavLinks.vue b/lib/default-theme/NavLinks.vue
index 17e8e402c0..8cb98f9d81 100644
--- a/lib/default-theme/NavLinks.vue
+++ b/lib/default-theme/NavLinks.vue
@@ -126,7 +126,7 @@ export default {
   .nav-links a
     &:hover, &.router-link-active
       color $textColor
-  .nav-item > a
+  .nav-item > a:not(.external)
     &:hover, &.router-link-active
       margin-bottom -2px
       border-bottom 2px solid lighten($accentColor, 8%)

From 9e78910f6e0595cc9ade5553ab1bbe2315caf075 Mon Sep 17 00:00:00 2001
From: Jason Yu <me@ycmjason.com>
Date: Sun, 13 May 2018 09:41:19 +0100
Subject: [PATCH 4/4] unformat code and remove OutboundLink from components

---
 lib/default-theme/NavLink.vue  | 14 ++++++++------
 lib/default-theme/NavLinks.vue |  3 +--
 lib/default-theme/Page.vue     |  2 --
 3 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/lib/default-theme/NavLink.vue b/lib/default-theme/NavLink.vue
index f97c5da05e..09d4dc85e0 100644
--- a/lib/default-theme/NavLink.vue
+++ b/lib/default-theme/NavLink.vue
@@ -1,13 +1,17 @@
 <template>
-  <router-link class="nav-link"
+  <router-link
+    class="nav-link"
     :to="link"
     v-if="!isExternal(link)"
-    :exact="link === '/'">{{ item.text }}</router-link>
-  <a v-else
+    :exact="link === '/'"
+  >{{ item.text }}</router-link>
+  <a
+    v-else
     :href="link"
     class="nav-link external"
     :target="isMailto(link) || isTel(link) ? null : '_blank'"
-    :rel="isMailto(link) || isTel(link) ? null : 'noopener noreferrer'">
+    :rel="isMailto(link) || isTel(link) ? null : 'noopener noreferrer'"
+  >
     {{ item.text }}
     <OutboundLink/>
   </a>
@@ -15,10 +19,8 @@
 
 <script>
 import { isExternal, isMailto, isTel, ensureExt } from './util'
-import OutboundLink from './OutboundLink.vue'
 
 export default {
-  components: { OutboundLink },
   props: {
     item: {
       required: true
diff --git a/lib/default-theme/NavLinks.vue b/lib/default-theme/NavLinks.vue
index 8cb98f9d81..f29ee0acd5 100644
--- a/lib/default-theme/NavLinks.vue
+++ b/lib/default-theme/NavLinks.vue
@@ -21,13 +21,12 @@
 </template>
 
 <script>
-import OutboundLink from './OutboundLink.vue'
 import DropdownLink from './DropdownLink.vue'
 import { resolveNavLinkItem } from './util'
 import NavLink from './NavLink.vue'
 
 export default {
-  components: { OutboundLink, NavLink, DropdownLink },
+  components: { NavLink, DropdownLink },
   computed: {
     userNav () {
       return this.$themeLocaleConfig.nav || this.$site.themeConfig.nav || []
diff --git a/lib/default-theme/Page.vue b/lib/default-theme/Page.vue
index 05b1cbce12..ae3510718c 100644
--- a/lib/default-theme/Page.vue
+++ b/lib/default-theme/Page.vue
@@ -28,11 +28,9 @@
 </template>
 
 <script>
-import OutboundLink from './OutboundLink.vue'
 import { resolvePage, normalize, outboundRE, endingSlashRE } from './util'
 
 export default {
-  components: { OutboundLink },
   props: ['sidebarItems'],
   computed: {
     lastUpdated () {