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 4d762330f6a63c755b4accf385c5dd0a574803c3..2300e75ccee449808ad1c16710d43b3fee0d5915 100644 (file)
--- a/app/views/browse/_node_details.html.erb
+++ b/app/views/browse/_node_details.html.erb
@@ -1,24 +1,21 @@
-<table>
+<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 ("<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? %>
-    <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 => css_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>
+</div>