Skip to content

Fixed Issue #269 add additional social icons#371

Open
TomNaughton wants to merge 7 commits intogithub:mainfrom
TomNaughton:269-Add-additional-social-icons
Open

Fixed Issue #269 add additional social icons#371
TomNaughton wants to merge 7 commits intogithub:mainfrom
TomNaughton:269-Add-additional-social-icons

Conversation

@TomNaughton
Copy link

Link to issue

Twitch and Bluesky icons added to footer using official press pack SVGs.

image

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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 (IconTwitch and IconBluesky) 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.

@TomNaughton
Copy link
Author

@copilot open a new pull request to apply changes based on the comments in this thread

AndreaGriffiths11 and others added 5 commits March 12, 2026 11:55
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>
Copy link
Contributor

@AndreaGriffiths11 AndreaGriffiths11 left a comment

Choose a reason for hiding this comment

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

TY for your contribution!

@AndreaGriffiths11
Copy link
Contributor

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants