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 tooltips to user action icons and maybe remove one icon #2002

Closed
jfrome1 opened this issue Jan 30, 2021 · 0 comments · Fixed by #2106
Closed

Add tooltips to user action icons and maybe remove one icon #2002

jfrome1 opened this issue Jan 30, 2021 · 0 comments · Fixed by #2106
Assignees
Labels
new feature A new thing that doesn't exist yet
Milestone

Comments

@jfrome1
Copy link

jfrome1 commented Jan 30, 2021

Is your feature request related to a problem? Please describe.
Next to the user name on the peer review page are two icons: envelope and gears. Their function is not clear and the second icon is relevant only to Janeway admins, it seems.

Describe the solution you'd like
Tooltips should be added when hovering the mouse arrow over them to explain. Also, it seems that the gears icon takes me to a page that requires Janeway admin rights, so the tooltip should either indicate that it is for Janeway admins only, or (even better) that icon should not be shown to people who don't have the rights to access that page.

Describe alternatives you've considered
Leave current icons unexplained and let users figure out their function only by clicking.

@jfrome1 jfrome1 added the new feature A new thing that doesn't exist yet label Jan 30, 2021
@ajrbyers ajrbyers added this to the 1.3.10 milestone Mar 24, 2021
@ajrbyers ajrbyers self-assigned this Mar 24, 2021
ajrbyers added a commit that referenced this issue Mar 24, 2021
mauromsl pushed a commit that referenced this issue Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature A new thing that doesn't exist yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants