Design/UX #4205
closedNest 'Change Avatar' page in 'Commons Profile' tab
0%
Description
The ‘Change Avatar’ page is nested under the ‘Public Portfolio’ tab but still shows the ‘Commons Profile’ menu down the left side, with ‘Profile’ highlighted. Clicking ‘Public Portfolio’ tab takes user to the public view of the portfolio.
‘Change Avatar’ page should instead be nested under the ‘Commons Profile’ tab.
Files
Updated by Boone Gorges over 9 years ago
- Status changed from New to Assigned
- Target version set to 1.9
This is going to be sort of ugly to fix, so I'll do it. See #4207.
Updated by Boone Gorges about 9 years ago
- Target version changed from 1.9 to 1.10
Updated by Boone Gorges about 8 years ago
- Target version changed from 1.10 to 1.11
Updated by Boone Gorges over 7 years ago
- Target version changed from 1.11 to 1.12
Updated by Boone Gorges about 7 years ago
- File Screenshot_2017-10-10_15-44-39.png Screenshot_2017-10-10_15-44-39.png added
- Status changed from Assigned to Testing Required
Fixing this alongside #4207 meant rethinking a couple of things. I think I've got it settled in https://github.com/cuny-academic-commons/cac/commit/820363f832f0bf39fc8f67c0eefc1d963cad9e49.
In order to retain nav access to the Change Profile Photo screen, I retained the profile tab but renamed it Photo. It's visible only to logged-in users viewing their own profile. I think this will actually have the effect of increasing the use of custom avatars on the Commons, since it's a touch more visible.
This will need some review during the testing period to make sure that avatar upload etc continues to work. For the time being, I'll leave this ticket open for feedback.
Updated by Matt Gold about 7 years ago
Hi Boone -- it's kind of hard to see what is going on in that screenshot. Can you possibly upload a larger one showing more of the screen? Thank you
Updated by Boone Gorges about 7 years ago
The page is a bit of a mess, which is why I cropped the screenshot. But the relevant part is the Photo item on the sidebar nav.
Updated by Boone Gorges about 7 years ago
- Status changed from Testing Required to Resolved