Skip to content

feat: move width constraints from <main> to section/content level - #86

Merged
kenjdavidson merged 4 commits into
mainfrom
copilot/modify-main-template-styling
Mar 13, 2026
Merged

feat: move width constraints from <main> to section/content level#86
kenjdavidson merged 4 commits into
mainfrom
copilot/modify-main-template-styling

Conversation

Copilot AI commented Mar 13, 2026

Copy link
Copy Markdown
Contributor
  • Explore repo structure and understand current layout/styling
  • Update BaseLayout.astro: remove horizontal constraints from <main> (now flex-1 w-full py-10)
  • Update src/pages/index.astro: add max-w-5xl mx-auto w-full px-4 to each <section> (updated for WritingCard/grid from main)
  • Update src/pages/about.astro: wrap page content in max-w-5xl mx-auto w-full px-4 container
  • Update src/pages/writing/index.astro: wrap page content in container (updated for WritingCard/grid from main)
  • Update src/pages/projects/index.astro: wrap page content in container (updated for inline project rendering from main)
  • Update src/pages/404.astro: wrap page content in container
  • Remove src/pages/projects/[slug].astro: deleted on main (projects inlined)
  • Update src/layouts/BlogPostLayout.astro: add container class to <article>
  • Update src/pages/writing/tags/index.astro: wrap page content in container
  • Update src/pages/writing/tags/[tag].astro: wrap page content in container (updated for WritingCard/grid from main)
  • Update src/pages/writing/[year]/index.astro: wrap page content in container (updated for WritingCard/grid from main)
  • Update src/pages/writing/[year]/[month]/index.astro: wrap page content in container (updated for WritingCard/grid from main)
  • Sync all other changed files from main (WritingCard component, config.ts, .gitignore, copilot-instructions, screenshots)
  • Build verified successfully (107 pages built, no errors)
Original prompt

This section details on the original issue you should resolve

<issue_title>Main template styling</issue_title>
<issue_description>I'd like to modify the main page styling, there are two keys here:

  1. I'd like the main content to be 100% of the width, this is so that if I choose to have a section with an image background, the image will take up the entire width.

  2. The sections should be updated to contain the horizontal margins/width, so that even with a full image background, the content is the correct size for the page. This will also allow for different sized sections, text sections may be different widths than coding or other sections.

Image</issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: kenjdavidson <6210735+kenjdavidson@users.noreply.github.com>
Copilot AI changed the title [WIP] Update main page styling for full-width content feat: move width constraints from <main> to section/content level Mar 13, 2026
Copilot AI requested a review from kenjdavidson March 13, 2026 14:19
@kenjdavidson
kenjdavidson marked this pull request as ready for review March 13, 2026 17:17
@kenjdavidson

Copy link
Copy Markdown
Owner

@copilot can you re-apply this task against the latest main so that I don't need to resolve conflicts?

Co-authored-by: kenjdavidson <6210735+kenjdavidson@users.noreply.github.com>

Copilot AI commented Mar 13, 2026

Copy link
Copy Markdown
Contributor Author

@copilot can you re-apply this task against the latest main so that I don't need to resolve conflicts?

Done — rebased onto the latest main in commit 877e0b0. All conflicts resolved by applying the layout changes (full-width <main>, section-level width constraints) on top of main's updates (inline project rendering, WritingCard/grid layout, [slug].astro removal, etc.). Build passes with 107 pages.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • telemetry.astro.build
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node /home/REDACTED/work/kenjdavidson.github.io/kenjdavidson.github.io/node_modules/.bin/astro build (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@kenjdavidson
kenjdavidson merged commit ff9aeb2 into main Mar 13, 2026
1 check passed
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.

Main template styling

2 participants