Remove unnecessary h() calls on model attributes

[rails.git] / app / views / browse / _common_details.html.erb

diff --git a/app/views/browse/_common_details.html.erb b/app/views/browse/_common_details.html.erb
index 95ddafcd5205a0dd992726c10b53aab2063c5ddf..669e0fe6dee4358452740c2540c041080d4cb0c7 100644 (file)
--- a/app/views/browse/_common_details.html.erb
+++ b/app/views/browse/_common_details.html.erb
@@ -1,32 +1,31 @@
-<div class='browse-section common'>
+<h4>
   <% if common_details.changeset.tags['comment'].present? %>
-    <div style="font-weight: bold; margin-bottom: 5px;">
-      <%= linkify(h(common_details.changeset.tags['comment'])) %>
-    </div>
+    <%= linkify(common_details.changeset.tags["comment"]) %>
   <% else %>
-    <div style="font-weight: bold; margin-bottom: 5px;">(no comment)</div>
+    <%= t "browse.no_comment" %>
   <% end %>
-  <div>
-    <% if common_details.visible? %>
-      <%= t 'browse.common_details.edited' %> 
-    <% else %>
-      <%= t 'browse.common_details.deleted' %> 
-    <% end %>
-      <abbr><%= distance_of_time_in_words_to_now(common_details.timestamp) %> ago</abbr>
+</h4>
 
-    <% if common_details.changeset.user.data_public? %>
-      <%= t 'browse.common_details.by' %>
-      <%= link_to h(common_details.changeset.user.display_name), :controller => "user", :action => "view", :display_name => common_details.changeset.user.display_name %>
-    <% end %>
-  </div>
+<div class="details">
+  <%= t "browse.#{common_details.visible? ? :edited : :deleted}_by_html",
+        :time => time_ago_in_words(common_details.timestamp, :scope => :'datetime.distance_in_words_ago'),
+        :user => changeset_user_link(common_details.changeset),
+        :title => l(common_details.timestamp) %>
+</div>
+
+<div class="details">
+  <%= t "browse.version" %>
+  #<%= common_details.version %>
+  &middot;
+  <%= t "browse.in_changeset" %>
+  #<%= link_to common_details.changeset_id, :action => :changeset, :id => common_details.changeset_id %>
+</div>
 
-  <div>
-    <%= t 'browse.common_details.version' %> 
-    #<%= h(common_details.version) %>
-    &middot;
-    <%= t 'browse.common_details.in_changeset' %> 
-    #<%= link_to common_details.changeset_id, :action => :changeset, :id => common_details.changeset_id %>
-  </div>
+<% if @type == "node" and common_details.visible? %>
+<div class="details geo">
+  <%= t "browse.location" %>
+  <%= link_to(content_tag(:span, number_with_delimiter(common_details.lat), :class => "latitude") + ", " + content_tag(:span, number_with_delimiter(common_details.lon), :class => "longitude"), :controller => "site", :action => "index", :anchor => "map=18/#{common_details.lat}/#{common_details.lon}") %>
 </div>
+<% end %>
 
-<%= render :partial => "tag_details", :object => common_details %>
+<%= render :partial => "tag_details", :object => common_details.tags %>