From 466632ac4d492258ac4aeb29979ce7af7452b4e7 Mon Sep 17 00:00:00 2001
From: George Cushen <gcushen@users.noreply.github.com>
Date: Mon, 16 Oct 2017 00:45:45 +0100
Subject: [PATCH] Use noopener for external URLs that open in new tabs (Close
 #286)

---
 layouts/partials/footer_container.html | 5 +++--
 layouts/partials/share.html            | 8 ++++----
 layouts/partials/widgets/about.html    | 6 +++---
 layouts/partials/widgets/projects.html | 2 +-
 layouts/project/single.html            | 2 +-
 5 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/layouts/partials/footer_container.html b/layouts/partials/footer_container.html
index 82465677..0072f22f 100644
--- a/layouts/partials/footer_container.html
+++ b/layouts/partials/footer_container.html
@@ -4,8 +4,9 @@
 
       {{ with .Site.Copyright }}{{ . | markdownify}} &middot; {{ end }}
 
-      Powered by the <a href="https://github.com/gcushen/hugo-academic" target="_blank">Academic
-      theme</a> for <a href="http://gohugo.io" target="_blank">Hugo</a>.
+      Powered by the
+      <a href="https://sourcethemes.com/academic/" target="_blank" rel="noopener">Academic theme</a> for
+      <a href="https://gohugo.io" target="_blank" rel="noopener">Hugo</a>.
 
       <span class="pull-right" aria-hidden="true">
         <a href="#" id="back_to_top">
diff --git a/layouts/partials/share.html b/layouts/partials/share.html
index 0cf03469..aa899b0d 100644
--- a/layouts/partials/share.html
+++ b/layouts/partials/share.html
@@ -4,28 +4,28 @@
     <li>
       <a class="twitter"
          href="https://twitter.com/intent/tweet?text={{ .Title | html }}&amp;url={{ .Permalink | html }}"
-         target="_blank">
+         target="_blank" rel="noopener">
         <i class="fa fa-twitter"></i>
       </a>
     </li>
     <li>
       <a class="facebook"
          href="https://www.facebook.com/sharer.php?u={{ .Permalink | html }}"
-         target="_blank">
+         target="_blank" rel="noopener">
         <i class="fa fa-facebook"></i>
       </a>
     </li>
     <li>
       <a class="linkedin"
          href="https://www.linkedin.com/shareArticle?mini=true&amp;url={{ .Permalink | html }}&amp;title={{ .Title | html }}"
-         target="_blank">
+         target="_blank" rel="noopener">
         <i class="fa fa-linkedin"></i>
       </a>
     </li>
     <li>
       <a class="weibo"
          href="http://service.weibo.com/share/share.php?url={{ .Permalink | html }}&amp;title={{ .Title | html }}"
-         target="_blank">
+         target="_blank" rel="noopener">
         <i class="fa fa-weibo"></i>
       </a>
     </li>
diff --git a/layouts/partials/widgets/about.html b/layouts/partials/widgets/about.html
index c05b5ce0..1d41e1da 100644
--- a/layouts/partials/widgets/about.html
+++ b/layouts/partials/widgets/about.html
@@ -21,7 +21,7 @@
         {{/* Support legacy `organization` option. */}}
         {{ if $.Site.Params.organization }}
         <h3 itemprop="worksFor" itemscope itemtype="http://schema.org/Organization">
-          {{ with $.Site.Params.organization_url }}<a href="{{ . }}" target="_blank" itemprop="url">{{ end }}
+          {{ with $.Site.Params.organization_url }}<a href="{{ . }}" target="_blank" itemprop="url" rel="noopener">{{ end }}
             <span itemprop="name">{{ $.Site.Params.organization }}</span>
           {{ if $.Site.Params.organization_url }}</a>{{ end }}
         </h3>
@@ -29,7 +29,7 @@
 
         {{ range $.Site.Params.organizations }}
         <h3 itemprop="worksFor" itemscope itemtype="http://schema.org/Organization">
-          {{ with .url }}<a href="{{ . }}" target="_blank" itemprop="url">{{ end }}
+          {{ with .url }}<a href="{{ . }}" target="_blank" itemprop="url" rel="noopener">{{ end }}
           <span itemprop="name">{{ .name }}</span>
           {{ if .url }}</a>{{ end }}
         </h3>
@@ -42,7 +42,7 @@
         {{ range $.Site.Params.social }}
         {{ $pack := or .icon_pack "fa" }}
         <li>
-          <a itemprop="sameAs" href="{{ .link | safeURL }}" target="_blank">
+          <a itemprop="sameAs" href="{{ .link | safeURL }}" target="_blank" rel="noopener">
             <i class="{{ $pack }} {{ $pack }}-{{ .icon }} big-icon"></i>
           </a>
         </li>
diff --git a/layouts/partials/widgets/projects.html b/layouts/partials/widgets/projects.html
index d441e44e..f09a3f4a 100644
--- a/layouts/partials/widgets/projects.html
+++ b/layouts/partials/widgets/projects.html
@@ -40,7 +40,7 @@
           {{ if .Content }}
           <a href="{{ .Permalink }}">
           {{ else }}
-            {{ with .Params.external_link }}<a href="{{ . }}" target="_blank">{{ end }}
+            {{ with .Params.external_link }}<a href="{{ . }}" target="_blank" rel="noopener">{{ end }}
           {{ end }}
           {{ .Title }}
           {{ if .Content }}</a>{{ else }}{{ with .Params.external_link }}</a>{{ end }}{{ end }}
diff --git a/layouts/project/single.html b/layouts/project/single.html
index d576aa22..08aa2e36 100644
--- a/layouts/project/single.html
+++ b/layouts/project/single.html
@@ -16,7 +16,7 @@
     </div>
 
     {{ with .Params.external_link }}
-    <a class="btn btn-primary btn-outline" href="{{ . }}" target="_blank">{{ i18n "open_project_site" }}</a>
+    <a class="btn btn-primary btn-outline" href="{{ . }}" target="_blank" rel="noopener">{{ i18n "open_project_site" }}</a>
     {{ end }}
 
     <div class="article-style" itemprop="articleBody">