Add some strategic rel="nofollow" attributes to the data browser pages
authorTom Hughes <tom@compton.nu>
Fri, 26 Feb 2010 19:00:50 +0000 (19:00 +0000)
committerTom Hughes <tom@compton.nu>
Fri, 26 Feb 2010 19:00:50 +0000 (19:00 +0000)
to control what search engines will explore.

app/views/browse/_changeset_details.html.erb
app/views/browse/_map.html.erb
app/views/browse/_navigation.html.erb
app/views/browse/_node_details.html.erb

index 24d5da08c3c00d4bb5b2adda0ed15cf24f677a42..8245936e7c942e09c2b49ad8811041e00667f05b 100644 (file)
@@ -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>
index 5bb83b6bfb5f523234d719d24929f3c3e7d64e4b..25359fec02c480af8a3f96e259ebf48c64ae6e2a 100644 (file)
@@ -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' %>
index ed483d8ec48aeba3a7997fb62725758cb354a828..6d2549472dbc0bb2d056872e42c14eaf86ab10d3 100644 (file)
@@ -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>
index 7982e68052b448a2d69cccaa73f8e90d8b5c617c..f7d4e0e7c4fcd7e47c8334fbc42430ba196f77a7 100644 (file)
@@ -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? %>