Skip to content

fix: compact footer layout and remove duplicate links - #34

Merged
sebastian-noel merged 1 commit into
mainfrom
fix/footer
Oct 23, 2026
Merged

sebastian-noel merged 1 commit into
mainfrom
fix/footer

Conversation

@sebastian-noel

@sebastian-noel sebastian-noel commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Pull Request

Description

The footer used excessive vertical space and repeated social links. Make it more compact with a larger current logo, shorter club description, and responsive layout.

Type of Change

  • feat: New feature
  • fix: Bug fix
  • chore: Maintenance (dependencies, config, etc.)
  • docs: Documentation only
  • style: Code style (formatting, no logic change)
  • refactor: Code refactor (no new feature or fix)

Changes Made

  • Use the new GPK logo at a larger size and reduce footer padding and link spacing.
  • Replace the Connect column with one labeled social-icon row, retaining Discord and YouTube.
  • Remove the duplicate Challenges Archive link and simplify the club description.
  • Stack content on mobile and add visible keyboard focus styles.

Screenshots

gpk-footer-320 gpk-footer-390 gpk-footer-1440

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have tested my changes locally (1440px, 390px, and 320px widths)
  • npm run lint passes without errors
  • npm run build completes successfully

Related Issues

N/A

Copilot AI balanced review requested due to automatic review settings October 23, 2026 03:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@sebastian-noel
sebastian-noel merged commit 6fd00f1 into main Oct 23, 2026
2 checks passed
@sebastian-noel
sebastian-noel deleted the fix/footer branch October 23, 2026 03:05
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.

2 participants