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

t:summary resolves icons against the wrong path (regression of the ionicon PR)

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • core
    • Jenkins version 2.335
    • 2.336 (Feb 22, 2022)

      This modification altered the behavior of the t:summary tag to resolve icons against the root path instead against the icon path.

      This references '<t:summary icon="icon-notepad">' in the cloudbees folder plugin, which should display a 48x48 notepad icon but results in

      I guess that can be mitigated by updating the plugins' references, but the introduced change is somewhat of breaking, because it removes the functionality to set icons via "icon-<icon>" at all, therefore it would be better to restore this functionality in core again.

      Mind taking a look janfaracik ?

          [JENKINS-67828] t:summary resolves icons against the wrong path (regression of the ionicon PR)

          Alexander Brandes created issue -
          Tim Jacomb made changes -
          Assignee New: Tim Jacomb [ timja ]
          Tim Jacomb made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Tim Jacomb made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Tim Jacomb made changes -
          Remote Link New: This issue links to "https://github.com/jenkinsci/jenkins/pull/6282 (Web Link)" [ 27401 ]
          Harry King made changes -
          Attachment New: image-2022-02-17-22-45-27-861.png [ 57283 ]
          Harry King made changes -
          Attachment Original: image-2022-02-17-22-45-27-861.png [ 57283 ]
          Harry King made changes -
          Attachment New: image-2022-02-17-22-45-58-465.png [ 57284 ]
          Mark Waite made changes -
          Link New: This issue is duplicated by JENKINS-67832 [ JENKINS-67832 ]
          Mark Waite made changes -
          Released As New: 2.336 (Feb 22, 2022)
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Fixed but Unreleased [ 10203 ]
          Alexander Brandes made changes -
          Link New: This issue duplicates JENKINS-67868 [ JENKINS-67868 ]

            timja Tim Jacomb
            notmyfault Alexander Brandes
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: