Improve browse design

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

diff --git a/app/views/browse/_node_details.html.erb b/app/views/browse/_node_details.html.erb
index 45b830f0472120b03a2c2cfc4f4b0d19f4b4ad5d..2300e75ccee449808ad1c16710d43b3fee0d5915 100644 (file)
--- a/app/views/browse/_node_details.html.erb
+++ b/app/views/browse/_node_details.html.erb
@@ -1,28 +1,21 @@
-<% if node_details.redacted? %>
-<p><%= raw t 'browse.redacted.message', :type => t('browse.redacted.type.node'), :redaction_link => link_to(node_details.redaction) %></p>
-<% else %>
-<table class="browse_details" id="<%= node_details.version %>">
+<div class='browse-section'>
+  <% if node_details.redacted? %>
+    <%= t 'browse.redacted.message_html',
+          :type => t('browse.redacted.type.node'),
+          :version => node_details.version,
+          :redaction_link => link_to(t('browse.redacted.redaction',
+                                       :id => node_details.redaction.id), node_details.redaction) %>
+  <% else %>
+    <%= render :partial => "common_details", :object => node_details %>
 
-  <%= render :partial => "common_details", :object => node_details %>
-
-  <tr>
-    <th><%= t 'browse.node_details.coordinates' %></th>
-    <td><div class="geo"><%= link_to(content_tag(:span, number_with_delimiter(node_details.lat), :class => "latitude") + ", " + content_tag(:span, number_with_delimiter(node_details.lon), :class => "longitude"), {: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? %>
-    <tr valign="top">
-      <th><%= t 'browse.node_details.part_of' %></th>
-      <td>
-        <table cellpadding="0">
-          <% node_details.ways.each do |way| %>
-            <tr><td><%= link_to h(printable_name(way)), { :action => "way", :id => way.id.to_s }, :class => link_class('way', way), :title => link_title(way) %></td></tr>
-          <% end %>
-          <%= render :partial => "containing_relation", :collection => node_details.containing_relation_members %>
-        </table>
-      </td>
-    </tr>      
+    <% unless node_details.ways.empty? and node_details.containing_relation_members.empty? %>
+      <h4><%= t 'browse.node_details.part_of' %></h4>
+      <ul>
+        <% node_details.ways.each do |way| %>
+          <li><%= link_to h(printable_name(way)), { :action => "way", :id => way.id.to_s }, :class => link_class('way', way), :title => link_title(way) %></li>
+        <% end %>
+        <%= render :partial => "containing_relation", :collection => node_details.containing_relation_members %>
+      </ul>
+    <% end %>
   <% end %>
-
-</table>
-<% end %>
+</div>