Skip to content

fix: equalize blog card heights in grid rows. - #2575

Open
shreyanSarkar13 wants to merge 1 commit into
expressjs:mainfrom
shreyanSarkar13:shreyan-dev
Open

shreyanSarkar13 wants to merge 1 commit into
expressjs:mainfrom
shreyanSarkar13:shreyan-dev

Conversation

@shreyanSarkar13

@shreyanSarkar13 shreyanSarkar13 commented Nov 10, 2026 •

Copy link
Copy Markdown

Summary

Blog cards on the blog index (and related-posts sections) had uneven heights
when titles wrapped to different line counts, so rows looked misaligned.

Cause

The grid already stretches each column wrapper to the row height, but
.post-card only grew to fit its own content, so cards with shorter titles
ended up shorter than their neighbours.

Fix

Set height: 100% and box-sizing: border-box on .post-card in
src/components/patterns/PostCard/PostCard.css. Cards now fill their grid
cell, and the existing flex layout keeps the author/date row pinned to the
bottom of each card.

Scope

CSS-only change to a single file. No markup, content, or behavior changes.

Screenshots

Before / After: (attach screenshots of the blog page at desktop width)

Testing

  • [✅] Blog index at desktop (3 columns), tablet (2), and mobile (1)
  • [✅] Tag filter and pagination still render correctly
  • [✅] Related-posts section on a blog post
  • [✅] Hover/focus border still looks correct

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or

(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or

(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.

(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.

@bjohansebas Please review this fix that has been made in 'Blogs' page, if possible.
Thank You.

…her across so that it never breaks in layout.
@shreyanSarkar13
shreyanSarkar13 requested a review from a team as a code owner November 10, 2026 13:08
@netlify

netlify Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit 1f48693
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/6af3174c5412dfa7387e911b
😎 Deploy Preview https://deploy-preview-2575--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 98 (🟢 up 1 from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@krzysdz krzysdz added the design Change, update, or fix for site UI (not content) label Nov 11, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

design Change, update, or fix for site UI (not content)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants