Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-24834

CPP check report background colours not displaying for new or resolved issues since upgrade

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Since upgrading to 1.581, the background colours on the CPP check page are not showing (red for new, green for resolved). I wonder if it's down to this change in the changelog

      "Explicitly set background color of various UI elements to white. (issue 24625)"

        Attachments

          Issue Links

            Activity

            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Michal Turek
            Path:
            src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/details.jelly
            src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/index.jelly
            src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckSourceAll/index.jelly
            http://jenkins-ci.org/commit/cppcheck-plugin/cb9f0426890e0b938ca711d18698a0007a8e9fb5
            Log:
            JENKINS-24834 CPP check report background colours not displaying for new or resolved issues since upgrade

            • White background color was set to all TD elements in core's JENKINS-24625 which overwrote settings of outer TR.
            • Missing mandatory "title" element added to l:layout.
            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Michal Turek Path: src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/details.jelly src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/index.jelly src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckSourceAll/index.jelly http://jenkins-ci.org/commit/cppcheck-plugin/cb9f0426890e0b938ca711d18698a0007a8e9fb5 Log: JENKINS-24834 CPP check report background colours not displaying for new or resolved issues since upgrade White background color was set to all TD elements in core's JENKINS-24625 which overwrote settings of outer TR. Missing mandatory "title" element added to l:layout.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Michal Turek
            Path:
            src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/details.jelly
            src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/index.jelly
            src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckSourceAll/index.jelly
            http://jenkins-ci.org/commit/cppcheck-plugin/1e722026de8af8af24fd1cd9e28582f8df4a4ab6
            Log:
            Merge pull request #25 from mixalturek/master

            JENKINS-24834 CPP check report background colours not displaying for n...

            Compare: https://github.com/jenkinsci/cppcheck-plugin/compare/53b0e1f3a40a...1e722026de8a

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Michal Turek Path: src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/details.jelly src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/index.jelly src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckSourceAll/index.jelly http://jenkins-ci.org/commit/cppcheck-plugin/1e722026de8af8af24fd1cd9e28582f8df4a4ab6 Log: Merge pull request #25 from mixalturek/master JENKINS-24834 CPP check report background colours not displaying for n... Compare: https://github.com/jenkinsci/cppcheck-plugin/compare/53b0e1f3a40a...1e722026de8a
            Hide
            mixalturek Michal Turek added a comment -

            Fixed, will be released in version 1.20.

            Show
            mixalturek Michal Turek added a comment - Fixed, will be released in version 1.20.

              People

              • Assignee:
                mixalturek Michal Turek
                Reporter:
                iwonbigbro Craig Phillips
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: