cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Add some strategic rel="nofollow" attributes to the data browser pages

Browse source 
to control what search engines will explore.
This commit is contained in:
Tom Hughes 2010-02-26 19:00:50 +00:00
parent c6077fd124
commit e19be03313
4 changed files with 9 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

2
app/views/browse/_changeset_details.html.erb
View 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>

4
app/views/browse/_map.html.erb
View 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' %>

10
app/views/browse/_navigation.html.erb
View 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>

2
app/views/browse/_node_details.html.erb
View 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? %>