Project

General

Profile

Design/UX #4203

Change instances of 'Avatar' to 'Photo'

Added by Samantha Raddatz over 6 years ago. Updated over 6 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
User Experience
Target version:
Start date:
2015-06-25
Due date:
% Done:

0%

Estimated time:

Description

On the profile page, the profile photo is referred to as an ‘Avatar’ rather than a photo. This term should be consistent with the drop down menu option language (‘Profile Photo’).

Change all instances of ‘Avatar’ to ‘Photo’ or ‘Profile Photo.’

5changephoto.png (330 KB) 5changephoto.png see 2.2, underlined 'avatar' text should be changed to 'photo' Samantha Raddatz, 2015-06-25 10:06 AM

History

#1 Updated by Boone Gorges over 6 years ago

  • Status changed from New to Assigned
  • Assignee changed from Boone Gorges to Daniel Jones
  • Target version set to 1.9

Dan, can you go hunting please? :)

#2 Updated by Daniel Jones over 6 years ago

I think I got most of them: https://github.com/cuny-academic-commons/cac/commit/64cb1c52a6f9af04f133a84eca7b0c55bcdb3575

I wasn't able to add my changes to CAC Advanced Profiles, got an error message because it's in a submodule. There was just one instance but it's one of the more obvious ones - do you know how I can make that change? Does anything stick out here as missing to you? Think I've got it on account creation and editing, and even in the admin areas. I left references group avatars as avatar.

#3 Updated by Boone Gorges over 6 years ago

  • Status changed from Assigned to Resolved

I wasn't able to add my changes to CAC Advanced Profiles, got an error message because it's in a submodule.

This template can be overridden in our theme. (Not ideal, but meh.) Fixed in https://github.com/cuny-academic-commons/cac/commit/3996dd6658a0e04748ddda21647c0fa8f924cde4

Sam/Matt, do you want to pull this back into a minor release? My one concern about doing so is that it might obsolesce some of our documentation/screenshots.

Thanks, Dan!

#4 Updated by Samantha Raddatz over 6 years ago

This is a minor enough semantic change that I'm not too concerned about it being immediately reflected in the codex, so I think we should go for it.
There have been enough smaller updates made lately that we should review all documentation for processes soon. Probably worth considering at the end of the summer.

#5 Updated by Stephen Real over 6 years ago

I am adding Scott as a watcher on this so that he is aware of the documentation issues.

#6 Updated by scott voth over 6 years ago

I will update the appropriate documentation.

Also available in: Atom PDF