mirror of
https://github.com/dani-garcia/vaultwarden.git
synced 2025-12-10 17:23:04 +03:00
admin diagnostics page: update badges not consistent #610
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @tessus on GitHub.
Subject of the issue
Deployment environment
Your environment (Generated via diagnostics page)
Config (Generated via diagnostics page)
Show Running Config
Environment settings which are overridden:
Steps to reproduce
/admin/diagnosticsExpected behaviour
The badges should look the same.
Actual behaviour
The badges use different color for the font. (background is the same)
Troubleshooting data
Please let me know what the correct badge is: Black or white font?
I can create a PR, unless the devs are really bored....
@stefan0xC commented on GitHub:
I think it should be black for both (better contrast). (How does it look in dark mode?)
@Gerardv514 commented on GitHub:
If the suggestion for the above screenshot is to make the label black text for both, will that also include the other labels on the admin diagnostic page?
I saw the below screenshot from another thread here, and noticed the others are white text.
Or is it just the yellow update badges that will be black text?
@tessus commented on GitHub:
Yep, just the yellow ones. The green ones are clearly legible with the white text, since green is much darker. With yellow the difference in hue between badge and text is too low.
e.g. the other badges (server installed: Update and Branched) already use black text.
We could have also made the text for all badges white, but I believe that would make the badges less legible.
@BlackDex commented on GitHub:
Indeed. That should be the same thx for reporting!