-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Shared Severity color palette (Borealis Update) #203387
Comments
An additional couple of updates in order to have full information and being able to set up a conversation in early January. First, two main principles that I would personally introduce as only guidelines for teams to follow:
I believe that if we follow these rules, giving full freedom to teams to apply other levels according to their own needs, we will end up with a consistent use of severity colors in the end. Second, we realised that teams were already experimenting with these colors and we thought it would be better to have control of the process in order to avoid ending up with tons of hard-coded color values. EUI team has been able to include these variables (as proper tokens) in their latest release. This means that we can already start using them even if, potentially, they might change in the future. This will help us a lot, being able to point something already available to engineers instead of just pointing to design work (PR) |
Following the issue raised here (#202985) and given the upcoming release of Borealis, we thought it could be a good idea to take advantage of this transitioning phase to put together a Severity color palette the could be shared across solutions.
Currently there is a little bit of inconsistency when dealing with this kind of data, the lack of clear guidelines resulted in a different use of colors across the product, which is not great for our developers and designers that every time needs to find a solution while at the same time generating a scattered user experienced.
A first mapping of cases can show us exactly this behaviour, the following image shows how different this use can be and it shows perfectly the need for a clear set of shared guidelines
Before introducing our proposal, a few constraints/requirements that need to be specified:
Critical
in Observability might means a different level of severity thanCritical
in Security)Proposal
A new set of 15 colors divided into severity groups, spanning from Red shades for Danger levels, through yellow for Warning, while providing at the same time a group of colors for neutral categories of data.
Specific color primitives and hex codes:
Few notes:
Danger 90
as highest level of severity allows us to be consistent with EUI. With this approach, the same color will be used in all contextsUsage
This color palette will be provided as a list of color from which teams can pick according to their needs. Therefore it will be needed, from teams, to be a cautious in how it gets used and to make a couple of choices.
The main principle to follow would be the following, to always use the whole extension of this scale, which means to try to distribute your own levels of severity within proper groups of colors first (yellow for warning, red for danger zones) and to distribute them as much as possible.
This should allow us to get the following results:
An example of this use, picking the same scales shown in the first image of this issue:
Which will get us this final result:
Conclusions
This proposal is meant to be a solid base of colors that could be shipped quickly and be ready, if we are lucky, for 9.0. It's not meant to be perfect nor final, the conversation is still open and I would love to hear feedback from anybody who might be interested.
I do think though that, in order to start building a good user experience, the best would be to start having this in our library and eventually adjust it according to needs, rather than wait a long time before having this available for teams.
Please feel free to add anything to think it's relevant, apologies for the long issue! 😊
The text was updated successfully, but these errors were encountered: