-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Missing social icons #21492
Missing social icons #21492
Conversation
Generated by 🚫 Danger |
📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
|
📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## trunk #21492 +/- ##
=======================================
Coverage 39.44% 39.44%
=======================================
Files 2121 2121
Lines 99564 99564
Branches 15313 15313
=======================================
Hits 39277 39277
Misses 56806 56806
Partials 3481 3481 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for addressing this. 🙇🏻♂️
These icons are better than an empty space. However, I can't help but note that all the existing icons are contained within a filled circle. Ideally we update this for consistency at some point.
I'm not sure we can. Nextdoor's brand guidelines state, "Nextdoor trademarks must be used as provided by Nextdoor with no modifications." Instagram is similar, "Anyone using Instagram’s assets should only use the logos and screenshots found on our Brand Resource Center site and follow these guidelines." |
Sure. We could either ignore the guidelines or remove the existing modifications—i.e., the circles surrounding all other logos. I'd be fine with the latter. Side note: I'm sure these companies' brand guidelines likely include information discouraging colorizing the logos, which we are doing based on each connection's status. |
Good call. I reduced it to 64 x 64. |
Quality Gate passedIssues Measures |
Fixes #21490
Our social feature was missing icons for Bluesky, Nextdoor, and Threads. This PR adds the missing icons, and also adds a generic icon for services that don't have an icon in the future.
To test:
To view the icons as they appear when enabled, you can change this line from
> 0
to== 0
.