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

Timestamp background color white

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      This is the theme URL I'm using:
      https://jenkins-contrib-themes.github.io/jenkins-material-theme/dist/material-light-blue.css

      Then I have a build with the timestamp and ansi color wrappers and one shell step that does this:

      #!/bin/bash
      
      echo "This looks terrible"
      echo -e "\e[31m This is red\e[0m"
      echo -e "\e[32m This doesn't reset"
      echo "The end"
      

      The attached screenshot is the build output. It does not do this with timestamper 1.8.4 (which we had previously).

      This seems to be related to JENKINS-34019

        Attachments

          Issue Links

            Activity

            Hide
            stevengbrown Steven G Brown added a comment -

            Another example can be found here: JENKINS-38573.

            Show
            stevengbrown Steven G Brown added a comment - Another example can be found here: JENKINS-38573 .
            Hide
            stevengbrown Steven G Brown added a comment -

            I'm at a loss. Any ideas how to fix this without reintroducing JENKINS-34019 ?

            Show
            stevengbrown Steven G Brown added a comment - I'm at a loss. Any ideas how to fix this without reintroducing JENKINS-34019 ?
            Show
            dankirkd Daniel Kirkdorffer added a comment - - edited Steven G Brown Why can't you use the suggestion Evan Borgstrom made in the comments of JENKINS-34019 and here https://github.com/jenkinsci/timestamper-plugin/commit/c31dc9571d5d1121d3f7e43d81059e0818e58b7a#commitcomment-19199648?
            Hide
            stevengbrown Steven G Brown added a comment -

            That would fix this issue, but reintroduce JENKINS-34019. (The white background for the timestamps overrides the styling applied by the AnsiColor plugin.) I'd like to have a plan for dealing with both.

            Show
            stevengbrown Steven G Brown added a comment - That would fix this issue, but reintroduce JENKINS-34019 . (The white background for the timestamps overrides the styling applied by the AnsiColor plugin.) I'd like to have a plan for dealing with both.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Steven Brown
            Path:
            src/main/resources/hudson/plugins/timestamper/annotator/TimestampAnnotatorFactory2/style.css
            http://jenkins-ci.org/commit/timestamper-plugin/6b763a9b72afa21d7f3fe3a8e850612029bac335
            Log:
            JENKINS-38390 JENKINS-34019 Revert timestamps style added by c31dc95

            The style matched the default Jenkins theme, but clashed with custom themes.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Steven Brown Path: src/main/resources/hudson/plugins/timestamper/annotator/TimestampAnnotatorFactory2/style.css http://jenkins-ci.org/commit/timestamper-plugin/6b763a9b72afa21d7f3fe3a8e850612029bac335 Log: JENKINS-38390 JENKINS-34019 Revert timestamps style added by c31dc95 The style matched the default Jenkins theme, but clashed with custom themes.
            Hide
            stevengbrown Steven G Brown added a comment -

            All changes made under JENKINS-34019 have been reverted. I have an idea about how it can be fixed within the AnsiColor plugin instead.

            Custom themes should look normal again.

            Fix released with Timestamper 1.8.7.

            Show
            stevengbrown Steven G Brown added a comment - All changes made under JENKINS-34019 have been reverted. I have an idea about how it can be fixed within the AnsiColor plugin instead. Custom themes should look normal again. Fix released with Timestamper 1.8.7.
            Hide
            dankirkd Daniel Kirkdorffer added a comment -

            Fixed verified. Thanks.

            Show
            dankirkd Daniel Kirkdorffer added a comment - Fixed verified. Thanks.

              People

              • Assignee:
                stevengbrown Steven G Brown
                Reporter:
                b_dean Ben Dean
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: