From 7285b7f2208d0ff4bf0c9f6e6aa61e4e24ea5c74 Mon Sep 17 00:00:00 2001
From: Jerome Lacoste <jerome.lacoste@gmail.com>
Date: Tue, 31 Oct 2023 20:24:04 +0100
Subject: [PATCH 1/5] Fix build status icons in Executed Tasks for given build

---
 .../hudson/plugins/batch_task/BatchRun/index.jelly    | 11 ++++++-----
 .../plugins/batch_task/BatchRunAction/index.jelly     |  2 +-
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/main/resources/hudson/plugins/batch_task/BatchRun/index.jelly b/src/main/resources/hudson/plugins/batch_task/BatchRun/index.jelly
index 20e6d94..de346fc 100644
--- a/src/main/resources/hudson/plugins/batch_task/BatchRun/index.jelly
+++ b/src/main/resources/hudson/plugins/batch_task/BatchRun/index.jelly
@@ -1,5 +1,7 @@
 <!--
   Displays the execution log
+
+  can be inspired by https://github.com/jenkinsci/jenkins/blob/master/core/src/main/resources/hudson/model/Run/console.jelly
 -->
 <?jelly escape-by-default='true'?>
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form" xmlns:i="jelly:fmt">
@@ -22,16 +24,15 @@
     </l:side-panel>
     <l:main-panel>
       <h1>
-        <img src="${rootURL}/images/48x48/${it.buildStatusUrl}" width="48" height="48" alt="${it.iconColor.description}" />
-        ${%title(it.taskName)}
+          <t:buildCaption it="${it}">${%title(it.taskName)}</t:buildCaption>
       </h1>
 
       <j:choose>
         <!-- Do progressive console output -->
-        <j:when test="${it.running}">
-          <pre id="out"></pre>
+        <j:when test="${it.isLogUpdated()}">
+          <pre id="out" class="console-output"></pre>
           <div id="spinner">
-            <img src="${rootURL}/images/spinner.gif" />
+            <l:progressAnimation/>
           </div>
           <t:progressiveText href="progressiveLog" idref="out" spinner="spinner" />
         </j:when>
diff --git a/src/main/resources/hudson/plugins/batch_task/BatchRunAction/index.jelly b/src/main/resources/hudson/plugins/batch_task/BatchRunAction/index.jelly
index f9f03f7..4a5bcb9 100644
--- a/src/main/resources/hudson/plugins/batch_task/BatchRunAction/index.jelly
+++ b/src/main/resources/hudson/plugins/batch_task/BatchRunAction/index.jelly
@@ -8,7 +8,7 @@
         <j:forEach var="r" items="${it.records}">
           <tr>
             <td>
-              <img src="${rootURL}/images/32x32/${r.buildStatusUrl}" alt="${r.iconColor.description}" />
+              <t:ballColorTd it="${r.iconColor}" />
             </td><td>
               <a href="${r.id}/">${r.displayName} (${r.timestampString} ago)</a>
             </td>

From c0a669b908371de118aaacc48ecd737b8c035e04 Mon Sep 17 00:00:00 2001
From: Jerome Lacoste <jerome.lacoste@gmail.com>
Date: Tue, 31 Oct 2023 21:14:04 +0100
Subject: [PATCH 2/5] Also change the icon for executing tasks

---
 .../batch_task/BatchTaskAction/index.jelly    | 21 ++++++++++++++++---
 1 file changed, 18 insertions(+), 3 deletions(-)

diff --git a/src/main/resources/hudson/plugins/batch_task/BatchTaskAction/index.jelly b/src/main/resources/hudson/plugins/batch_task/BatchTaskAction/index.jelly
index 424a980..97b13f2 100644
--- a/src/main/resources/hudson/plugins/batch_task/BatchTaskAction/index.jelly
+++ b/src/main/resources/hudson/plugins/batch_task/BatchTaskAction/index.jelly
@@ -53,9 +53,24 @@
             </td>
             <td>
               <j:if test="${t.hasBuildPermission()}">
-                <form action="${t.name}/execute" method="post">
-                  <input type="image" src="${rootURL}/images/${iconSize}/clock.gif" tooltip="${%Execute}" />
-                </form>
+
+                <j:set var="id" value="${h.generateId()}"/>
+                <l:task href="${t.name}/execute" icon="icon-clock icon-md" onclick="${it.parameterized?null:'return build_' + id + '(this)'}" permission="${it.BUILD}" post="${!it.parameterized}" title="${%Execute}"/>
+                <script>
+                    function build_${id}(a) {
+                        fetch(a.href, {
+                          method: 'post',
+                          headers: crumb.wrap({}),
+                        })
+                        hoverNotification('${%Task scheduled}',a.parentNode);
+                        return false;
+                    }
+                </script>
+
+
+                <!--form action="${t.name}/execute" method="post">
+                  <input type="image" icon="icon-clock icon-md" tooltip="${%Execute}" />
+                </form-->
               </j:if>
             </td>
             <td></td>

From c1df76d4d55d0502d3589a3e854fabbd7d05be3c Mon Sep 17 00:00:00 2001
From: Jerome Lacoste <jerome.lacoste@gmail.com>
Date: Tue, 31 Oct 2023 21:38:37 +0100
Subject: [PATCH 3/5] Some more icon fixes

---
 .../hudson/plugins/batch_task/BatchRun/index.jelly     | 10 ++++++----
 .../plugins/batch_task/BatchTask/sidepanel.jelly       |  6 +++---
 2 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/src/main/resources/hudson/plugins/batch_task/BatchRun/index.jelly b/src/main/resources/hudson/plugins/batch_task/BatchRun/index.jelly
index de346fc..c3b1c07 100644
--- a/src/main/resources/hudson/plugins/batch_task/BatchRun/index.jelly
+++ b/src/main/resources/hudson/plugins/batch_task/BatchRun/index.jelly
@@ -4,21 +4,23 @@
   can be inspired by https://github.com/jenkinsci/jenkins/blob/master/core/src/main/resources/hudson/model/Run/console.jelly
 -->
 <?jelly escape-by-default='true'?>
-<j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form" xmlns:i="jelly:fmt">
+<j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form" xmlns:i="jelly:fmt" xmlns:p="/lib/hudson/project">
   <l:layout title="${%title(it.displayName)}" norefresh="true">
     <l:header />
     <l:side-panel>
       <l:tasks>
         <l:task icon="images/24x24/up.gif" href="../" title="${%Back to List}" />
-        <l:task icon="images/24x24/terminal.gif" href="${request.originalRequestURI}" title="${%Console}" />
+        <p:console-link/>
+        <!--l:task icon="images/24x24/terminal.gif" href="${request.originalRequestURI}" title="${%Console}" /-->
         <j:set var="isRunPath" value="${request.findAncestor(it.owner)!=null}" />
+
         <j:set var="p" value="${it.previous}" />
         <j:if test="${p!=null}">
-          <l:task icon="images/24x24/previous.gif" href="${isRunPath?rootURL+'/'+p.url:'../'+p.number}/" title="${%Previous}" />
+          <l:task contextMenu="false" href="${isRunPath?rootURL+'/'+p.url:'../'+p.number}/" icon="icon-previous icon-md" title="${%Previous}"/>
         </j:if>
         <j:set var="n" value="${it.next}" />
         <j:if test="${n!=null}">
-          <l:task icon="images/24x24/next.gif" href="${isRunPath?rootURL+'/'+n.url:'../'+n.number}/" title="${%Next}" />
+          <l:task contextMenu="false" href="${isRunPath?rootURL+'/'+n.url:'../'+n.number}/" icon="icon-next icon-md" title="${%Next}"/>
         </j:if>
       </l:tasks>
     </l:side-panel>
diff --git a/src/main/resources/hudson/plugins/batch_task/BatchTask/sidepanel.jelly b/src/main/resources/hudson/plugins/batch_task/BatchTask/sidepanel.jelly
index 04d948f..86990e1 100644
--- a/src/main/resources/hudson/plugins/batch_task/BatchTask/sidepanel.jelly
+++ b/src/main/resources/hudson/plugins/batch_task/BatchTask/sidepanel.jelly
@@ -12,14 +12,14 @@
   <l:side-panel>
     <l:tasks>
       <j:set var="url" value="${h.getNearestAncestorUrl(request,it)}"/>
-      <l:task icon="images/24x24/up.gif" href="../../" title="${%Back to Task List}" />
+      <l:task icon="icon-previous icon-md" href="../../" title="${%Back to Task List}" />
       <j:if test="${it.hasBuildPermission()}">
         <j:if test="${!it.disabled}">
-          <l:task icon="images/24x24/clock.gif" href="${url}/execute" title="${%Build Now}" />
+          <l:task icon="icon-clock icon-md" href="${url}/execute" title="${%Build Now}" />
         </j:if>
       </j:if>
       <j:if test="${it.hasDeletePermission()}">
-        <l:task icon="images/24x24/edit-delete.gif" href="${url}/delete" title="${%Delete Task}" />
+        <l:task icon="icon-edit-delete icon-md" href="${url}/delete" title="${%Delete Task}" />
       </j:if>
       <!-- Configure doesn't seem to lead anywhere -->
       <!--j:if test="${it.hasConfigurePermission()}">

From 6f82308de26f745005b7cc5bc371fa0af6a11fca Mon Sep 17 00:00:00 2001
From: Jerome Lacoste <jerome.lacoste@gmail.com>
Date: Tue, 31 Oct 2023 21:57:57 +0100
Subject: [PATCH 4/5] Some more icon fixes

---
 .../resources/hudson/plugins/batch_task/BatchRun/index.jelly | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/main/resources/hudson/plugins/batch_task/BatchRun/index.jelly b/src/main/resources/hudson/plugins/batch_task/BatchRun/index.jelly
index c3b1c07..c54d4fe 100644
--- a/src/main/resources/hudson/plugins/batch_task/BatchRun/index.jelly
+++ b/src/main/resources/hudson/plugins/batch_task/BatchRun/index.jelly
@@ -9,9 +9,8 @@
     <l:header />
     <l:side-panel>
       <l:tasks>
-        <l:task icon="images/24x24/up.gif" href="../" title="${%Back to List}" />
-        <p:console-link/>
-        <!--l:task icon="images/24x24/terminal.gif" href="${request.originalRequestURI}" title="${%Console}" /-->
+        <l:task icon="icon-up icon-md" href="../" title="${%Back to List}" />
+        <l:task icon="icon-terminal icon-md" href="${request.originalRequestURI}" title="${%Console}" />
         <j:set var="isRunPath" value="${request.findAncestor(it.owner)!=null}" />
 
         <j:set var="p" value="${it.previous}" />

From f0e387315aa73ad9b420c77412cc533a7ea94059 Mon Sep 17 00:00:00 2001
From: Jerome Lacoste <jerome.lacoste@gmail.com>
Date: Wed, 1 Nov 2023 05:11:33 +0100
Subject: [PATCH 5/5] Some more icon fixes

---
 .../batch_task/BatchTaskAction/index.jelly    | 23 ++-----------------
 1 file changed, 2 insertions(+), 21 deletions(-)

diff --git a/src/main/resources/hudson/plugins/batch_task/BatchTaskAction/index.jelly b/src/main/resources/hudson/plugins/batch_task/BatchTaskAction/index.jelly
index 97b13f2..b766d1a 100644
--- a/src/main/resources/hudson/plugins/batch_task/BatchTaskAction/index.jelly
+++ b/src/main/resources/hudson/plugins/batch_task/BatchTaskAction/index.jelly
@@ -20,7 +20,7 @@
     <l:main-panel>
       <h1>${%Batch Tasks}</h1>
       <t:setIconSize/>
-      <table class="sortable pane bigtable" id="projectstatus">
+      <table class="jenkins-table sortable" id="projectstatus">
         <tr>
           <th style="padding-left:1.5em">${%S}</th>
           <th initialSortDir="down">${%Task}</th>
@@ -52,26 +52,7 @@
               </j:choose>
             </td>
             <td>
-              <j:if test="${t.hasBuildPermission()}">
-
-                <j:set var="id" value="${h.generateId()}"/>
-                <l:task href="${t.name}/execute" icon="icon-clock icon-md" onclick="${it.parameterized?null:'return build_' + id + '(this)'}" permission="${it.BUILD}" post="${!it.parameterized}" title="${%Execute}"/>
-                <script>
-                    function build_${id}(a) {
-                        fetch(a.href, {
-                          method: 'post',
-                          headers: crumb.wrap({}),
-                        })
-                        hoverNotification('${%Task scheduled}',a.parentNode);
-                        return false;
-                    }
-                </script>
-
-
-                <!--form action="${t.name}/execute" method="post">
-                  <input type="image" icon="icon-clock icon-md" tooltip="${%Execute}" />
-                </form-->
-              </j:if>
+              <l:task href="${t.name}/execute" icon="icon-clock icon-md" permission="${it.BUILD}" post="${!it.parameterized}" title=""/> <!-- tooltip ${%Execute} -->
             </td>
             <td></td>
           </tr>