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

"Monitors" terminology should not be used in GUI

    XMLWordPrintable

Details

    • 2.226

    Description

      After JENKINS-60920 (I guess?), the header bar shows <bell icon> monitors 2 and the tooltip shows

      bq . There are 2 active administrative monitors.

      We should not be using the bare term "monitor" in the GUI. This word in English has various meanings (video displays, bureaucratic police, ...) but none of them would be intuitive in this context. In some spots in the GUI like /configure we do use the phrase "administrative monitor" which is more tolerable, though something like "alert" or "notification" would be more conventional. (I think the Java API introduced AdministrativeMonitor to describe how the implementations are written—by monitoring some condition and conditionally displaying the result to administrators—but this does not well capture the user's perspective.)

      I would suggest just deleting the word from the toolbar. The bell icon suffices for people to understand that there are some alerts they can see by clicking on it.

      Attachments

        Issue Links

          is caused by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. JENKINS-60920 Update the Header and Breadcrumb styles

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          links to

          Web Link PR

          Activity

            Ascending order - Click to sort in descending order
            View 4 older comments
            romenrg Romen Rodriguez-Gil added a comment - - edited

            I'd agree with Jesse, we don't need to have text at all, just the bell and the number, as is common pattern in many other cases, such as GitHub.

            fqueiruga, are you OK if I assign this to myself as discussed in the UX SIG? I'll be happy to work on that so that I get more familiar with the Jenkins UI and can keep contributing to it.

            Edit: Just chatted with Félix and he is OK, so I'm assigning it to me

            romenrg Romen Rodriguez-Gil added a comment - - edited I'd agree with Jesse, we don't need to have text at all, just the bell and the number, as is common pattern in many other cases, such as GitHub. fqueiruga , are you OK if I assign this to myself as discussed in the UX SIG? I'll be happy to work on that so that I get more familiar with the Jenkins UI and can keep contributing to it. Edit: Just chatted with Félix and he is OK, so I'm assigning it to me
            batmat Baptiste Mathus added a comment -

            Woot <3! Gracias Romen! Go go go ! \o/

            batmat Baptiste Mathus added a comment - Woot <3! Gracias Romen! Go go go ! \o/
            romenrg Romen Rodriguez-Gil added a comment -

            PR created
            https://github.com/jenkinsci/jenkins/pull/4555

            romenrg Romen Rodriguez-Gil added a comment - PR created https://github.com/jenkinsci/jenkins/pull/4555
            romenrg Romen Rodriguez-Gil added a comment -

            Just FTR, noting here that the linked PR ended up including a color change in the monitor alert, both for the old UI and for the new. The "proposed changelog entries" were modified in the PR accordingly.

            romenrg Romen Rodriguez-Gil added a comment - Just FTR, noting here that the linked PR ended up including a color change in the monitor alert, both for the old UI and for the new. The "proposed changelog entries" were modified in the PR accordingly.
            olivergondza Oliver Gondža added a comment -

            I am removing the `lts-candidate` as the is too minor to qualify for LTS backport.

            olivergondza Oliver Gondža added a comment - I am removing the `lts-candidate` as the is too minor to qualify for LTS backport.

            People

              romenrg Romen Rodriguez-Gil
              jglick Jesse Glick
              Votes:
              2 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: