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.

Sign up for GitHub

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

Jump to bottom

[BITV/A11Y]: Header accessibility #31239

Closed
3 tasks done
PVince81 opened this issue Feb 17, 2022 · 1 comment · Fixed by #31584
Closed
3 tasks done

[BITV/A11Y]: Header accessibility #31239

PVince81 opened this issue Feb 17, 2022 · 1 comment · Fixed by #31584
Assignees
@CarlSchwan
Milestone
Nextcloud 25

Comments

@PVince81
Copy link
Member

PVince81 commented Feb 17, 2022 •
edited by AndyScherzinger
Loading

  • Nextcloud logo should have more contrast on focus → we will change to thick outline
  • Right icons also don't have enough change between unfocused and focused
  • Increase contrast in app icons to at least 3:1
@PVince81 PVince81 added enhancement 1. to develop Accepted and waiting to be taken care of feature: accessibility labels Feb 17, 2022
@CarlSchwan
Copy link
Member

All three issues should be fixed with #31584

@AndyScherzinger AndyScherzinger mentioned this issue May 12, 2022
Merged
@AndyScherzinger AndyScherzinger linked a pull request May 12, 2022 that will close this issue
Improve accessibility with more visible focus indication for non vue apps #31584
Merged
@AndyScherzinger AndyScherzinger added this to the Nextcloud 25 milestone May 16, 2022
@AndyScherzinger AndyScherzinger removed enhancement 1. to develop Accepted and waiting to be taken care of feature: accessibility labels May 16, 2022
@AndyScherzinger AndyScherzinger changed the title [A11Y]: Header accessibility [BITV/A11Y]: Header accessibility Jul 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CarlSchwan CarlSchwan

Labels
None yet
Projects
None yet
Milestone
Nextcloud 25
Development

Successfully merging a pull request may close this issue.

Improve accessibility with more visible focus indication for non vue apps nextcloud/server
3 participants
@PVince81 @AndyScherzinger @CarlSchwan