Use PNG for better transparency handling

maven-plugin/src/main/java/hudson/maven/MavenProbeAction.java

@@ -59,7 +59,7 @@ public final class MavenProbeAction implements Action {
 
     public String getIconFileName() {
         if(channel==null)   return null;
-        return "computer.gif";
+        return "computer.png";
     }
 
     public String getDisplayName() {

maven-plugin/src/main/java/hudson/maven/reporters/MavenAbstractArtifactRecord.java

@@ -169,7 +169,7 @@ public abstract class MavenAbstractArtifactRecord<T extends AbstractBuild<?,?>>
     }
 
     public final String getIconFileName() {
-        return "redo.gif";
+        return "redo.png";
     }
 
     public final String getDisplayName() {

maven-plugin/src/main/java/hudson/maven/reporters/MavenSiteArchiver.java

@@ -154,7 +154,7 @@ public class MavenSiteArchiver extends MavenReporter {
 
         public String getIconFileName() {
             if(getSiteDir(project).exists())
-                return "help.gif";
+                return "help.png";
             else
                 // hide it since we don't have site yet.
                 return null;

maven-plugin/src/main/resources/hudson/maven/MavenBuild/actions.jelly

@@ -24,7 +24,7 @@ THE SOFTWARE.
 
 <?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">
-  <l:task icon="images/24x24/gear.gif" href="${buildUrl.baseUrl}/executedMojos" title="${%Executed Mojos}" />
+  <l:task icon="images/24x24/gear.png" href="${buildUrl.baseUrl}/executedMojos" title="${%Executed Mojos}" />
   <!-- include super class definition -->
   <st:include page="/hudson/model/Actionable/actions.jelly" />
 </j:jelly>
\ No newline at end of file

maven-plugin/src/main/resources/hudson/maven/MavenBuild/executedMojos.jelly

@@ -57,7 +57,7 @@ THE SOFTWARE.
               <tr>
                 <td>
                   <a href="${m.getPluginLink(cache)}">
-                    <img src="${imagesURL}/24x24/gear.gif" alt="" height="24" width="24"/>
+                    <img src="${imagesURL}/24x24/gear.png" alt="" height="24" width="24"/>
                     ${m.groupId}:${m.artifactId}
                   </a>
                 </td>
@@ -70,7 +70,7 @@ THE SOFTWARE.
                 <td align="center">${m.readableExecutionId}</td>
                 <td data="${m.duration}">${m.durationString}</td>
                 <td align="center">
-                  <a href="${rootURL}/fingerprint/${m.digest}/"><img src="${imagesURL}/16x16/fingerprint.gif" alt="${%fingerprint}" height="16" width="16" /></a>
+                  <a href="${rootURL}/fingerprint/${m.digest}/"><img src="${imagesURL}/16x16/fingerprint.png" alt="${%fingerprint}" height="16" width="16" /></a>
                 </td>
               </tr>
             </j:forEach>

maven-plugin/src/main/resources/hudson/maven/MavenModuleSet/actions.jelly

@@ -26,12 +26,12 @@ THE SOFTWARE.
 <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:if test="${it.hasDisabledModule()}">
     <l:isAdmin>
-      <l:task icon="images/24x24/delete-document.gif" href="deleteAllDisabledModules" title="${%Delete All Disabled Modules}" />
+      <l:task icon="images/24x24/delete-document.png" href="deleteAllDisabledModules" title="${%Delete All Disabled Modules}" />
     </l:isAdmin>
   </j:if>
   <!-- Icon TBD -->
   <j:set var="url" value="${h.getNearestAncestorUrl(request,it)}"/>
-  <l:task icon="images/24x24/document.gif" href="${url}/modules" title="${%Modules}" />
+  <l:task icon="images/24x24/document.png" href="${url}/modules" title="${%Modules}" />
 
   <!-- include super class definition -->
   <st:include page="/hudson/model/Actionable/actions.jelly" />

maven-plugin/src/main/resources/hudson/maven/MavenModuleSet/index.jelly

@@ -42,7 +42,7 @@ THE SOFTWARE.
             ${act.displayName}
           </t:summary>
         </j:forEach>
-        <t:summary icon="folder.gif" href="ws/" permission="${it.WORKSPACE}">
+        <t:summary icon="folder.png" href="ws/" permission="${it.WORKSPACE}">
           ${%Workspace}
         </t:summary>
 
@@ -50,13 +50,13 @@ THE SOFTWARE.
             build="${it.lastSuccessfulBuild}" baseURL="lastSuccessfulBuild/"
             permission="${it.lastSuccessfulBuild.ARTIFACTS}" />
 
-        <t:summary icon="notepad.gif" href="changes">
+        <t:summary icon="notepad.png" href="changes">
           ${%Recent Changes}
         </t:summary>
 
         <j:set var="tr" value="${it.lastBuild.testResultAction}"/>
         <j:if test="${tr!=null}">
-          <t:summary icon="clipboard.gif">
+          <t:summary icon="clipboard.png">
             <a href="lastBuild/testReport/">${%Latest Test Result}</a>
             <st:nbsp/>
             <t:test-result it="${tr}" />

maven-plugin/src/main/resources/hudson/maven/MavenModuleSetBuild/main.jelly

@@ -30,7 +30,7 @@ THE SOFTWARE.
         <j:choose>
           <j:when test="${m.value.size()==0}">
             <td class="no-wrap">
-              <img src="${imagesURL}/16x16/grey.gif" alt="" height="16" width="16"/>
+              <img src="${imagesURL}/16x16/grey.png" alt="" height="16" width="16"/>
               <st:nbsp/>${m.key.displayName} (${%noRun})
             </td><td>
             </td>
@@ -53,7 +53,7 @@ THE SOFTWARE.
           <j:otherwise>
             <!-- there are multiple builds for this module, so use a different format -->
             <td class="no-wrap">
-              <img src="${imagesURL}/16x16/empty.gif" alt="" height="16" width="16"/>
+              <img src="${imagesURL}/16x16/empty.png" alt="" height="16" width="16"/>
               <st:nbsp/>
               <a href="${rootURL}/${m.key.url}">${m.key.displayName}</a>
             </td><td>

maven-plugin/src/main/resources/hudson/maven/MavenProbeAction/sidepanel.jelly

@@ -26,11 +26,11 @@ THE SOFTWARE.
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:s="/lib/form">
   <l:side-panel>
     <l:tasks>
-      <l:task icon="images/24x24/document.gif" href="systemProperties" title="${%System Properties}" />
-      <l:task icon="images/24x24/document.gif" href="envVars" title="${%Environment Variables}" />
-      <l:task icon="images/24x24/document.gif" href="threads" title="${%Thread Dump}" />
+      <l:task icon="images/24x24/document.png" href="systemProperties" title="${%System Properties}" />
+      <l:task icon="images/24x24/document.png" href="envVars" title="${%Environment Variables}" />
+      <l:task icon="images/24x24/document.png" href="threads" title="${%Thread Dump}" />
       <l:hasPermission permission="${app.RUN_SCRIPTS}">
-        <l:task icon="images/24x24/notepad.gif" href="script" title="${%Script Console}" />
+        <l:task icon="images/24x24/notepad.png" href="script" title="${%Script Console}" />
       </l:hasPermission>
     </l:tasks>
   </l:side-panel>

maven-plugin/src/main/resources/hudson/maven/reporters/MavenAbstractArtifactRecord/badge.jelly

@@ -28,7 +28,7 @@ THE SOFTWARE.
     <a href="${link}redeploy/">
       <img width="16" height="16" alt="[deployed]"
            tooltip="${%Deployed to repository}"
-           src="${imagesURL}/16x16/redo.gif"/>
+           src="${imagesURL}/16x16/redo.png"/>
     </a>
   </j:if>
 </j:jelly>
\ No newline at end of file

maven-plugin/src/main/resources/hudson/maven/reporters/MavenAbstractArtifactRecord/index.jelly

@@ -31,7 +31,7 @@ THE SOFTWARE.
     </l:side-panel>
     <l:main-panel>
       <h1>
-        <img src="${imagesURL}/48x48/redo.gif" width="48" height="48" alt="" />
+        <img src="${imagesURL}/48x48/redo.png" width="48" height="48" alt="" />
         ${%Redeploy Artifacts}
       </h1>