Merge pull request #25 from mixalturek/master

[JENKINS-24834] CPP check report background colours not displaying for n...
jenkinsci · Sep 27, 2014 · 1e72202 · 1e72202
2 parents 53b0e1f + cb9f042
commit 1e72202
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/details.jelly b/src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/details.jelly
@@ -44,28 +44,28 @@
                     <j:set var="inconclusiveCss" value="inconclusive"/>
                 </j:if>
 
-                <tr class="${elt.diffState.css} ${inconclusiveCss}">
-                    <td class="pane">${elt.diffState.text}</td>
-                    <td class="pane">
+                <tr>
+                    <td class="pane ${elt.diffState.css} ${inconclusiveCss}">${elt.diffState.text}</td>
+                    <td class="pane ${elt.diffState.css} ${inconclusiveCss}">
                         <j:if test="${elt.isSourceIgnored()}">
                             ${cppcheckFile.fileName}
                         </j:if>
                         <j:if test="${not elt.isSourceIgnored()}">
                             <a href="source.${cppcheckFile.key}">${cppcheckFile.fileName}</a>
                         </j:if>
                     </td>
-                    <td class="pane" data="${cppcheckFile.lineNumber}">
+                    <td class="pane ${elt.diffState.css} ${inconclusiveCss}" data="${cppcheckFile.lineNumber}">
                         <j:if test="${elt.isSourceIgnored()}">
                             ${cppcheckFile.lineNumberString}
                         </j:if>
                         <j:if test="${not elt.isSourceIgnored()}">
                             <a href="source.${cppcheckFile.key}#${cppcheckFile.linkLineNumber}">${cppcheckFile.lineNumberString}</a>
                         </j:if>
                     </td>
-                    <td class="pane">${cppcheckFile.severity}</td>
-                    <td class="pane">${cppcheckFile.cppCheckId}</td>
-                    <td class="pane">${cppcheckFile.inconclusive}</td>
-                    <td class="pane">${cppcheckFile.messageHtml}</td>
+                    <td class="pane ${elt.diffState.css} ${inconclusiveCss}">${cppcheckFile.severity}</td>
+                    <td class="pane ${elt.diffState.css} ${inconclusiveCss}">${cppcheckFile.cppCheckId}</td>
+                    <td class="pane ${elt.diffState.css} ${inconclusiveCss}">${cppcheckFile.inconclusive}</td>
+                    <td class="pane ${elt.diffState.css} ${inconclusiveCss}">${cppcheckFile.messageHtml}</td>
                 </tr>
             </j:forEach>
         </tbody>

diff --git a/src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/index.jelly b/src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/index.jelly
@@ -1,5 +1,5 @@
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:l="/lib/layout">
-    <l:layout norefresh="true">
+    <l:layout norefresh="true" title="${%Cppcheck Results}">
         <st:include it="${it.owner}" page="sidepanel.jelly"/>
         <l:main-panel>
 

diff --git a/src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckSourceAll/index.jelly b/src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckSourceAll/index.jelly
@@ -1,7 +1,7 @@
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:l="/lib/layout">
 
     <st:header name="Content-Type" value="text/html;charset=UTF-8"/>
-    <l:layout norefresh="true">
+    <l:layout norefresh="true" title="${%Cppcheck Results}">
         <st:include it="${it.owner}" page="sidepanel.jelly"/>
 
         <l:main-panel>