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

update notice odd spacing before closing parenthesis

    XMLWordPrintable

    Details

    • Similar Issues:
    • Epic Link:
    • Released As:
      Jenkins 2.253

      Description

      New version of Jenkins (2.226) is available for download (changelog).
      [Manage Jenkins]

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          jsoref Josh Soref created issue -
          Hide
          Permalink
          danielbeck Daniel Beck added a comment -

          This exists exclusively in the popup, and not on Manage Jenkins which isn't really clear from your report (inferred from "Manage Jenkins" link).

          .page-header__hyperlinks A has a margin-right: 0.25rem; from base-styles-v2.css.

          Pinging Félix Queiruga Balado – seems like the UI refresh caught more elements than expected due to the popup.

          Show
          danielbeck Daniel Beck added a comment - This exists exclusively in the popup, and not on Manage Jenkins which isn't really clear from your report (inferred from "Manage Jenkins" link). .page-header__hyperlinks A has a margin-right: 0.25rem; from base-styles-v2.css . Pinging Félix Queiruga Balado – seems like the UI refresh caught more elements than expected due to the popup.
          Hide
          Permalink
          danielbeck Daniel Beck added a comment -

          Steps to reproduce:

          docker run -p 8080:8080 -p 50000:50000 jenkins/jenkins:2.225 and get through the setup.

          Show
          danielbeck Daniel Beck added a comment - Steps to reproduce: docker run -p 8080:8080 -p 50000:50000 jenkins/jenkins:2.225 and get through the setup.
          danielbeck Daniel Beck made changes -
          Field Original Value New Value
          Labels newbie-friendly regression
          Hide
          Permalink
          fqueiruga Félix Queiruga Balado added a comment -

          The issue comes from the fact that the HTML code for the monitors popup is located within the header and the .page-headerhyperlinks div. I already needed to make a hack so that the header styles would not apply here (https://github.com/jenkinsci/jenkins/blob/144550b2fc4ef59ce1af0abe4dace9e4fe394c89/core/src/main/resources/jenkins/management/AdministrativeMonitorsDecorator/resources.css#L92). A solution would come from resetting the margin there as well.

          Show
          fqueiruga Félix Queiruga Balado added a comment - The issue comes from the fact that the HTML code for the monitors popup is located within the header and the .page-header hyperlinks div. I already needed to make a hack so that the header styles would not apply here ( https://github.com/jenkinsci/jenkins/blob/144550b2fc4ef59ce1af0abe4dace9e4fe394c89/core/src/main/resources/jenkins/management/AdministrativeMonitorsDecorator/resources.css#L92 ). A solution would come from resetting the margin there as well.
          fqueiruga Félix Queiruga Balado made changes -
          Epic Link JENKINS-60919 [ 204316 ]
          timja Tim Jacomb made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          timja Tim Jacomb made changes -
          Status In Progress [ 3 ] In Review [ 10005 ]
          Hide
          Permalink
          timja Tim Jacomb added a comment -

          Resolved in https://github.com/jenkinsci/jenkins/pull/4898

          Show
          timja Tim Jacomb added a comment - Resolved in https://github.com/jenkinsci/jenkins/pull/4898
          timja Tim Jacomb made changes -
          Released As Jenkins 2.253
          Resolution Fixed [ 1 ]
          Status In Review [ 10005 ] Resolved [ 5 ]

            People

            Assignee:
            Unassigned Unassigned
            Reporter:
            jsoref Josh Soref
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: