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

"Monitors" terminology should not be used in GUI

    XMLWordPrintable

    Details

    • Similar Issues:
    • Epic Link:
    • Released As:
      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

            Activity

            Hide
            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.

            Félix Queiruga Balado, 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

            Show
            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. Félix Queiruga Balado , 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
            Hide
            batmat Baptiste Mathus added a comment -

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

            Show
            batmat Baptiste Mathus added a comment - Woot <3! Gracias Romen! Go go go ! \o/
            Show
            romenrg Romen Rodriguez-Gil added a comment - PR created https://github.com/jenkinsci/jenkins/pull/4555
            Hide
            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.

            Show
            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.
            Hide
            olivergondza Oliver Gondža added a comment -

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

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

              People

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

                Dates

                Created:
                Updated:
                Resolved: