Details
-
Type:
Bug
-
Status: Resolved (View Workflow)
-
Priority:
Trivial
-
Resolution: Fixed
-
Component/s: core
-
Labels:
-
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
- is caused by
-
JENKINS-60920 Update the Header and Breadcrumb styles
-
- Resolved
-
- links to
Activity
Woot <3! Gracias Romen! Go go go ! \o/
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.
I am removing the `lts-candidate` as the is too minor to qualify for LTS backport.
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