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

[BITV] Share tab a11y issues #33740

Closed
10 tasks done
PVince81 opened this issue Aug 29, 2022 · 15 comments
Closed
10 tasks done

[BITV] Share tab a11y issues #33740

PVince81 opened this issue Aug 29, 2022 · 15 comments
Assignees
Labels
4. to release Ready to be released and/or waiting for tests to finish accessibility bug
Milestone

Comments

@PVince81
Copy link
Member

PVince81 commented Aug 29, 2022

Share tab accessibility issues:


@PVince81 PVince81 added bug 1. to develop Accepted and waiting to be taken care of accessibility labels Aug 29, 2022
@PVince81 PVince81 added this to the Nextcloud 25 milestone Aug 29, 2022
@blizzz blizzz modified the milestones: Nextcloud 25, Nextcloud 26 Oct 19, 2022
@Pytal
Copy link
Member

Pytal commented Nov 18, 2022

Transferred ""Copy to clipboard" should be button instead of link"

@Pytal Pytal self-assigned this Nov 18, 2022
@Pytal Pytal added 2. developing Work in progress and removed 1. to develop Accepted and waiting to be taken care of labels Nov 18, 2022
@Pytal
Copy link
Member

Pytal commented Nov 18, 2022

@Pytal
Copy link
Member

Pytal commented Nov 18, 2022

  • "Password" field needs accessible label, e.g. aria-label="password"

Not needed as screen readers already read out the placeholder message

{{ t('files_sharing', 'Enter a password') }}

@Pytal
Copy link
Member

Pytal commented Nov 18, 2022

@Pytal
Copy link
Member

Pytal commented Nov 18, 2022

@Pytal
Copy link
Member

Pytal commented Nov 25, 2022

Transferred ""Share label" needs visible text label above"

@Pytal
Copy link
Member

Pytal commented Nov 26, 2022

Transferred ""Link copied" needs to use showSuccess which includes aria-live"

@Pytal
Copy link
Member

Pytal commented Nov 29, 2022

Transferred "Entries use transparency when not active, leading to contrast issues"

@Pytal
Copy link
Member

Pytal commented Dec 1, 2022

Transferred "When leaving the share link menu, one is being put back into the file list instead of just closing the menu"

@Pytal
Copy link
Member

Pytal commented Dec 14, 2022

Transferred into nextcloud-libraries/nextcloud-vue#3526

  • Focus of search suggestions is not clearly visible, needs border instead of background
  • aria-label needs to be removed for unclickable avatars

@Pytal
Copy link
Member

Pytal commented Jan 5, 2023

Focus feedback of menu goes full width

Might be missing something but all the focus feedback in the share tab seem to be normal, is there a specific menu that needs to be changed @PVince81?

@Pytal

This comment was marked as outdated.

@PVince81
Copy link
Member Author

PVince81 commented Jan 5, 2023

@Pytal this is a rather old ticket so it's likely that some issues I mentioned were already fixed recently

@Pytal
Copy link
Member

Pytal commented Jan 5, 2023

@Pytal this is a rather old ticket so it's likely that some issues I mentioned were already fixed recently

Seems fixed after investigations

@Pytal
Copy link
Member

Pytal commented Jan 12, 2023

Transferred "Share link row should be fully focusable / clickable to open menu."

@Pytal Pytal added 4. to release Ready to be released and/or waiting for tests to finish 3. to review Waiting for reviews and removed 2. developing Work in progress 4. to release Ready to be released and/or waiting for tests to finish 3. to review Waiting for reviews labels Jan 12, 2023
@Pytal Pytal closed this as completed Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish accessibility bug
Projects
None yet
Development

No branches or pull requests

3 participants