Handle each type separately

Due to the nature of accordions having every type housed inside a single
one meant that closing one section closed the rest. However each type
now has their own accordion with a unique ID.

Author: Andrew

app/assets/javascripts/assets.js

@@ -1,5 +1,5 @@
 function changeIconOnClick() {
-  $('#accordion h5 a').click(function(e){
+  $('h5 a').click(function(e){
     var card = $(e.target).closest('div.card');
     var img = $('i', card)[0];
 

app/views/assets/index.html.erb

@@ -25,25 +25,26 @@
     </div>
   </div>
   <br>
-  <div id="accordion" class="center w-25">
     <% if @assets and not @assets.empty? %>
       <% @assets.each do |type, assets| %>
-        <div class="card mb-1">
-          <div class="card-header">
-            <h5 class="mb-0">
-              <a data-toggle="collapse" href="#<%= type %>">
-                <%= type.downcase.capitalize %>
-              </a>
-              <i class="fa fa-minus fa-pull-right mt-1"></i>
-            </h5>
-          </div>
+        <div id="accordion-<%= type.downcase %>" class="center w-25">
+          <div class="card mb-1">
+            <div class="card-header">
+              <h5 class="mb-0">
+                <a data-toggle="collapse" href="#<%= type %>">
+                  <%= type.downcase.capitalize %>
+                </a>
+                <i class="fa fa-minus fa-pull-right mt-1"></i>
+              </h5>
+            </div>
 
-          <div id="<%= type %>" class="collapse show" data-parent="#accordion">
-            <div class="card-body">
-              <div class="list-group">
-                <% assets.each do |a| %>
-                  <%= link_to a, assets_path + '/' + a, class: 'list-group-item list-group-item-action' %>
-                <% end %>
+            <div id="<%= type %>" class="collapse show" data-parent="#accordion-<%= type.downcase %>">
+              <div class="card-body">
+                <div class="list-group">
+                  <% assets.each do |a| %>
+                    <%= link_to a, assets_path + '/' + a, class: 'list-group-item list-group-item-action' %>
+                  <% end %>
+                </div>
               </div>
             </div>
           </div>
@@ -52,5 +53,4 @@
     <% else %>
       No asset information found.
     <% end %>
-  </div>
 </div>