server icon indicating copy to clipboard operation
server copied to clipboard

[BITV] 9.1.4.11/6.3 - All buttons "Change scope level of ..." do not meet the contrast requirements for graphical user interface components - actual contrast is 2.8:1. (1)

Open AndyScherzinger opened this issue 2 years ago • 1 comments

  • [ ] Fix
  • [ ] Backport

97384da12ad39b1f7335e094bda19bb2

Details

https://report.bitvtest.de/default-en/d63601ac-cb34-4645-8256-66bec78964a0.html#checkpoint-3dbfb7615a-v6-n3

AndyScherzinger avatar Mar 01 '23 19:03 AndyScherzinger

Dear @jancborchardt, @nimishavijay, @marcoambrosini

Here we have a next accessibility problem. The first thought how to fix this issue was to increase a contrast for interactive element. But the problem is that this icon-button is actually a dropdown. And dropdown have to be a dropdown and not a icon-button.

Screenshot from 2023-06-22 11-06-30

  • Should i replace all this buttons with NcSelect? But i think that would look like "too much in one place".
  • And besides this we have to restructure "sections" (full name, email, phone number ...) because right now we have an input fields which are connected to there labels. Where should a dropdown (NcSelect) belong to?

Please give me a feedback. Thanks a lot!

JuliaKirschenheuter avatar Jun 22 '23 09:06 JuliaKirschenheuter

Probably action buttons are ok. I will check them again.

JuliaKirschenheuter avatar Jun 26 '23 11:06 JuliaKirschenheuter

Agreed that using a dropdown component in place of the icon wouldn't be the best solution.

Possibly this solution is out of scope, but during the initial designs for Nextcloud profile, there were some ideas for a single column layout for the settings, which would look similar to the new contacts layout. This would allow more space for using a dropdown component if required. What do you think about this?

nimishavijay avatar Jun 26 '23 22:06 nimishavijay

Thanks @nimishavijay for your answer! Do you mean that we can use dropdowns (select) instead and after some time would have a new design look for this page that everything would look much nicer with more place?

JuliaKirschenheuter avatar Jun 27 '23 14:06 JuliaKirschenheuter

@JuliaKirschenheuter With the current layout we cannot use dropdown components, because as you mentioned it will look "too much". If we switch to a different layout, then we will have enough space to use dropdowns, and everything will look nicer. If a layout change is too much work for this issue, then we won't use dropdowns and we can find another solution :) What do you think? I will share mockups of the new layout or look for a different solution depending on what's more suitable for you :)

nimishavijay avatar Jun 29 '23 01:06 nimishavijay

@nimishavijay could we probably just use an action button like everywhere?

Screenshot from 2023-07-11 17-23-09

...ant that would be already accessible

JuliaKirschenheuter avatar Jul 11 '23 15:07 JuliaKirschenheuter

Could we probably just use an action button like everywhere?

@JuliaKirschenheuter yes that would be the simplest solution and focus on fixing the reported issue – let’s go with that. :)

jancborchardt avatar Jul 12 '23 13:07 jancborchardt

I think we have a bigger problem there. It seems to me that all our action buttons have to be more visible and discoverable without focus or hover on it. For now it is only possible to recognize that we have an action button only on focus or hover state. Only an icon in place of action button is unfortunately not enough.

Some examples:

Screenshot from 2023-07-18 16-36-28 Screenshot from 2023-07-18 16-36-12 Screenshot from 2023-07-18 16-35-56 Screenshot from 2023-07-18 16-35-48 Screenshot from 2023-07-18 16-35-29

@nimishavijay @marcoambrosini Could we set an high contrast outline to each icon-only action button by default and a double outline (or something what you could suggest) to focused state? (hover state isn't as important because a cursor will be changed).

Thank you!

JuliaKirschenheuter avatar Jul 18 '23 14:07 JuliaKirschenheuter

Can we check this one with @michaelnissenbaum I would hope that it being a button component and the icon having the right contrast to be "enough". I really wouldn't want to have all these items to have a borders by default, but am happy to be proven otherwise.

AndyScherzinger avatar Jul 18 '23 14:07 AndyScherzinger

We had an information that this action buttons were not enough discoverable even if they would have higher contrast. Or would it be really enough if we just increase a contrast on icon-only action button?
@michaelnissenbaum thanks for your help!

JuliaKirschenheuter avatar Jul 18 '23 15:07 JuliaKirschenheuter

Hi @JuliaKirschenheuter. All interactive graphical controls implemented using icons, SVGs, or elements like "img" need to have a minimum contrast ratio of 3:1 with the background. They should also be clearly distinguishable without keyboard focus or mouseover interactions.

michaelnissenbaum avatar Jul 19 '23 10:07 michaelnissenbaum

@michaelnissenbaum for me your statement than translates to the icon needs to have the right contrast but there is no need to have a border around it (in an unselected/-focussed state)

AndyScherzinger avatar Jul 19 '23 10:07 AndyScherzinger

@AndyScherzinger That's correct.

michaelnissenbaum avatar Jul 19 '23 10:07 michaelnissenbaum

@nextcloud/designers fine by you if we then just raise the contrast on the svgs? I would expect yes.

AndyScherzinger avatar Jul 19 '23 10:07 AndyScherzinger

From my side yes but it should probably also become a button if not already done. Wasnt there alrready a PR that fixes this somewhere?

szaimen avatar Jul 19 '23 10:07 szaimen

From my side yes but it should probably also become a button if not already done. Wasnt there alrready a PR that fixes this somewhere?

Not yet, i will push my changes now

JuliaKirschenheuter avatar Jul 19 '23 12:07 JuliaKirschenheuter

1. As follow-up: focused (i will open a ticket for it) If an action menu is opened, action button itself have to be focused to. For now it is not a case.

Peek 2023-07-19 14-27

2. As follow-up: focused (i will open a ticket for it) not sure about styles inside opened action menu: should we set a double outline on active + focused / hovered action menu item? @marcoambrosini, @nimishavijay

Peek 2023-07-19 14-33

JuliaKirschenheuter avatar Jul 19 '23 12:07 JuliaKirschenheuter

@JuliaKirschenheuter could you point me to the aria documentation that addresses this? I couldn't find this rule.

As a rule of thumb, we want to avoid displaying those double outlines to people that use mouses, trackpads and fingers as pointing devices. As they already know which part of the page they're interacting with.

marcoambrosini avatar Jul 20 '23 06:07 marcoambrosini