From a23ac1c89a167189ff1923114aa7e26f5041047b Mon Sep 17 00:00:00 2001
From: Michael Trimarchi <michael@amarulasolutions.com>
Date: Fri, 29 Dec 2023 09:47:49 +0100
Subject: [PATCH] TriggerMonitor: Fix grey icon anime bug and modernize icon
 animation

Use the new icon from jenkins build class and drop legacy icon

Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com>
---
 .../hudsontrigger/actions/manual/TriggerMonitor.java   | 10 +++++-----
 .../hudsontrigger/GerritCause/description.jelly        |  2 +-
 .../ManualTriggerAction/ajaxTriggerMonitor.jelly       |  6 +++---
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/main/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/actions/manual/TriggerMonitor.java b/src/main/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/actions/manual/TriggerMonitor.java
index c001b9168..22d8ccf23 100644
--- a/src/main/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/actions/manual/TriggerMonitor.java
+++ b/src/main/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/actions/manual/TriggerMonitor.java
@@ -205,17 +205,17 @@ void setBuild(Run build) {
         @SuppressWarnings("unused") //called from jelly
         public String getBallColor() {
             if (!triggerScanStarted) {
-                return BallColor.GREY.getImage();
+                return BallColor.GREY.getIconClassName();
             } else if (!triggerScanDone) {
-                return BallColor.GREY_ANIME.getImage();
+                return BallColor.GREY_ANIME.getIconClassName();
             } else if (isUnTriggered()) {
-                return BallColor.DISABLED.getImage();
+                return BallColor.DISABLED.getIconClassName();
             } else {
                 Result result = getLeastFavorableResult();
                 if (result != null) {
-                    return result.color.getImage();
+                    return result.color.getIconClassName();
                 } else {
-                    return BallColor.GREY_ANIME.getImage();
+                    return BallColor.GREY_ANIME.getIconClassName();
                 }
             }
         }
diff --git a/src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritCause/description.jelly b/src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritCause/description.jelly
index 0d197c917..fa86a9ba2 100644
--- a/src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritCause/description.jelly
+++ b/src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritCause/description.jelly
@@ -44,7 +44,7 @@
                     <j:choose>
                         <j:when test="${other.hasBuild()}">
                             <td width="18">
-                                <img src="${imagesURL}/16x16/${other.build.iconColor.image}"/>
+                                <l:icon class="${build.build.iconColor.iconClassName}"/>
                             </td>
                             <td>
                                 <a href="${rootURL}/${other.project.url}">${other.project.displayName}</a>
diff --git a/src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/actions/manual/ManualTriggerAction/ajaxTriggerMonitor.jelly b/src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/actions/manual/ManualTriggerAction/ajaxTriggerMonitor.jelly
index ee4132f6f..035795ef3 100644
--- a/src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/actions/manual/ManualTriggerAction/ajaxTriggerMonitor.jelly
+++ b/src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/actions/manual/ManualTriggerAction/ajaxTriggerMonitor.jelly
@@ -30,7 +30,7 @@
             <j:forEach items="${monitor.eventsIterator}" var="state">
                 <tr>
                     <td width="14">
-                        <img src="${imagesURL}/16x16/${state.ballColor}"/>
+                        <l:icon class="${state.ballColor}" />
                     </td>
                     <td colspan="2" style="font-weight: bold;">
                         <a href="${it.getGerritUrl(state.event)}" target="_new">
@@ -56,7 +56,7 @@
                                 <j:choose>
                                     <j:when test="${build.hasBuild()}">
                                         <td width="18">
-                                            <img src="${imagesURL}/16x16/${build.build.iconColor.image}"/>
+                                            <l:icon class="${build.build.iconColor.iconClassName}"/>
                                         </td>
                                         <td>
                                             <a href="${rootURL}/${build.project.url}">${build.project.displayName}</a>
@@ -67,7 +67,7 @@
                                     </j:when>
                                     <j:otherwise>
                                         <td width="14">
-                                            <img src="${rootURL}/images/build-status/build-status-sprite.svg#never-built" class="icon-grey-anime icon-sm"/>
+                                            <l:icon class="icon-grey-anime icon-sm"/>
                                         </td>
                                         <td>
                                             <a href="${rootURL}/${build.project.url}">${build.project.displayName}</a>