From f3be2cae65188335f95dbe9a8c892dd8f330aa31 Mon Sep 17 00:00:00 2001
From: Christopher Baines <mail@cbaines.net>
Date: Thu, 3 May 2018 21:25:18 +0100
Subject: Improve the jobs partial

Fix some of the ids, add a caret, and improve the line length.
---
 app/views/shared/_jobs.html.erb | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

(limited to 'app')

diff --git a/app/views/shared/_jobs.html.erb b/app/views/shared/_jobs.html.erb
index e70e8ef..a151389 100644
--- a/app/views/shared/_jobs.html.erb
+++ b/app/views/shared/_jobs.html.erb
@@ -1,21 +1,23 @@
 
 <div class="panel-group"
-     id="accordion"
+     id="jobs"
      role="tablist"
      aria-multiselectable="true">
 
   <% jobs.each do |job| %>
 
     <div class="panel panel-default">
-      <div class="panel-heading" role="tab" id="headingOne">
+      <div class="panel-heading" role="tab" id="jobHeading<%= job.id %>">
         <h4 class="panel-title">
           <a role="button"
              data-toggle="collapse"
              data-parent="#accordion"
-             href="#collapseOne"
+             href="#job_<%= job.id %>"
              aria-expanded="true"
-             aria-controls="collapseOne">
+             aria-controls="job_<%= job.id %>"
+          >
             <%= job.job_class %>
+            <span class="caret"></span>
           </a>
           <span class="pull-right">
             <% if job.instance_of? FinishedTerraformJob %>
@@ -29,7 +31,11 @@
           </span>
         </h4>
       </div>
-      <div id="collapseOne" class="panel-collapse collapse in" role="tabpanel" aria-labelledby="headingOne">
+      <div id="job_<%= job.id %>"
+           class="panel-collapse collapse in"
+           role="tabpanel"
+           aria-labelledby="jobHeading<%= job.id %>"
+      >
         <div class="panel-body">
           <div class="row">
             <div class="col-md-10">
-- 
cgit v1.2.3