-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[5.2] Change Alert Colors to Accessible Colors (AA) for Quickicons #38885
base: 5.2-dev
Are you sure you want to change the base?
[5.2] Change Alert Colors to Accessible Colors (AA) for Quickicons #38885
Conversation
Changed Colors of Quickicons Added chosen colors to scss vars
|
I thought to make consistent AAA Colors according to the recommendations in the article. Not good? I can take the AA colors as well, but I tought the more accessible the better. |
@nadjak77 worked on the variables and hue-values for Atum |
PR changes not only the colors of the group of quick icons. Lots of changes. Colors, contrast, etc. |
Yes, what does the a11y Team think about improving the colors and therefore constrast for accessibility? @chmst |
I think you have missed the significance of @Kostelano comment |
Should I aim AAA or AA @brianteeman i thought the more accessible the better. Of course changing colors to be more accessible involves changing the constrast. That's the main goal. |
that is a question for the accessibility team. the point you missed from @Kostelano comment is that this pr impacts much more tha the quickicons |
I like the idea. |
Yes, because it has to fit together. I changed the color and contrast of the other quick icons as well to match the color |
As far as I know, Joomla 4 is compliant with WCAC AA. If we want to make it AAA compliant, we should do that everywhere in the particular client (admin or site or installation) and not only at one place. |
Ok, I will change the colors to the lower accessibility standard no problem |
We don't promise AAA compliance, which does not mean that colours may not be AAA thats always fine and no need for discussion. I like the colours but I think that it is wrong to make some changes here and there without a concept. The whole colour scheme was calculated carefull by @coolcat-creations - with automatically generatng a a11y colour scheme when a user selects a colour for the template. |
Dear @coolcat-creations
For technical reasons we should only touch _quickicons.scss otherwise we will have lots of sideeffects, we can not control |
Got it, I will change it to AA colors :-) |
I changed the colors to have the same smooth appeareance like before, anyway improved some contrasts and fixed the warning color. Is it ok for you now? |
build/media_source/templates/administrator/atum/scss/_variables.scss
Outdated
Show resolved
Hide resolved
build/media_source/templates/administrator/atum/scss/blocks/_icons.scss
Outdated
Show resolved
Hide resolved
Fix variable Names
So should I change the info classes to black, right? This will affect the whole dashboard. @brianteeman @crystalenka |
I think affecting the whole dashboard is okay at this point. It will be consistent with each other and more accessible as a whole. |
This pull request has been automatically rebased to 4.3-dev. |
Changed Colors of Quickicons
Added chosen colors to scss vars
Pull Request for PR #38813
Summary of Changes
I changed the colors of the Quickicons to accessible colors (AAA) and followed this article for recommendations:
https://medium.com/handsome-perspectives/a-guide-to-color-accessibility-in-product-design-aa3e8919be0
Testing Instructions
I am not sure if it's ready to test yet.
If we have accessible colors for alerts they should be consistent through the system but it seems that the colors are managed also in an outside repo (https://github.com/joomla-projects/custom-elements), I am not sure how to change them there and test it.
Also I do not understand the logic behind how it was done before. I wanted to use the scss variables but they can't be used with css variables so I put in the values static. Which I think is wrong because the colors should be managed by scss variables in a central file.
Actual result BEFORE applying this Pull Request
Note that I changed the class of the Privacy Request Icon to simulate the warning color.
Expected result AFTER applying this Pull Request
Note that I changed the class of the Privacy Request Icon to simulate the warning color.
Note that I changed the class of the left bottom quick-icon to warning for testing purposes.
Documentation Changes Required