Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BUG]: Accessibility issue – In Password app hidden button after the password record. #633

Open
6 tasks done
citygabor opened this issue Jan 27, 2024 · 2 comments
Open
6 tasks done
Labels
bug Something in the app is not working as expected

Comments

@citygabor
Copy link

⚠️ This issue respects the following points: ⚠️

  • This is a single bug, not a question or a configuration/webserver/proxy issue.
  • This is not a bug in the browser extension or another client.
  • This issue is not already reported on Github (I've searched it).
  • Nextcloud Server and the Passwords App is up to date. See Nextcloud Apps.
  • There are no warnings and errors reported in the Passwords App settings in the admin area
  • The following apps are not installed: Rainloop

Server Information

TrueNAS - Scale
Kobia 2023.10.01

Client Information

Client:
MS Edge version:
121.0.2277.83
OS:
Windows 11 Professional verzió 23H2 (OS Build 22631.3007)

Bug description

Hello Marius David Wieschollek ,

Server:
OS:
Truenas 2023.10.01
Password version:
2024.1.21
NextCloud version:
28.0.1

Client:
MS Edge version:
121.0.2277.83
OS:
Windows 11 Professional verzió 23H2 (OS Build 22631.3007)

Can you Please correct one missing/hidden button issue in the Password app?
I am using my screen readers app (JAWS, Narrator, NVDA) and none of them reads that 3 dots menu after the password(s) .
When I recorded at least one password in the Password app. It was very difficult to find the menu where I can move to a different folder or how can I delete it or anything like this.

Than suddenly noticed When I navigate from the bottom of the screen to the top and reading each html element in the backward order.
I noticed that my JAWS app was silent on an HTML element which is the 3 dots button for a visual person.

See the HTML elements found by JAWS screen reader app:
End of main Regio
1 password
2 hours ago
 – I assume this is the button.
Secure Graphic
AdminUser
Modified

Please make as a real button and add a label as well to read it using screen readers.

Also I ran a Accessibility Insight Fast Pass result and it found also the same:
Please find this line in the report: 1 Failed button-name Ensures buttons have discernible text

I am attaching also the report. Please rename the extension of the file from *.pdf to *.html. I was not able to upload html file.

Thanks a lot,
regards,
Gabor
FastPass_20240127_PasswordsNextcloud.pdf

Steps to reproduce

Just try to use any of the screen reader app and make the focus of that button. You will see that all of the screen reader apps will be silent.
Also run the Microsoft Accesibility Insight app with Fast Pass and you will find the related button.

Expected behavior

As usually the screen readers read the buttons when they are labeled. This would be the goal here as well to read it and not with a button without label syntax. You may name that button as "Action" Button. I belive that the 3 dots can be still keep it as it is as the html element will be labeled and the screen readers can find those labels.

Nextcloud Logs

No response

Browser Logs

No response

@citygabor citygabor added the bug Something in the app is not working as expected label Jan 27, 2024
marius-wieschollek added a commit that referenced this issue Jan 27, 2024
Signed-off-by: Marius David Wieschollek <[email protected]>
marius-wieschollek added a commit that referenced this issue Jan 28, 2024
Signed-off-by: Marius David Wieschollek <[email protected]>
marius-wieschollek added a commit that referenced this issue Jan 28, 2024
Signed-off-by: Marius David Wieschollek <[email protected]>
@marius-wieschollek
Copy link
Owner

I have added a link to the menu and all menu items so they should now be recognized by the software.

You can test this on the test server here: https://next.passwordsapp.org/login with the login data "john" and the password "PasswordsApp" (in english).

The accessibility should become better at a future point when i migrate the main area to Nextclouds own ui components which have accessibility features built in already.

@citygabor
Copy link
Author

citygabor commented Jan 28, 2024 via email

marius-wieschollek added a commit that referenced this issue Jan 28, 2024
Signed-off-by: Marius David Wieschollek <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something in the app is not working as expected
Projects
None yet
Development

No branches or pull requests

2 participants