From: Tom Hughes <tom@compton.nu>
Date: Fri, 26 Feb 2010 19:00:50 +0000 (+0000)
Subject: Add some strategic rel="nofollow" attributes to the data browser pages
X-Git-Tag: live~6336^2~6
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/e19be03313fb226b8397d67dd6dc33d58a1444d0?hp=c6077fd124c79559e45174f8e5df9ba4a1d09bc3

Add some strategic rel="nofollow" attributes to the data browser pages
to control what search engines will explore.
---

diff --git a/app/views/browse/_changeset_details.html.erb b/app/views/browse/_changeset_details.html.erb
index 24d5da08c..8245936e7 100644
--- a/app/views/browse/_changeset_details.html.erb
+++ b/app/views/browse/_changeset_details.html.erb
@@ -36,7 +36,7 @@
           </tr>
           <tr>
             <td><%=minlon -%></td>
-            <td>(<a href='/?minlon=<%= minlon %>&minlat=<%= minlat %>&maxlon=<%= maxlon %>&maxlat=<%= maxlat %>&box=yes' title='<%= t 'browse.changeset_details.show_area_box' %>'><%= t 'browse.changeset_details.box' %></a>)</td>
+            <td>(<a href='/?minlon=<%= minlon %>&minlat=<%= minlat %>&maxlon=<%= maxlon %>&maxlat=<%= maxlat %>&box=yes' title='<%= t 'browse.changeset_details.show_area_box' %>' rel='nofollow'><%= t 'browse.changeset_details.box' %></a>)</td>
             <td><%=maxlon -%></td>
           </tr>
           <tr>
diff --git a/app/views/browse/_map.html.erb b/app/views/browse/_map.html.erb
index 5bb83b6bf..25359fec0 100644
--- a/app/views/browse/_map.html.erb
+++ b/app/views/browse/_map.html.erb
@@ -7,10 +7,10 @@
       <div id="small_map" style="width:250px; height: 300px; border: solid 1px black">
       </div>
       <span id="loading"><%= t 'browse.map.loading' %></span>
-      <a id="area_larger_map" href=""></a>
+      <a id="area_larger_map" href="" rel="nofollow"></a>
       <% unless map.instance_of? Changeset %>
         <br />
-        <a id="object_larger_map" href=""></a>
+        <a id="object_larger_map" href="" rel="nofollow"></a>
       <% end %>
     <% else %>
       <%= t 'browse.map.deleted' %>
diff --git a/app/views/browse/_navigation.html.erb b/app/views/browse/_navigation.html.erb
index ed483d8ec..6d2549472 100644
--- a/app/views/browse/_navigation.html.erb
+++ b/app/views/browse/_navigation.html.erb
@@ -4,20 +4,20 @@
       <% if @prev_by_user %>
         <%= link_to t('browse.changeset_navigation.paging.user.prev', :id => @prev_by_user.id.to_s),
                     { :id => @prev_by_user.id },
-                    { :title => t('browse.changeset_navigation.user.prev_tooltip', :user => @prev_by_user.user.display_name) } %>
+                    { :title => t('browse.changeset_navigation.user.prev_tooltip', :user => @prev_by_user.user.display_name), :rel => "nofollow" } %>
         |
       <% end %>
       <%=
       user = (@prev_by_user || @next_by_user).user.display_name
       link_to h(user),
               { :controller => "changeset", :action => "list", :display_name => user },
-              { :title => t('browse.changeset_navigation.user.name_tooltip', :user => h(user)) }
+              { :title => t('browse.changeset_navigation.user.name_tooltip', :user => h(user)), :rel => "nofollow" }
       %>
       <% if @next_by_user %>
         |
         <%= link_to t('browse.changeset_navigation.paging.user.next', :id => @next_by_user.id.to_s),
                     { :id => @next_by_user.id },
-                    { :title => t('browse.changeset_navigation.user.next_tooltip', :user => @next_by_user.user.display_name) } %>
+                    { :title => t('browse.changeset_navigation.user.next_tooltip', :user => @next_by_user.user.display_name), :rel => "nofollow" } %>
       <% end %>
     </span>
     <br/>
@@ -26,7 +26,7 @@
     <% if @prev %>
       <%= link_to t('browse.changeset_navigation.paging.all.prev', :id => @prev.id.to_s),
                   { :id => @prev.id },
-                  { :title => t('browse.changeset_navigation.all.prev_tooltip') } %>
+                  { :title => t('browse.changeset_navigation.all.prev_tooltip'), :rel => "nofollow" } %>
     <% end %>
     <% if @prev and @next %>
       | 
@@ -34,7 +34,7 @@
     <% if @next %>
       <%= link_to t('browse.changeset_navigation.paging.all.next', :id => @next.id.to_s),
                   { :id => @next.id },
-                  { :title => t('browse.changeset_navigation.all.next_tooltip') } %>
+                  { :title => t('browse.changeset_navigation.all.next_tooltip'), :rel => "nofollow" } %>
     <% end %>
   </span>
 </div>
diff --git a/app/views/browse/_node_details.html.erb b/app/views/browse/_node_details.html.erb
index 7982e6805..f7d4e0e7c 100644
--- a/app/views/browse/_node_details.html.erb
+++ b/app/views/browse/_node_details.html.erb
@@ -4,7 +4,7 @@
 
   <tr>
     <th><%= t 'browse.node_details.coordinates' %></th>
-    <td><div class="geo"><%= link_to ("<span class='latitude'>#{number_with_delimiter(node_details.lat)}</span>, <span class='longitude'>#{number_with_delimiter(node_details.lon)}</span>"), {:controller => 'site', :action => 'index', :lat => h(node_details.lat), :lon => h(node_details.lon), :zoom => "18"} %></div></td>
+    <td><div class="geo"><%= link_to ("<span class='latitude'>#{number_with_delimiter(node_details.lat)}</span>, <span class='longitude'>#{number_with_delimiter(node_details.lon)}</span>"), {:controller => 'site', :action => 'index', :lat => h(node_details.lat), :lon => h(node_details.lon), :zoom => "18"}, { :rel => "nofollow" } %></div></td>
   </tr>
 
   <% unless node_details.ways.empty? and node_details.containing_relation_members.empty? %>