Improve browse design

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

diff --git a/app/views/browse/_way_details.html.erb b/app/views/browse/_way_details.html.erb
index 149153f60ad43311f11db4d643b9e6217e7e5634..30e29d989d2c79a66a5269ed8a683f3cd7aa8cee 100644 (file)
--- a/app/views/browse/_way_details.html.erb
+++ b/app/views/browse/_way_details.html.erb
@@ -1,33 +1,33 @@
-<table>
+<div class='browse-section'>
+  <% if way_details.redacted? %>
+    <%= t 'browse.redacted.message_html',
+          :type => t('browse.redacted.type.way'),
+          :version => way_details.version,
+          :redaction_link => link_to(t('browse.redacted.redaction',
+                                       :id => way_details.redaction.id), way_details.redaction) %>
+  <% else %>
+    <%= render :partial => "common_details", :object => way_details %>

 
 

-  <%= render :partial => "common_details", :object => way_details %>
-
-  <tr valign="top">
-    <th><%= t'browse.way_details.nodes' %></th>
-    <td>
-      <table cellpadding="0">
+    <% unless way_details.way_nodes.empty? %>
+      <h4><%= t'browse.way_details.nodes' %></h4>
+      <ul>

         <% way_details.way_nodes.each do |wn| %>
         <% way_details.way_nodes.each do |wn| %>

-          <tr><td>
-            <%= link_to h(printable_name(wn.node)), { :action => "node", :id => wn.node_id.to_s }, :class => css_class('node', wn.node), :title => link_title(wn.node) %>
-            <% related_ways = wn.node.ways.reject { |w| w.id == way_details.id } %>
-           <% if related_ways.size > 0 then %>
-              (<%= t 'browse.way_details.also_part_of', :count => related_ways.size, :related_ways => related_ways.map { |w| link_to(h(printable_name(w)), { :action => "way", :id => w.id.to_s }, :class => css_class('way', w), :title => link_title(w) ) }.to_sentence %>)
+          <li>
+            <%= link_to h(printable_name(wn.node)), { :action => "node", :id => wn.node_id.to_s }, :class => link_class('node', wn.node), :title => link_title(wn.node) %>
+            <% related_ways = wn.node.ways.reject { |w| w.id == wn.way_id } %>
+            <% if related_ways.size > 0 then %>
+              (<%= raw t 'browse.way_details.also_part_of', :count => related_ways.size, :related_ways => related_ways.map { |w| link_to(h(printable_name(w)), { :action => "way", :id => w.id.to_s }, :class => link_class('way', w), :title => link_title(w) ) }.to_sentence %>)

             <% end %>
             <% end %>

-         </td></tr>
+          </li>

         <% end %>
         <% end %>

-      </table>
-    </td>
-  </tr>
+      </ul>
+    <% end %>

 
 

-  <% unless way_details.containing_relation_members.empty? %>
-    <tr valign="top">
-      <th><%= t'browse.way_details.part_of' %></th>
-      <td>
-        <table cellpadding="0">
-          <%= render :partial => "containing_relation", :collection => way_details.containing_relation_members %>
-        </table>
-      </td>
-    </tr>      
+    <% unless way_details.containing_relation_members.empty? %>
+      <h4><%= t'browse.way_details.part_of' %></h4>
+      <ul>
+        <%= render :partial => "containing_relation", :collection => way_details.containing_relation_members %>
+      </ul>
+    <% end %>

   <% end %>
   <% end %>

-
-</table>
+</div>