Skip to content

ENG-2380 Extract shared section header and align Foundations section width on landing page - #1538

Open
jsmorabito wants to merge 1 commit into
mainfrom
eng-2380-extract-shared-section-header-and-align-foundations-section
Open

jsmorabito wants to merge 1 commit into
mainfrom
eng-2380-extract-shared-section-header-and-align-foundations-section

Conversation

@jsmorabito

@jsmorabito jsmorabito commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

Reviewer brief

  • Result: Foundations uses the shared SectionHeader (eyebrow "Foundations", title "A shared structure for research work in motion") and lines up with the other landing page sections.
  • Review focus: SectionHeader now renders the eyebrow conditionally with mb-3 in place of the title's mt-3, so existing headings keep the same spacing. The Foundations container goes from max-w-6xl to max-w-7xl, which scales its cards up slightly at wide desktop widths.
  • Follow-up: ENG-2316 (stacked on this PR) builds the new About section on this header.

Verification

  • pnpm ci:validate passes.
  • Checked in the browser that Foundations, Plugins and Resources share the same left edge and width at desktop width, and that existing section headings are unchanged.

Loom video

Foundation Section Alignment and Heading Updates

Scope check

  • Ran $scope-check against ENG-2380 and the final diff.
  • Scope beyond Done When: None.

Standards check

  • Ran $dg-pr-adherence-check against the final diff and PR metadata.

Local delegated full review

  • Ran a comprehensive review of the entire final diff in a subagent with a fresh context. Use $dg-delegated-full-review when no other full-review workflow is available.

🤖 Generated with Claude Code

…width on landing page

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@supabase

supabase Bot commented Oct 9, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@linear-code

linear-code Bot commented Oct 9, 2026

Copy link
Copy Markdown

ENG-2380

@vercel

vercel Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
discourse-graph Ready Ready Preview Oct 9, 2026 9:26pm UTC

Request Review

@jsmorabito
jsmorabito marked this pull request as ready for review October 9, 2026 21:36
@jsmorabito
jsmorabito requested a review from mdroidian October 9, 2026 21:36

@devin-ai-integration devin-ai-integration Bot 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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

This branch was successfully deployed

1 active deployment
Preview — 77fcbce9 Deployed Oct 9, 2026 by vercel[bot]
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.

1 participant