How do I change my profile picture?

[iseulde]

Nowhere does it mention the profile picture under "Your Profile", not even under the help tab of the page. There's a link "Documentation on User Profiles" to the codex, but nothing there either. People are left with the support forum and search engines to try.

[carolinegeven]

I've worked on this on the contributor day of WordCamp Netherlands.

I've created two patches with two possible solutions to this issue. I'd love to get some feedback about the solutions. I've attached screenshots to show the different approaches.

32286-helptab.diff - added an explanation to the help tab.

32286-form.diff - added a Gravatar preview and an explanation to the profile fields.

[iCaleb]

I like this a lot. I've been asked quite a few times how to set a profile picture, and we definitely cannot assume that a user will just *know* that they should go to gravatar.com.

I'm in favor of the second patch, 32286-form.diff. However, what would be the harm in displaying this for all users on the site? I don't think a IS_PROFILE_PAGE check may be necessary.

And then I'm not good at translation functions, but I think the sprintf and printf could be merged perhaps?

[carolinegeven]

Replying to iCaleb:

I like this a lot. I've been asked quite a few times how to set a profile picture, and we definitely cannot assume that a user will just *know* that they should go to gravatar.com.

I'm in favor of the second patch, 32286-form.diff. However, what would be the harm in displaying this for all users on the site? I don't think a IS_PROFILE_PAGE check may be necessary.

And then I'm not good at translation functions, but I think the sprintf and printf could be merged perhaps?

Thanks! I appreciate the feedback!

The check isn't necessary, you're right about that. The reason I decided on the IS_PROFILE_PAGE check, is because only the user him/herself can change their profile pic on Gravatar. As a super admin, I wouldn't be able to change the profile picture for someone else. If people prefer to always display that, then I'm all for that too.

The reason I seperared the printf and sprintf, is for readability. That can be merged should that be the preferred way!

[sdavis2702]

I like this a lot. I'd rather see it in the "About Yourself" section, though... perhaps below the Biographical Info section. It may also be worth noting that the email address field is used to pull the Gravatar image as a reminder for those who already have it set up but maybe decided to use a different email address for the site. Thoughts?

[SergeyBiryukov]

32286.patch​ moves the preview to the About Yourself section: 32286.after.png​.

[iseulde]

Glad to see some interest here! Should there be a filter on the link to Gravatar? Or something so that plugins can change the text?

[mrahmadawais]

@iseulde I wonder how it never came up before. Looks pretty good @SergeyBiryukov

[SergeyBiryukov]

Let's make the text filterable.

[obrienlabs]

Adds a filter on the description and moves the IS_PROFILE_PAGE down to only display the description on IS_PROFILE_PAGE, but always show the avatar on other user edit pages

[obrienlabs]

I think it might make sense to have the avatar show when you access any user edit page, not just your own user edit page. So I'm attaching a new patch that moves the IS_PROFILE_PAGE conditional down a little to only display the description div when IS_PROFILE_PAGE is true.

I also (think) I've got the text filterable correct and hope I have the formatting correct, too. I didn't know where to add the filter callback, so I added it to the end of user-functions.php. I had to update the printf to sprintf when adding the filter portion. The printf was adding an integer to the end of the sentence.

This is my first time doing a patch submission, a little nervous and I sure hope I got everything right!

[SergeyBiryukov]

Replying to obrienlabs:

This is my first time doing a patch submission, a little nervous and I sure hope I got everything right!

Looks good, just two things:

• No need for the callback function, apply_filters() is enough.
• New filters need to be documented as per the ​documentation standards.

I also think we should always apply the filter, so that plugin authors could alter the text displayed for other users too if they want.

See 32286.2.patch​.

[SergeyBiryukov]

In 35610:

Users: Add user's Gravatar to 'About the user' section on user profile page.

Add user_profile_picture_description filter for the description text displayed under the Gravatar.

Props carolinegeven, obrienlabs, SergeyBiryukov.
Fixes #32286.

[ocean90]

Should the section be hidden if the show_avatars setting is disabled?

[SergeyBiryukov]

In 35613:

Users: Only show the profile picture if 'Show Avatars' option in Discussion Settings is enabled.

Props ocean90.
See #32286.