adds rel nofollow on links

This commit is contained in:
Alberto Garcia Cabeza
2017-04-21 16:59:11 +02:00
parent 3d0b7f1034
commit 8b42d57e18
3 changed files with 21 additions and 13 deletions

View File

@@ -1,6 +1,6 @@
<% if user_signed_in? %> <% if user_signed_in? %>
<li> <li>
<%= link_to notifications_path, class: "notifications" do %> <%= link_to notifications_path, rel: "nofollow", class: "notifications" do %>
<span class="sr-only"><%= t("layouts.header.notifications") %></span> <span class="sr-only"><%= t("layouts.header.notifications") %></span>
<% if current_user.notifications_count > 0 %> <% if current_user.notifications_count > 0 %>
<span class="icon-circle" aria-hidden="true"></span> <span class="icon-circle" aria-hidden="true"></span>
@@ -12,19 +12,24 @@
<% end %> <% end %>
</li> </li>
<li> <li>
<%= link_to t("layouts.header.my_activity_link"), user_path(current_user) %> <%= link_to t("layouts.header.my_activity_link"),
user_path(current_user), rel: "nofollow" %>
</li> </li>
<li> <li>
<%= link_to t("layouts.header.my_account_link"), account_path %> <%= link_to t("layouts.header.my_account_link"),
account_path, rel: "nofollow" %>
</li> </li>
<li> <li>
<%= link_to t("devise_views.menu.login_items.logout"), destroy_user_session_path, method: :delete %> <%= link_to t("devise_views.menu.login_items.logout"),
destroy_user_session_path, rel: "nofollow", method: :delete %>
</li> </li>
<% else %> <% else %>
<li> <li>
<%= link_to t("devise_views.menu.login_items.login"), new_user_session_path %> <%= link_to t("devise_views.menu.login_items.login"),
new_user_session_path, rel: "nofollow" %>
</li> </li>
<li> <li>
<%= link_to t("devise_views.menu.login_items.signup"), new_user_registration_path, class: "button" %> <%= link_to t("devise_views.menu.login_items.signup"),
new_user_registration_path, rel: "nofollow", class: "button" %>
</li> </li>
<% end %> <% end %>

View File

@@ -25,14 +25,14 @@
<div class="small-12 medium-4 column"> <div class="small-12 medium-4 column">
<h2> <h2>
<%= link_to t("layouts.footer.transparency_title"), setting['transparency_url'].presence || t("layouts.footer.transparency_url") %> <%= link_to t("layouts.footer.transparency_title"), setting['transparency_url'].presence || t("layouts.footer.transparency_url"), rel: "nofollow" %>
</h2> </h2>
<p><%= t("layouts.footer.transparency_text") %></p> <p><%= t("layouts.footer.transparency_text") %></p>
</div> </div>
<div class="small-12 medium-4 column"> <div class="small-12 medium-4 column">
<h2> <h2>
<%= link_to t("layouts.footer.open_data_title"), setting['opendata_url'].presence || t("layouts.header.external_link_opendata_url") %> <%= link_to t("layouts.footer.open_data_title"), setting['opendata_url'].presence || t("layouts.header.external_link_opendata_url"), rel: "nofollow" %>
</h2> </h2>
<p><%= t("layouts.footer.open_data_text") %></p> <p><%= t("layouts.footer.open_data_text") %></p>
</div> </div>
@@ -54,7 +54,7 @@
<ul> <ul>
<% if setting['twitter_handle'] %> <% if setting['twitter_handle'] %>
<li class="inline-block"> <li class="inline-block">
<%= link_to "https://twitter.com/#{setting['twitter_handle']}", target: "_blank", title: t("social.twitter") + t('shared.target_blank_html') do %> <%= link_to "https://twitter.com/#{setting['twitter_handle']}", target: "_blank", title: t("social.twitter") + t('shared.target_blank_html'), rel: "nofollow" do %>
<span class="sr-only"><%= t("social.twitter") %></span> <span class="sr-only"><%= t("social.twitter") %></span>
<span class="icon-twitter" aria-hidden="true"></span> <span class="icon-twitter" aria-hidden="true"></span>
<% end %> <% end %>
@@ -62,7 +62,7 @@
<% end %> <% end %>
<% if setting['facebook_handle'] %> <% if setting['facebook_handle'] %>
<li class="inline-block"> <li class="inline-block">
<%= link_to "https://www.facebook.com/#{setting['facebook_handle']}/", target: "_blank", title: t("social.facebook") + t('shared.target_blank_html') do %> <%= link_to "https://www.facebook.com/#{setting['facebook_handle']}/", target: "_blank", title: t("social.facebook") + t('shared.target_blank_html'), rel: "nofollow" do %>
<span class="sr-only"><%= t("social.facebook") %></span> <span class="sr-only"><%= t("social.facebook") %></span>
<span class="icon-facebook" aria-hidden="true"></span> <span class="icon-facebook" aria-hidden="true"></span>
<% end %> <% end %>
@@ -70,7 +70,7 @@
<% end %> <% end %>
<% if setting['blog_url'] %> <% if setting['blog_url'] %>
<li class="inline-block"> <li class="inline-block">
<%= link_to setting['blog_url'], target: "_blank", title: t("social.blog") + t('shared.target_blank_html') do %> <%= link_to setting['blog_url'], target: "_blank", title: t("social.blog") + t('shared.target_blank_html'), rel: "nofollow" do %>
<span class="sr-only"><%= t("social.blog") %></span> <span class="sr-only"><%= t("social.blog") %></span>
<span class="icon-blog" aria-hidden="true"></span> <span class="icon-blog" aria-hidden="true"></span>
<% end %> <% end %>
@@ -78,7 +78,7 @@
<% end %> <% end %>
<% if setting['youtube_handle'] %> <% if setting['youtube_handle'] %>
<li class="inline-block"> <li class="inline-block">
<%= link_to "https://www.youtube.com/#{setting['youtube_handle']}", target: "_blank", title: t("social.youtube") + t('shared.target_blank_html') do %> <%= link_to "https://www.youtube.com/#{setting['youtube_handle']}", target: "_blank", title: t("social.youtube") + t('shared.target_blank_html'), rel: "nofollow" do %>
<span class="sr-only"><%= t("social.youtube") %></span> <span class="sr-only"><%= t("social.youtube") %></span>
<span class="icon-youtube" aria-hidden="true"></span> <span class="icon-youtube" aria-hidden="true"></span>
<% end %> <% end %>
@@ -86,7 +86,7 @@
<% end %> <% end %>
<% if setting['telegram_handle'] %> <% if setting['telegram_handle'] %>
<li class="inline-block"> <li class="inline-block">
<%= link_to "https://www.telegram.me/#{setting['telegram_handle']}", target: "_blank", title: t("social.telegram") + t('shared.target_blank_html') do %> <%= link_to "https://www.telegram.me/#{setting['telegram_handle']}", target: "_blank", title: t("social.telegram") + t('shared.target_blank_html'), rel: "nofollow" do %>
<span class="sr-only"><%= t("social.telegram") %></span> <span class="sr-only"><%= t("social.telegram") %></span>
<span class="icon-telegram" aria-hidden="true"></span> <span class="icon-telegram" aria-hidden="true"></span>
<% end %> <% end %>

View File

@@ -3,12 +3,14 @@
<%= link_to t("layouts.header.external_link_transparency"), <%= link_to t("layouts.header.external_link_transparency"),
setting['transparency_url'].presence || t("layouts.header.external_link_transparency_url"), setting['transparency_url'].presence || t("layouts.header.external_link_transparency_url"),
target: "_blank", target: "_blank",
rel: "nofollow",
title: t('shared.target_blank_html') %> title: t('shared.target_blank_html') %>
</li> </li>
<li> <li>
<%= link_to t("layouts.header.external_link_opendata"), <%= link_to t("layouts.header.external_link_opendata"),
setting['opendata_url'].presence || t("layouts.header.external_link_opendata_url"), setting['opendata_url'].presence || t("layouts.header.external_link_opendata_url"),
target: "_blank", target: "_blank",
rel: "nofollow",
title: t('shared.target_blank_html') %> title: t('shared.target_blank_html') %>
</li> </li>
@@ -17,6 +19,7 @@
<%= link_to t("layouts.header.external_link_blog"), <%= link_to t("layouts.header.external_link_blog"),
setting['blog_url'], setting['blog_url'],
target: "_blank", target: "_blank",
rel: "nofollow",
title: t('shared.target_blank_html') %> title: t('shared.target_blank_html') %>
</li> </li>
<% end %> <% end %>