?
avatar brianteeman
brianteeman
24 May 2021

In atum the color for a warning is yellow and the color for danger is red which is pretty standard. However when we get to the alerts the danger is still red but the warning is an ugly brown mud color. I'm assuming that wherever the calculation is done to set the color there is a mistake but this is beyond my css.

alert warning

image

alert danger

image

alert info

image

alert success

image

avatar brianteeman brianteeman - open - 24 May 2021
avatar joomla-cms-bot joomla-cms-bot - change - 24 May 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 24 May 2021
avatar himanshu007-creator
himanshu007-creator - comment - 26 May 2021

$state-warning-text: darken($warning, 24%);
$state-warning-bg: lighten($warning, 44%);
$state-warning-border: darken($warning, 24%);

darken($warning, 24%); is responsible for this Muddish yellow color.
If we set $state-warning-text: $warning; and $state-warning-border: $warning; , $warning: is yellow then it would look like this. If this is desired, I'll send the PR right away.
ss

avatar brianteeman
brianteeman - comment - 26 May 2021

Changing the values as you suggest didnt change the text color as you show and i still get a text color of mud

Having said that I doubt the yellow on yellow has enough contrast and the mud color seems to work ok and have sufficient contrast.

Please try to create a PR and check the colors with the contrast checker https://webaim.org/resources/contrastchecker/

avatar himanshu007-creator
himanshu007-creator - comment - 27 May 2021

Yes, yellow on yellow don't have enough contrast              reference

Present Mud color have a good contrast though
2021-05-27

The contrast ratio is not good, so I didn't sent a PR. These are some valid combinations, but having such background will not establish an identical behavior with other alert styles.

avatar brianteeman
brianteeman - comment - 27 May 2021

If you can work it out so that the text is mud and the rest is yellow that would be great

avatar brianteeman brianteeman - change - 27 May 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-05-27 16:47:56
Closed_By brianteeman
avatar brianteeman brianteeman - close - 27 May 2021

Add a Comment

Login with GitHub to post a comment