Skip to content

Refactor shared page header markup into a common include#504

Draft
Copilot wants to merge 1 commit into
mainfrom
copilot/refactor-common-includes
Draft

Refactor shared page header markup into a common include#504
Copilot wants to merge 1 commit into
mainfrom
copilot/refactor-common-includes

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 25, 2026

bio, games, and index each carried their own copy of the document/header shell, which made simple header changes fan out across multiple templates. This consolidates that shared structure into a single include while keeping per-page metadata and asset loading intact.

  • What changed

    • Added src/_includes/page-header.njk for the shared page shell:
      • document + <html>
      • common <head> scaffolding
      • shared body classes
      • GitHub ribbon
      • site header include

  • Template consolidation

    • Updated src/index.njk, src/games.njk, and src/_includes/bio.njk to:
      • define page-specific seo
      • opt into Tailwind CDN only where already used
      • include the shared page-header.njk instead of duplicating the shell

  • Behavior preserved

    • index continues using the built Tailwind stylesheet
    • games and bio continue using the Tailwind CDN path
    • Existing SEO values and page content structure remain page-local
{% set seo = {
  title: "Developer Arcade | Coding Games, XP, and Achievements",
  description: "Play coding mini-games, earn XP, and unlock achievements in the NextCommunity Developer Arcade."
} %}
{% set useTailwindCdn = true %}
{% include "page-header.njk" %}

@jbampton jbampton added this to the Turbo milestone May 25, 2026
@jbampton jbampton moved this to Review in progress in Next May 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BaseMax BaseMax Awaiting requested review from BaseMax BaseMax will be requested when the pull request is marked ready for review BaseMax is a code owner

@jbampton jbampton Awaiting requested review from jbampton jbampton will be requested when the pull request is marked ready for review jbampton is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@jbampton jbampton

Copilot code review Copilot

Labels

good first issue Good for newcomers in progress nunjucks

Projects

Next
Status: Review in progress

Milestone

Turbo

Development

Successfully merging this pull request may close these issues.

2 participants

@jbampton