#57493 closed enhancement (fixed)
Add "Robohash" support for avatar_default
Reported by: | juliobox | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 6.2 | Priority: | normal |
Severity: | trivial | Version: | |
Component: | Users | Keywords: | has-patch has-screenshots needs-user-docs |
Focuses: | Cc: |
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)
Change History (14)
This ticket was mentioned in PR #3859 on WordPress/wordpress-develop by @juliobox.
2 years ago
#1
- Keywords has-patch added
#2
in reply to:
↑ description
@
2 years ago
#3
@
2 years ago
Thanks for the ticket and patch!
I added a commit to your PR to remove an unwanted whitespace for better WPCS compliance :)
#4
@
2 years ago
- Keywords has-screenshots added
- Milestone changed from Awaiting Review to 6.2
Moving for 6.2 consideration.
#6
@
2 years ago
- Keywords needs-user-docs added
Note:
This HelpHub page needs to be updated: https://wordpress.org/support/article/settings-discussion-screen/#avatars
This ticket was mentioned in Slack in #core by audrasjb. View the logs.
2 years ago
#10
@
2 years ago
- Keywords commit removed
- Resolution fixed deleted
- Status changed from closed to reopened
Reopening to address a follow-up commit: I noticed we also need to update two function docblocks. Follow-up commit incoming.
@audrasjb commented on PR #3859:
2 years ago
#12
Committed in https://core.trac.wordpress.org/changeset/55238
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