Revert the nofollow attributes - they aren't really needed if sites

obey robots.txt and don't do what Firefishy thought they did.

diff --git a/app/views/browse/_changeset_details.html.erb b/app/views/browse/_changeset_details.html.erb
index 8245936e7c942e09c2b49ad8811041e00667f05b..24d5da08c3c00d4bb5b2adda0ed15cf24f677a42 100644 (file)
--- 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' %>' rel='nofollow'><%= 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' %>'><%= 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 25359fec02c480af8a3f96e259ebf48c64ae6e2a..5bb83b6bfb5f523234d719d24929f3c3e7d64e4b 100644 (file)
--- 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="" rel="nofollow"></a>
+      <a id="area_larger_map" href=""></a>
       <% unless map.instance_of? Changeset %>
         <br />
-        <a id="object_larger_map" href="" rel="nofollow"></a>
+        <a id="object_larger_map" href=""></a>
       <% end %>
     <% else %>
       <%= t 'browse.map.deleted' %>

diff --git a/app/views/browse/_navigation.html.erb b/app/views/browse/_navigation.html.erb
index 6d2549472dbc0bb2d056872e42c14eaf86ab10d3..ed483d8ec48aeba3a7997fb62725758cb354a828 100644 (file)
--- 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), :rel => "nofollow" } %>
+                    { :title => t('browse.changeset_navigation.user.prev_tooltip', :user => @prev_by_user.user.display_name) } %>
         |
       <% 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)), :rel => "nofollow" }
+              { :title => t('browse.changeset_navigation.user.name_tooltip', :user => h(user)) }
       %>
       <% 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), :rel => "nofollow" } %>
+                    { :title => t('browse.changeset_navigation.user.next_tooltip', :user => @next_by_user.user.display_name) } %>
       <% 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'), :rel => "nofollow" } %>
+                  { :title => t('browse.changeset_navigation.all.prev_tooltip') } %>
     <% 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'), :rel => "nofollow" } %>
+                  { :title => t('browse.changeset_navigation.all.next_tooltip') } %>
     <% end %>
   </span>
 </div>

diff --git a/app/views/browse/_node_details.html.erb b/app/views/browse/_node_details.html.erb
index f7d4e0e7c4fcd7e47c8334fbc42430ba196f77a7..7982e68052b448a2d69cccaa73f8e90d8b5c617c 100644 (file)
--- 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"}, { :rel => "nofollow" } %></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"} %></div></td>
   </tr>
 
   <% unless node_details.ways.empty? and node_details.containing_relation_members.empty? %>