act-rules.github.io icon indicating copy to clipboard operation
act-rules.github.io copied to clipboard

"Element in sequential focus order has visible focus" (oj04fd)

Open susanatx opened this issue 4 years ago • 2 comments

I think the expectation is off (bolding mine):

"For each target element, there is at least one device pixel inside the scrolling area of the viewport whose color is different when the element is focused from when it is not."

The addition of a border or increase in line weight for examples doesn't equal a change in color; it's adding something that wasn't there to begin with. As written this also seems like the entire background of the page changing when an element receives focus would pass which is not right.

There are also some general editorial changes that would help with plain language (e.g. use of the word "unicity") and punctuation consistency.

susanatx avatar May 05 '21 21:05 susanatx

Adding a border (or outline) is changing the colour of some pixels from white (background colour) to black (text colour) (or whatever other colours are used). So it is passing the rule.

The rule is indeed fairly lax in what it accepts as indicator. So is WCAG. A single pixel changing colour, or the full page changing colour, can be considered as a "visible indicator" by WCAG. We had quite some discussions on how big or small the indicator should be when writing the rule, and considered that we should stay on the safe side and accept almost anything in the rule (to avoid creating false positives). See discussions in the original PR for more details.

Jym77 avatar May 27 '21 12:05 Jym77

Hi @susanatx . The rule has changed a bit. The word color is no longer used in the expectation. I also agree with @Jym77 and think that the example you gave is covered by the rule. I'll work on the editorial changes but can you explain a bit more about the punctuation consistency?

ajanec01 avatar Sep 06 '21 16:09 ajanec01

As best I can tell this issue has been resolved.

WilcoFiers avatar Aug 31 '23 09:08 WilcoFiers