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

Job trend page shows previous icons and use previous icon color for trend graph

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • core
    • None
    • Jenkins 2.289.1
    • 2.312, 2.341

      https://github.com/jenkinsci/jenkins/pull/5065 added new build status icons, but missed to also update the job trend page icons.

      Also the trend graph uses the colors from the previous icons and should be updated to use the newer icon colors.

          [JENKINS-65928] Job trend page shows previous icons and use previous icon color for trend graph

          Mark Waite added a comment -

          Thanks for reporting the issue. I've confirmed that the issue is visible as described. Are you interested in submitting a pull request to correct the issue?

          Mark Waite added a comment - Thanks for reporting the issue. I've confirmed that the issue is visible as described. Are you interested in submitting a pull request to correct the issue?

          markewaite sadly I'm not familiar with Java, so I'm not able to fix this problem.

          Christian Loos added a comment - markewaite sadly I'm not familiar with Java, so I'm not able to fix this problem.

          benebsiny I reviewed your changes and it looks like you missed the second part of this issue.

          The trend graph still shows the wrong color.
          At least for successful builds the color should be changed from blue to green.
          Maybe also the colors for unstable and aborted builds can be adjusted to match the icon color.

          Christian Loos added a comment - benebsiny I reviewed your changes and it looks like you missed the second part of this issue. The trend graph still shows the wrong color. At least for successful builds the color should be changed from blue to green. Maybe also the colors for unstable and aborted builds can be adjusted to match the icon color.

          Benny Cheng added a comment -

          cloos I missed the notification of your new message, so sorry. I mistook the trend graph for timeline graph, so I didn't fix it in the last pull request. Now I make a new pull request for the new color in the trend graph. Have a look at it. https://github.com/jenkinsci/jenkins/pull/5836

          Benny Cheng added a comment - cloos I missed the notification of your new message, so sorry. I mistook the trend graph for timeline graph, so I didn't fix it in the last pull request. Now I make a new pull request for the new color in the trend graph. Have a look at it. https://github.com/jenkinsci/jenkins/pull/5836

            benebsiny Benny Cheng
            cloos Christian Loos
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: