Merge pull request #221 from alphagov/no-opener

Add noopener and noreferrer to new window links
alphagov · Jan 4, 2017 · 237f8b0 · 237f8b0
2 parents 85998eb + def3f77
commit 237f8b0
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/app/views/shared/_share_buttons.html.erb b/app/views/shared/_share_buttons.html.erb
@@ -2,11 +2,13 @@
 <div class="share-buttons" data-module="track-share-button-clicks">
   <%= link_to share_urls[:facebook],
     target: "_blank",
+    rel: "noopener noreferrer",
     class: "share-button-link js-share-facebook" do %>
     <span class="share-button share-button-facebook"></span><span class="visually-hidden">Share on </span>Facebook<% end %>
 
   <%= link_to share_urls[:twitter],
     target: "_blank",
+    rel: "noopener noreferrer",
     class: "share-button-link js-share-twitter" do %>
     <span class="share-button share-button-twitter"></span><span class="visually-hidden">Share on </span>Twitter<% end %>
 </div>