-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
introduce alt text for rename icon - fixes #12067 #12688
Conversation
Can we get this merged first #12394 ? |
@MorrisJobke rebase please |
I know ... that's the reason I changed the label ;) |
64611ff
to
1bfa405
Compare
Updated. Complete new approach. @PVince81 Is this a possible and valid solution or too hacky? |
I can also add parameters to fil the alt test right inside |
Looks good 👍 |
The inspection completed: No new issues |
🚀 Test PASSed. 🚀 |
@LukasReschke easy review one ;) |
👍 |
introduce alt text for rename icon - fixes #12067
But there is no little tooltip that indicates that this means "rename" (even not a tooltip from browser - tested with FF and Chrome)
fixes #12067
cc @owncloud/designers