Fix lists in rich text to display properly

[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 e176f51a6a291d0c4898dbc17949855801d26002..6bff887b3cd861c86f8f08a27a533d49af40fd64 100644 (file)
--- a/app/views/browse/_way_details.html.erb
+++ b/app/views/browse/_way_details.html.erb
@@ -1,33 +1,34 @@
-<table id="<%= way_details.version %>">
-
+<div class="browse_details" id="<%= way_details.version %>">
+<% if way_details.redacted? %>
+  <div class='browse-section'>
+    <%= t 'browse.redacted.message_html', :type => t('browse.redacted.type.way'), :redaction_link => link_to(t('browse.redacted.redaction', :id => way_details.redaction.id), way_details.redaction), :version => way_details.version %>
+  </div>
+<% else %>
   <%= render :partial => "common_details", :object => way_details %>
 
-  <tr valign="top">
-    <th><%= t'browse.way_details.nodes' %></th>
-    <td>
-      <table cellpadding="0">
-        <% 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 => link_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 => link_class('way', w), :title => link_title(w) ) }.to_sentence %>)
-            <% end %>
-         </td></tr>
-        <% end %>
-      </table>
-    </td>
-  </tr>
+  <% unless way_details.way_nodes.empty? %>
+    <div class='browse-section'>
+      <h4><%= t'browse.way_details.nodes' %></h4>
+        <ul>
+          <% way_details.way_nodes.each do |wn| %>
+            <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 %>
+            </li>
+          <% end %>
+        </ul>
+    </div>
+  <% end %>
 
   <% unless way_details.containing_relation_members.empty? %>
-    <tr valign="top">
-      <th><%= t'browse.way_details.part_of' %></th>
-      <td>
-        <table cellpadding="0">
+    <div class='browse-section'>
+      <h4><%= t'browse.way_details.part_of' %></h4>
+      <ul>
           <%= render :partial => "containing_relation", :collection => way_details.containing_relation_members %>
-        </table>
-      </td>
-    </tr>      
+      </ul>
   <% end %>
-
-</table>
+<% end %>
+</div>