Skip to content

fix: use CSS subgrid to ensure alignment in CallToAction#746

Merged
danielroe merged 3 commits intonpmx-dev:mainfrom
wojtekmaj:subgrid
Feb 2, 2026
Merged

fix: use CSS subgrid to ensure alignment in CallToAction#746
danielroe merged 3 commits intonpmx-dev:mainfrom
wojtekmaj:subgrid

Conversation

@wojtekmaj
Copy link
Contributor

Before:

image

After:

image

Copilot AI review requested due to automatic review settings February 2, 2026 14:19
@vercel
Copy link

vercel bot commented Feb 2, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 2, 2026 2:50pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 2, 2026 2:50pm
npmx-lunaria Ignored Ignored Feb 2, 2026 2:50pm

Request Review

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 fixes alignment issues in the CallToAction component by replacing flexbox with CSS Grid's subgrid feature. The three call-to-action cards now properly align their headers, descriptions, and action buttons across columns when viewed on larger screens.

Changes:

  • Converted parent container to use CSS Grid with explicit row sizing (auto, 1fr, auto)
  • Updated child cards to use grid-rows-subgrid to inherit parent grid rows and ensure vertical alignment
  • Applied changes only at the sm breakpoint and above, preserving mobile layout

@danielroe danielroe added this pull request to the merge queue Feb 2, 2026
Merged via the queue into npmx-dev:main with commit d13805e Feb 2, 2026
13 checks passed
@wojtekmaj wojtekmaj deleted the subgrid branch February 2, 2026 16:15
taskylizard pushed a commit to taskylizard/npmx.dev that referenced this pull request Feb 7, 2026
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.

3 participants