-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Plugins: added a tooltip to the 'add a plugin' icon-only button #3056
Conversation
So is it "Add Plugin"/"Install Plugin" or "Browse Plugins"? I know they amount to the same thing, but the text vs icon in this PR seems to give conflicting information. |
If it were only so simple Isaac. You have to first go to the plugin browser to find a plugin, then you can install it. We don't currently have plans to allow people to upload their own files so calling it 'Add a plugin' or 'Install a plugin' could lead to confusion. All we can do right now is direct them to browse the plugin repo. An alternative would be to do away with the icon-only button and replace it with a 'Browse plugins' text button. The sidebar button would have to change from 'Add' to 'Browse' also (which is somewhat ambiguous as well). I do wish there were a clear winner solution here. |
If it's browse, a Search icon seems like the best option ;) Or text, like you said. Either way, you're all geniuses, I'm sure you'll find the best solution. |
I think this is an improvement over what we have. The issue of the missing 'back' navigation once you get to the plugin browser and the 'add' language in the sidebar still need to be addressed and at which point I think we can delivery a solution that satisfies all three needs. |
LGTM. |
Plugins: added a tooltip to the 'add a plugin' icon-only button
Previously we were relying on only an icon to explain what that button did and it had no aria label for screen readers. It now has a tooltip when you hover AND a screen reader label.
This issue was raised in #2257
Thanks to @ebinnion for helping me figure out how to make this work!
Before:
There is simply no tooltip, nothing to see!
After:
