Social Icons: add Gravatar service by jeherve · Pull Request #56544 · WordPress/gutenberg · GitHub
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Social Icons: add Gravatar service #56544

Merged
merged 1 commit into from
Nov 30, 2023

Conversation

jeherve
Copy link
Contributor

@jeherve jeherve commented Nov 27, 2023

What?

This introduces a new Social Link icon, for the Gravatar service.

Why?

Gravatar is used in other places within WordPress, and offers profile pages that one can use to list their presence on the web. It can be useful to link to that from your blog.

How?

This adds the new icon and service, as well as tests.

Testing Instructions

  1. Go to Posts > Add New
  2. Add the Social Links block
  3. Within the block, add a new Gravatar block
  4. Publish your post
  5. View on the frontend.

Testing Instructions for Keyboard

Same as above, this behavior does not change in this PR.

Screenshots or screencast

Editor

Screenshot 2023-11-27 at 10 27 00

Frontend, when a label is added for the icon

image

This introduces a new Social Link icon, for the Gravatar service.

Gravatar is used in other places within WordPress, and offers profile pages that one can use to list their presence on the web. It can be useful to link to that from your blog.
@t-hamano t-hamano self-requested a review November 27, 2023 15:35
Copy link
Contributor

@t-hamano t-hamano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

Since Gravatar is closely associated with WordPress users, and the Gravatar page allows you to customize a lot of information about your profile, I think it makes sense to add this service.

It's totally fine in the codebase, but I'd like to ping @richtabor @jameskoster @jasmussen for design feedback.

@t-hamano t-hamano added Needs Design Feedback Needs general design feedback. [Block] Social Affects the Social Block - used to display Social Media accounts [Type] Enhancement A suggestion for improvement. labels Nov 27, 2023
@jameskoster
Copy link
Contributor

Seems reasonable to me.

Copy link
Contributor

@apeatling apeatling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All works as expected. I could add a Gravatar icon in the editor, and it displayed correctly on the front end. ✅

Editor:

Screenshot 2023-11-29 at 2 39 52 PM

Front End:

Screenshot 2023-11-29 at 2 39 56 PM

@apeatling apeatling merged commit 786fa78 into WordPress:trunk Nov 30, 2023
52 of 56 checks passed
@github-actions github-actions bot added this to the Gutenberg 17.3 milestone Nov 30, 2023
@jeherve jeherve deleted the add/gravatar-social-icon branch November 30, 2023 10:51
derekblank pushed a commit that referenced this pull request Dec 7, 2023
This introduces a new Social Link icon, for the Gravatar service.

Gravatar is used in other places within WordPress, and offers profile pages that one can use to list their presence on the web. It can be useful to link to that from your blog.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Social Affects the Social Block - used to display Social Media accounts Needs Design Feedback Needs general design feedback. [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants