Opened 7 years ago

Last modified 5 years ago

#8059 confirmed Bug

aria-invalid cleanup

Reported by: Garry Yao Owned by:
Priority: Normal Milestone:
Component: Accessibility Version: 3.6.1
Keywords: Cc:

Description

With JAWS opened:

  1. Open link dialog;
  2. Without putting anything in the URL field to trigger the invalid error;
  3. Fill in the URL field, tab to the dialog buttons and shift-tab back to the input.
  4. Ask JAWS to announce the field;
  • Actual Result: JAWS read it as invalid text input.

Change History (4)

comment:1 Changed 7 years ago by Garry Yao

Wouldn't it better to have the re-validate happens upon field blurs? I'm confused as I saw some other implementation act the same way as with us right now.

comment:2 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.6.1CKEditor 3.6.2
Status: newconfirmed

I understand the reasoning behind our current behavior and the example you've linked. The idea is that the field enter an "invalid state" *upon validation*. Such validation happens with a specific action, which is the "Ok" button in our case (alla form post).

Doing validation on blur is a bit of a pain in the ass. Firstly, it's easy to avoid the validation, by just clicking on a field after the validated one. Other than that, the user may want to work on the fields on a different order, and it sucks when you have alerts saying that you need to fill fields that you planned to fill later.

We may do an usability enhancement for accessibility anyway, based on the "on blur" idea:

  • Revalidate on blur only if the field is already invalid.
  • If it's still invalid, nothing happens and no error messages are thrown.
  • If it's valid, the ARIA related validation stuff is removed from the field.
  • The full validation will still happen on "Ok", just like now.

There is no hush on this though, as our behavior is just normal, even if still not optimal.

Last edited 7 years ago by Frederico Caldeira Knabben (previous) (diff)

comment:3 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.6.2

comment:4 Changed 5 years ago by Mike Gifford

Would be good to see this is fixed in the 4.x branch though. It may not have been a rush 2 years ago, but I'd like to see this issue closed (and fixed).

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy