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

Button should be named "Edit user's profile" when another account can be edited by an admin #11800

Closed
ioanarusiczki opened this issue May 22, 2018 · 5 comments
Labels
repository:addons-frontend Issue relating to addons-frontend

Comments

@ioanarusiczki
Copy link

STR:

  1. Load AMO dev and login with an admin account
  2. View any other profile page - here's an example https://addons-dev.allizom.org/en-US/firefox/user/AnaTestUsernameadmin/
  3. Observe the Edit profile button

Expected result:
"Edit user's profile" button is displayed.

Actual result:
"Edit profile" button is displayed.

Notes:
While logged in to view a personal account of any type when "Edit profile" button is clicked the Edit profile page displays "View My Profile" and "Edit my profile". When an admin views another account there is View user's profile and Edit user's profile, therefore the button that redirects to edit profile page should be "Edit user's profile".

edit user s profile

@willdurand
Copy link
Member

therefore the button that redirects to edit profile page should be "Edit user's profile".

I don't think it needs to be changed for that use case. Not a lot of users have the right to edit other profiles anyway and the copy on this button works for everyone, so I am not really inclined to change it.

@ioanarusiczki
Copy link
Author

@willdurand I just thought that it would be more clear for the admin too if "user's" is added and also because the edit profile page already mentions the "user's".
forgot to mention above - it's reproducible on AMO dev with FF60(Win10)

@willdurand
Copy link
Member

I understand but the other page has a different copy. This neutral text works in both case, so I guess that's fine.

@ioanarusiczki
Copy link
Author

@willdurand I agree it could work both ways, this was a suggestion (enhancement) as it could improve the UX but on the other hand an admin knows the pages better than anyone else.

@willdurand
Copy link
Member

Yup, I am closing this issue for now because, as you said, "an admin knows the pages better than anyone else" 😉

@KevinMind KevinMind transferred this issue from mozilla/addons-frontend May 5, 2024
@KevinMind KevinMind added repository:addons-frontend Issue relating to addons-frontend migration:2024 labels May 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
repository:addons-frontend Issue relating to addons-frontend
Projects
None yet
Development

No branches or pull requests

4 participants