Skip to content
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

Open
wants to merge 28 commits into
base: 5.2-dev
Choose a base branch
from

Conversation

coolcat-creations
Copy link
Contributor

@coolcat-creations coolcat-creations commented Oct 1, 2022

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.

grafik

Expected result AFTER applying this Pull Request

Note that I changed the class of the Privacy Request Icon to simulate the warning color.

grafik

Note that I changed the class of the left bottom quick-icon to warning for testing purposes.

Documentation Changes Required

Changed Colors of Quickicons
Added chosen colors to scss vars
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.2-dev labels Oct 1, 2022
@coolcat-creations coolcat-creations changed the title Changed Featured color to yellow instead to warning variable Change Alert Colors to Accessible Colors (AAA) for Quickicons Oct 1, 2022
@brianteeman
Copy link
Contributor

  1. See the code style issues https://ci.joomla.org/joomla/joomla-cms/58312/1/25
  2. What was wrong with the danger, info and success colours that requires the change?
  3. I am not an scss guru but this pr hard codes colours that were previously generated based on variables. I suggest speaking to @drmenzelit for assistance as I believe she worked on this

@coolcat-creations
Copy link
Contributor Author

coolcat-creations commented Oct 1, 2022

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.

@drmenzelit
Copy link
Contributor

@nadjak77 worked on the variables and hue-values for Atum

@Kostelano
Copy link
Contributor

PR changes not only the colors of the group of quick icons. Lots of changes. Colors, contrast, etc.

@coolcat-creations
Copy link
Contributor Author

Yes, what does the a11y Team think about improving the colors and therefore constrast for accessibility? @chmst

@brianteeman
Copy link
Contributor

I think you have missed the significance of @Kostelano comment

@coolcat-creations
Copy link
Contributor Author

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.

@brianteeman
Copy link
Contributor

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

@N6REJ
Copy link
Contributor

N6REJ commented Oct 1, 2022

I like the idea.

@coolcat-creations
Copy link
Contributor Author

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

Yes, because it has to fit together. I changed the color and contrast of the other quick icons as well to match the color
of the corrected quick icons. But before I don't know if we even want to make it better accessible it does not make sense to discuss the changes.

@richard67
Copy link
Member

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.

@coolcat-creations
Copy link
Contributor Author

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

@chmst
Copy link
Contributor

chmst commented Oct 3, 2022

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.
This was dammaged during all the design chnges which followed - but the right way would be repairing the hue, then calculating the colours with better contrast.

@angieradtke
Copy link
Contributor

Dear @coolcat-creations
the solution doesn't seem optimal to me.
For two reasons.

  • Your colour adjustment brings some unrest into the design. You are using colors we not used before.
  • The red and green of the font, as we have it now, is ok in terms of contrast, only the icons could be a little stronger.

For technical reasons we should only touch _quickicons.scss otherwise we will have lots of sideeffects, we can not control

@coolcat-creations
Copy link
Contributor Author

Got it, I will change it to AA colors :-)

@crystalenka
Copy link
Member

Would suggest checking contrast with upcoming APCA formulas for more accurate visual contrast:

https://www.myndex.com/APCA/

The biggest problem is with the warning/yellowish orange color, so I would focus on changes there that are consistent with the other red/green colors that do pass AA contrast.

For example with the quick icon, using this combo for the text and default background:

Text/icon: #db9508
Background: #fff7e8
Screen Shot 2022-10-04 at 11 41 10

And for the hover state:
Background: db9508
Text: #ffffff
Icon: #ebd9ae
Screen Shot 2022-10-04 at 11 40 10

@coolcat-creations
Copy link
Contributor Author

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?

@coolcat-creations
Copy link
Contributor Author

coolcat-creations commented Nov 28, 2022

So should I change the info classes to black, right? This will affect the whole dashboard. @brianteeman @crystalenka

@crystalenka
Copy link
Member

I think affecting the whole dashboard is okay at this point. It will be consistent with each other and more accessible as a whole.

@Hackwar Hackwar added the bug label Apr 6, 2023
@HLeithner HLeithner changed the base branch from 4.2-dev to 4.3-dev May 2, 2023 16:29
@HLeithner
Copy link
Member

This pull request has been automatically rebased to 4.3-dev.

@obuisard obuisard changed the base branch from 4.3-dev to 5.0-dev July 15, 2023 00:57
@obuisard obuisard added PR-5.0-dev a11y Accessibility and removed PR-4.3-dev labels Jul 15, 2023
@HLeithner HLeithner added the RMDQ ReleaseManagerDecisionQueue label Oct 5, 2023
@HLeithner HLeithner added this to the Joomla! 5.0.1 milestone Oct 7, 2023
@HLeithner HLeithner changed the title Change Alert Colors to Accessible Colors (AA) for Quickicons [5.0] Change Alert Colors to Accessible Colors (AA) for Quickicons Oct 24, 2023
@bembelimen bembelimen changed the base branch from 5.0-dev to 5.1-dev January 23, 2024 00:24
@bembelimen bembelimen removed this from the Joomla! 5.0.2 milestone Jan 23, 2024
@HLeithner HLeithner changed the title [5.0] Change Alert Colors to Accessible Colors (AA) for Quickicons [5.1] Change Alert Colors to Accessible Colors (AA) for Quickicons Apr 24, 2024
@bembelimen bembelimen changed the base branch from 5.1-dev to 5.2-dev May 11, 2024 11:23
@bembelimen bembelimen changed the title [5.1] Change Alert Colors to Accessible Colors (AA) for Quickicons [5.2] Change Alert Colors to Accessible Colors (AA) for Quickicons May 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Accessibility bug NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.2-dev RMDQ ReleaseManagerDecisionQueue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet