? Pending

User tests: Successful: Unsuccessful:

avatar astridx
astridx
14 Apr 2020

Draft for Issue #28673.

Edit: To test this draft, it is necessary to use the npm ci command, as SCSS files need to be compiled.

Summary of Changes

I made it possible, that the drop down chevron on success and danger drop downs is white (not black).
For the red danger background the Contrast Ratio is even worse. It is 3.71.

When editing a module, the white arrow also fits better with the calendar symbol at start publishing.
Modules  Breadcrumbs   test   Administration

Next you see the white chevron on a drop down with the green success background while editing a plugin. Note: The chevrons on the drop downs with the gray background are still black.

Plugins  Action Log   Joomla   test   Administration

I made this as a draft, because I am not sure, if I understand the way we want to handle multiple background elements. Perhaps someone like to teach me?

176394e 14 Apr 2020 avatar astridx draft
ad4095e 14 Apr 2020 avatar astridx draft
avatar astridx astridx - open - 14 Apr 2020
avatar astridx astridx - change - 14 Apr 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Apr 2020
Category Administration Templates (admin)
avatar Quy
Quy - comment - 14 Apr 2020

See #28463 for hint to solve the issue.

avatar richard67 richard67 - change - 14 Apr 2020
Title
[4.0] [a11y] Conrast of Drop down chevron
[4.0] [a11y] Contrast of Drop down chevron
avatar richard67 richard67 - edited - 14 Apr 2020
avatar brianteeman
brianteeman - comment - 15 Apr 2020

I dont understand why there is a new image. I though the whole point of using an svg is that you can change its colour using css eg fill: white

avatar ciar4n
ciar4n - comment - 15 Apr 2020

I dont understand why there is a new image. I though the whole point of using an svg is that you can change its colour using css eg fill: white

Because it is a background image, it is not part of the DOM and therefore, to my knowledge, can't be changed via css. The svg will need to be added inline. Eg. https://chriswrightdesign.github.io/customforms/select-box-svg/

avatar brianteeman
brianteeman - comment - 15 Apr 2020

Thanks @ciar4n I didn't know about the background image but that makes total sense

avatar astridx
astridx - comment - 15 Apr 2020

Does anyone know the reason why we sometimes use FontAwesome and sometimes SVG images?
Modules  Breadcrumbs   test   Administration(1)

avatar ciar4n
ciar4n - comment - 15 Apr 2020

In a perfect world, they should be all the same. My preference would be inline svg. If you look deep enough, you will even find some css chevrons in the mix.

avatar astridx
astridx - comment - 16 Apr 2020

@ciar4n In a perfect world, they should be all the same. My preference would be inline svg. If you look deep enough, you will even find some css chevrons in the mix.

How would you do it exactly? If we enter the svg directly in the markup, we can not reuse things, right?

  1. If you save the SVG in a file and load it with <?php echo file_get_contents("my.svg"); ?>, you could reuse things.

  2. Or we can put the svg on the end of a page like this:

<svg>
<symbol viewBox="0 0 32 32" id="myid1"><title>youtube</title><path ../>
</symbol>
<symbol viewBox="0 0 32 32" id="myid2"><title>youtube</title><path ../>
</symbol>
</svg>

Then we can use it like this

<svg role="img" class="myclass" aria-labelledby="">
<use xlink:href="#myid1"></use>
</svg>
avatar ciar4n
ciar4n - comment - 16 Apr 2020

There have been a couple of recent PRs for the frontend views on this (#28262 & #28351)... a somewhat contentious issue.

avatar infograf768
infograf768 - comment - 18 Apr 2020

What is the advantage of using svg vs icons? As I see it, it rather makes things more complex than necessary.

avatar ciar4n
ciar4n - comment - 18 Apr 2020

Size... with SVG you just load what you need rather than an entire library (css & font). Also SVG allows us to drop in custom fonts rather than been restricted to a set library.

Regardless I think Font Awesome is fine for the backend as it is just one template that will likely be ever used. The frontend is a slightly different matter.

For this PR I think ideally we change the choices icon to Font Awesome, bringing it inline with the majority of the admin. If that is not possible, then this PR as it is fixes the issue.

avatar adj9
adj9 - comment - 18 Apr 2020

I have not tested this item.

With and without patches, the field state arrow is always black.
This is in the plugin and in the articles.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28680.

avatar adj9 adj9 - test_item - 18 Apr 2020 - Not tested
avatar astridx
astridx - comment - 18 Apr 2020

@adj9 Thanks for your test. May I ask you: How did you test? Did you use the patch tester?

avatar astridx astridx - change - 18 Apr 2020
The description was changed
avatar astridx astridx - edited - 18 Apr 2020
avatar astridx
astridx - comment - 18 Apr 2020

@ciar4n @infograf768 Thanks for your opinion. I'll look at that tomorrow.

avatar Quy
Quy - comment - 22 Apr 2020

Please test PR #28755

avatar Quy Quy - change - 22 Apr 2020
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2020-04-22 11:30:48
Closed_By Quy
Labels Added: ?
avatar Quy Quy - close - 22 Apr 2020

Add a Comment

Login with GitHub to post a comment