Studio: Clicking the user avatar in settings should got to their WP.com profile - #5
Conversation
derekblank
left a comment
There was a problem hiding this comment.
LGTM. I was able to verify that clicking the Gravatar button opened https://wordpress.com/me successfully.
kozer
left a comment
There was a problem hiding this comment.
I tested it and works as expected!
However, when the button is in :focus or :focus-visible state, the following outline appears.
Also note that the cursor pointer is also displayed if you are not directly on the image ( due to button padding ).
This is not a big deal, but maybe we want to improve that by removing padding/margin and outline on focus.
|
Hi @kozer - I think it is because I am using the |
|
@kozer I have removed the padding at the bottom/top and left/right but left the outline for the accessibility (this is what we already have in the sidebar implementation of Gravatar as well - example below):
What do you think? |
Yes, that's great Kat! Thanks! |




Closes https://github.com/Automattic/dotcom-forge/issues/6641
Proposed Changes
This PR adds a link to Gravatar image in the settings modal that brings the user to their WordPress.com account.
Testing Instructions
nvm use && npm install && npm starthttps://wordpress.com/mePre-merge Checklist