Fixed Issue #269 add additional social icons#371
Fixed Issue #269 add additional social icons#371TomNaughton wants to merge 7 commits intogithub:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds Twitch and Bluesky social media icons to the footer, resolving issue #269. The implementation includes new SVG icon components using official brand assets and integrates them into the existing footer navigation alongside other social media links.
- Created two new icon components (
IconTwitchandIconBluesky) with official SVG assets - Added corresponding URL and label configurations to the commons.json file
- Integrated both icons into the Footer component with proper accessibility attributes
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| public/icons/twitch.js | New Twitch icon component with official SVG from brand press pack |
| public/icons/bluesky.js | New Bluesky icon component with official SVG from brand press pack |
| content/commons.json | Added Twitch and Bluesky URL and label configurations for footer links |
| components/footer/Footer.jsx | Imported new icon components and added them to the footer social links section with proper accessibility attributes |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot open a new pull request to apply changes based on the comments in this thread |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
AndreaGriffiths11
left a comment
There was a problem hiding this comment.
TY for your contribution!
|
Hiya @abbycabs, TL;DR: PR #371 is blocked because code scanning is waiting for CodeQL results on commits 99f7f97 and 63d8331, but no CodeQL run exists for these commits (likely because the workflow doesn't trigger on fork PRs). All other checks pass. There's no run to approve or re-trigger — I need an admin to either merge with bypass or temporarily adjust the code scanning merge protection rule. Can someone unblock this? |
Link to issue
Twitch and Bluesky icons added to footer using official press pack SVGs.