Simplify link_to user calls on issue pages

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

diff --git a/app/views/browse/_node.html.erb b/app/views/browse/_node.html.erb
index 873360bb22ad0d687fba1d4f299510423fa44eeb..4e2ae0343d2e57f5ba7a460b5701892ed0d923b3 100644 (file)
--- a/app/views/browse/_node.html.erb
+++ b/app/views/browse/_node.html.erb
@@ -1,23 +1,35 @@
-<% if node.redacted? %>
-  <div class='browse-section browse-redacted'>
-    <%= t 'browse.redacted.message_html',
-          :type => t('browse.redacted.type.node'),
+<% if node.redacted? && !params[:show_redactions] %>
+  <div class="browse-section browse-redacted">
+    <%= t "browse.redacted.message_html",
+          :type => t("browse.redacted.type.node"),
           :version => node.version,
-          :redaction_link => link_to(t('browse.redacted.redaction',
+          :redaction_link => link_to(t("browse.redacted.redaction",
                                        :id => node.redaction.id), node.redaction) %>
   </div>
 <% else %>
-  <div class='browse-section browse-node'>
-    <%= render :partial => "common_details", :object => node %>
+  <%= tag.div :class => ["browse-section", "browse-node", { "text-muted" => node.redacted? }] do %>
+    <%= render :partial => "browse/common_details", :object => node %>
 
     <% unless node.ways.empty? and node.containing_relation_members.empty? %>
-      <h4><%= t 'browse.part_of' %></h4>
-      <ul>
-        <% node.ways.uniq.each do |way| %>
-          <li><%= link_to 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.containing_relation_members.uniq %>
-      </ul>
+      <h4><%= t "browse.part_of" %></h4>
+      <% unless node.ways.empty? %>
+        <details <%= "open" if node.ways.count < 10 %>>
+          <summary><%= t "browse.part_of_ways", :count => node.ways.uniq.count %></summary>
+          <ul class="list-unstyled">
+            <% node.ways.uniq.each do |way| %>
+              <li><%= element_single_current_link "way", way %></li>
+            <% end %>
+          </ul>
+        </details>
+      <% end %>
+      <% unless node.containing_relation_members.empty? %>
+        <details <%= "open" if node.containing_relation_members.count < 10 %>>
+          <summary><%= t "browse.part_of_relations", :count => node.containing_relation_members.uniq.count %></summary>
+          <ul class="list-unstyled">
+            <%= render :partial => "browse/containing_relation", :collection => node.containing_relation_members.uniq %>
+          </ul>
+        </details>
+      <% end %>
     <% end %>
-  </div>
+  <% end %>
 <% end %>