[BITV] 9.3.3.1/8.1 - Modale dialog "Authentication required" - If the "Password" input field is left empty, an error message appears. This error message "Failed to authenticate, please try again" is unspecific and does not give any information about which input caused the error. (1)

[AndyScherzinger]

Active users - Users - Nextcloud
https://try.nextcloud.com/ltd/a11y/index.php/settings/users

Details

https://report.bitvtest.de/default-en/d63601ac-cb34-4645-8256-66bec78964a0.html#checkpoint-c3b56e67ab-v8-n1

[JuliaKirschenheuter]

@michaelnissenbaum could you please check it if this issue is already resolved?

In my opinion it is but i want to be sure. Thanks a lot!

[JuliaKirschenheuter]

#38839

[michaelnissenbaum]

Hi @JuliaKirschenheuter. No, from my perspective, the issue still remains. This concerns the modal dialog with the error message "Failed to authenticate, please try again." The error message is unspecific and unclear in this case.

Additionally I found another issue. The button with the eye icon doesn't have alternative text - an empty "aria-label" attribute is used.

And the button with the arrow has an incorrect alternative text - aria-label="Clear text".

• Correct aria-label on action input and text field #40662

[JuliaKirschenheuter]

Thank you @michaelnissenbaum!

[JuliaKirschenheuter]

@artonge am i right that we would need much more backend differentiation for showing a detailed information?
Or could i probably just change this message:

to "your password was wrong. please try again"? Seems to be that we have only this one case.

[artonge]

I am not sure how this is an accessibility issue. There is only one field, so what's wrong is obvious in this case. No ?

In any case, we could do two things:

1. Prevent the submission of the form if the input is empty.
2. Clearly state that the password is wrong. No need for backend differentiation here, as only the password comes into play, so we know for sure that it is the password that is wrong. The current message comes from:

   server/core/src/OC/password-confirmation.js

   Lines 126 to 127 in f134244

   	config.error = t('core', 'Failed to authenticate, try again')
   	OC.PasswordConfirmation.requirePasswordConfirmation(self.callback, config)

[ShGKme]

Requires nextcloud/password-confirmation update