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

update notice odd spacing before closing parenthesis

    XMLWordPrintable

Details

    • 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
          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 fqueiruga – seems like the UI refresh caught more elements than expected due to the popup.

          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 fqueiruga – seems like the UI refresh caught more elements than expected due to the popup.
          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 added a comment - Steps to reproduce: docker run -p 8080:8080 -p 50000:50000 jenkins/jenkins:2.225 and get through the setup.
          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.

          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.
          timja Tim Jacomb added a comment -

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

          timja Tim Jacomb added a comment - Resolved in https://github.com/jenkinsci/jenkins/pull/4898

          People

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

            Dates

              Created:
              Updated:
              Resolved: