WordPress.org
Get WordPress

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 16 months ago

#57493 closed enhancement (fixed)

Add "Robohash" support for avatar_default

Reported by: juliobox Owned by: audrasjb
Priority: normal Milestone: 6.2
Component: Users Version:
Severity: trivial Keywords: has-patch has-screenshots needs-user-docs
Cc: Focuses:

Description

Hello

Since Gravatar is officially handling RoboHash as avatars, we should add it to WordPress.

Source: ​https://en.gravatar.com/site/implement/images/
Test: add_filter( 'pre_option_avatar_default', function() { return 'robohash'; } );
File: ​https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-admin/options-discussion.php#L276
Patch: 'robohash' => __( 'RoboHash (Generated)' ),
Github PR:
Thanks for reading.

Attachments (2)

Capture d’écran 2023-01-18 à 10.57.43.png​ (172.8 KB ) - added by audrasjb 4 years ago.
Works fine on the admin screen
Capture d’écran 2023-01-18 à 11.03.00.png​ (90.5 KB ) - added by audrasjb 4 years ago.
Works fine on front-end when robohash is selected as default avatar option

Download all attachments as: .zip

Change History (15)

This ticket was mentioned in ​PR #3859 on ​WordPress/wordpress-develop by ​@juliobox.
4 years ago #1

  • Keywords has-patch added

Add support for "RoboHash" avatars for Gravatar
This is a native option, source: ​https://en.gravatar.com/site/implement/images/

Trac ticket: https://core.trac.wordpress.org/ticket/57493#ticket

#2 in reply to: ↑ description @juliobox
4 years ago

Last edited 4 years ago by juliobox (previous) (diff)

#3 @audrasjb
4 years ago

Thanks for the ticket and patch!

I added a commit to your PR to remove an unwanted whitespace for better WPCS compliance :)

@audrasjb
4 years ago

Works fine on the admin screen

@audrasjb
4 years ago

Works fine on front-end when robohash is selected as default avatar option

#4 @audrasjb
4 years ago

  • Keywords has-screenshots added
  • Milestone Awaiting Review → 6.2

Moving for 6.2 consideration.

#5 @audrasjb
4 years ago

  • Keywords commit added

Self assigning for commit.

#6 @audrasjb
4 years ago

  • Keywords needs-user-docs added

Note:
This HelpHub page needs to be updated: ​https://wordpress.org/support/article/settings-discussion-screen/#avatars

#7 @audrasjb
4 years ago

  • Owner set to audrasjb
  • Status new → accepted

This ticket was mentioned in ​Slack in #core by audrasjb. ​View the logs.
4 years ago

#9 @audrasjb
4 years ago

  • Resolution → fixed
  • Status accepted → closed

In 55238:

Users: Add support for RoboHash Gravatar images in default user profile pics.

Gravatar includes support for RoboHash auto-generated images. This changeset adds it to the built-in feature for user profile images.

See Gravatar docs: ​https://en.gravatar.com/site/implement/images/.

Props juliobox, audrasjb.
Fixes #57493.

#10 @audrasjb
4 years ago

  • Keywords commit removed
  • Resolution fixed
  • Status closed → reopened

Reopening to address a follow-up commit: I noticed we also need to update two function docblocks. Follow-up commit incoming.

#11 @audrasjb
4 years ago

  • Resolution → fixed
  • Status reopened → closed

In 55239:

Users: Update get_avatar_url() and get_avatar() to include RoboHash support.

Follow-up to [55238].

Fixes #57493.

#13 @audrasjb
16 months ago

In 60269:

Users: Add support for Initials and Color Gravatar images in default user profile pics.

Gravatar includes support for Initials and Color auto-generated images. This changeset adds them to the built-in feature for user profile images.

Props haozi, audrasjb, getsyash, valentingrenier.
Fixes #63087.
See #57493.

Note: See TracTickets for help on using tickets.