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

Add ALT text to every clickable element in the top bar #9280

Closed
ChrisEdS opened this issue Jun 21, 2023 · 5 comments · Fixed by #9283
Closed

Add ALT text to every clickable element in the top bar #9280

ChrisEdS opened this issue Jun 21, 2023 · 5 comments · Fixed by #9283

Comments

@ChrisEdS
Copy link

ChrisEdS commented Jun 21, 2023

Describe the solution you'd like

Each element which is clickable should provide additional context using ALT text

Additional context

Screen.Recording.2023-06-21.at.16.56.54.mov

As you can see some elements have no alternative text when hovering

@pascalwengerter
Copy link
Contributor

@ChrisEdS can be moved to the web repo?

Plus, what would expect in term of hover texts for the

  • app switcher
  • logo (which can be changed using the theming)
  • search bar
  • user menu

@micbar
Copy link
Contributor

micbar commented Jun 21, 2023

@ChrisEdS why blue ticket?

@ChrisEdS
Copy link
Author

@ChrisEdS why blue ticket?

Need that for a customer

@ChrisEdS
Copy link
Author

ChrisEdS commented Jun 21, 2023

Plus, what would expect in term of hover texts for the

app switcher = Already implemented
logo (which can be changed using the theming) = "Back to home"
search bar = "Search filter" (the search bar itself has no "hand icon")
user menu = "Settings and more"

These are just ruff ideas

@ChrisEdS
Copy link
Author

@ChrisEdS can be moved to the web repo?

Sure, sorry for opening inside the wrong repo

@ChrisEdS ChrisEdS transferred this issue from owncloud/ocis Jun 21, 2023
@micbar micbar mentioned this issue Jul 24, 2023
68 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants