From c6bf2dbc78394a8e927b5e4dd7c20910f96ab627 Mon Sep 17 00:00:00 2001
From: nicejade <jeffygisgreat@gmail.com>
Date: Thu, 3 May 2018 22:24:59 +0800
Subject: [PATCH] =?UTF-8?q?feat:=20add=20rel=3Dnoopener=20for=20target=3D'?=
 =?UTF-8?q?=5Fblank'=20=F0=9F=98=8A=20=20#24?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/components/AdsPosition.vue        |  2 +-
 src/components/Contact.vue            |  1 +
 src/components/linksList/LinkItem.vue | 13 ++++++++-----
 src/partials/FooterNav.vue            | 11 ++++++++---
 src/views/manage/Adverts.vue          |  2 +-
 src/views/manage/Links.vue            |  3 ++-
 6 files changed, 21 insertions(+), 11 deletions(-)

diff --git a/src/components/AdsPosition.vue b/src/components/AdsPosition.vue
index 96ba5cb1..eeb860b1 100644
--- a/src/components/AdsPosition.vue
+++ b/src/components/AdsPosition.vue
@@ -4,7 +4,7 @@
     <el-carousel trigger="click" class="jade-gg-body"
       indicator-position="outside" :interval='3600' height="256px">
       <el-carousel-item v-for="(item, index) in advertsList" :key="index">
-        <a :href="item.path" target=_blank>
+        <a :href="item.path" target="_blank" rel="noreferrer noopener">
           <img :src="item.image" :alt="item.alt">
         </a>
       </el-carousel-item>
diff --git a/src/components/Contact.vue b/src/components/Contact.vue
index 55b0dc6c..610b6339 100644
--- a/src/components/Contact.vue
+++ b/src/components/Contact.vue
@@ -5,6 +5,7 @@
         :href="item.path + item.name"
         :title="item.title"
         target="_blank"
+        rel="noreferrer noopener"
         :class="item.class"
         class="social-btn">
         <icon class="icons" :name="item.class"></icon>
diff --git a/src/components/linksList/LinkItem.vue b/src/components/linksList/LinkItem.vue
index c467813d..0fa5a841 100644
--- a/src/components/linksList/LinkItem.vue
+++ b/src/components/linksList/LinkItem.vue
@@ -5,19 +5,22 @@
         @click.stop.prevent="onThemeClick(item.theme)">
         {{ fillThemeName(item.classify, item.theme) }}
       </span>
-      <a class="item username" :href="getUserPath(item.createdBy)" target="_blank">
+      <a class="item username" :href="getUserPath(item.createdBy)" target="_blank" rel="noreferrer noopener">
         {{ item.createdBy || '' }}
       </a>
       <span >{{ item.created | dateOffset }}</span>
       <a class="tag"
         v-for="(iitem, index) in item.tags" :key="index"
-        :href="getTagPath(iitem)"  target="_blank"
-      >
+        :href="getTagPath(iitem)"
+        target="_blank" rel="noreferrer noopener">
         {{ iitem }}
       </a>
     </div>
     <h3 class="title">
-      <a class="title-link" :href="item.urlPath" target="_blank">{{ item.title }}</a>
+      <a class="title-link" :href="item.urlPath"
+        target="_blank" rel="noreferrer noopener">
+        {{ item.title }}
+      </a>
     </h3>
     <div class="abstract" v-if="isAbstract">
       {{ item.abstract || $util.interceptString(item.desc) }}
@@ -30,7 +33,7 @@
       <div class="link-desc"
         v-html="this.obtainLinkDesc(item)">
       </div>
-      <hr class="segmenting-line">
+      <hr v-if="item.review" class="segmenting-line">
       <div v-if="item.review" class="link-review">
         <preview-md
           :value="reviewPrefix + item.review">
diff --git a/src/partials/FooterNav.vue b/src/partials/FooterNav.vue
index a09e4958..863cbda7 100644
--- a/src/partials/FooterNav.vue
+++ b/src/partials/FooterNav.vue
@@ -4,7 +4,7 @@
       <div class="footer-body">
         <div class="content">
           <p>
-            <a target="_blank" class="link gtag-track" href="/"
+            <a target="_blank" rel="noreferrer noopener" class="link gtag-track" href="/"
               data-action="footer-nicelinks" data-category="footer" data-label="footer-nicelinks">
               {{ $t('niceLinksStr') }}
             </a>
@@ -19,6 +19,7 @@
             :href="item.path + item.name"
             :title="item.title"
             target="_blank"
+            rel="noreferrer noopener"
             :class="item.class"
             class="social-btn gtag-track">
             <icon class="icons" :name="item.class"></icon>
@@ -26,8 +27,12 @@
         </div>
         <div class="icp">
           <span>{{ copyright }}</span>
-          <a class="link gtag-track" target="_blank" href="http://jeffjade.com"
-            data-action="footer-jeffjade" data-category="footer" data-label="footer-jeffjade">
+          <a class="link gtag-track"
+            href="https://jeffjade.com"
+            target="_blank" rel="noreferrer noopener"
+            data-action="footer-jeffjade"
+            data-category="footer"
+            data-label="footer-jeffjade">
             晚晴幽草轩
           </a>
           {{ $t('produced') }}
diff --git a/src/views/manage/Adverts.vue b/src/views/manage/Adverts.vue
index bac9f23b..eec5c303 100644
--- a/src/views/manage/Adverts.vue
+++ b/src/views/manage/Adverts.vue
@@ -7,7 +7,7 @@
             <el-carousel trigger="click" class="jade-gg-body"
               indicator-position="outside" :interval='3600' height="256px">
               <el-carousel-item v-for="(item, index) in tableData" :key="index">
-                <a :href="item.path" target=_blank>
+                <a :href="item.path" target="_blank" rel="noreferrer noopener">
                   <img :src="item.image">
                 </a>
               </el-carousel-item>
diff --git a/src/views/manage/Links.vue b/src/views/manage/Links.vue
index 20fe4622..773b8269 100644
--- a/src/views/manage/Links.vue
+++ b/src/views/manage/Links.vue
@@ -35,7 +35,8 @@
               </el-table-column>
               <el-table-column prop="urlPath" :label="$t('linkAddressStr')" min-width="180">
                 <template scope="scope">
-                  <a class="title-link" :href="scope.row.urlPath" target="_blank">
+                  <a class="title-link" :href="scope.row.urlPath"
+                    target="_blank" rel="noreferrer noopener">
                     {{ scope.row.title }}
                   </a>
                 </template>