Skip to content

Conversation

@ivan-ottinger
Copy link
Contributor

@ivan-ottinger ivan-ottinger commented Dec 17, 2025

Related issues

Proposed Changes

  • fix classname (which was likely changed in a recent WordPress Components update)
Before After
CleanShot 2025-12-17 at 16 11 24@2x CleanShot 2025-12-17 at 16 11 15@2x

Testing Instructions

  1. Check out the PR branch and build the app with npm install && npm start.
  2. Navigate to Add site → Start from a Blueprint.
  3. The featured blueprints should be formatted correctly.

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@ivan-ottinger ivan-ottinger self-assigned this Dec 17, 2025
@ivan-ottinger ivan-ottinger marked this pull request as ready for review December 17, 2025 15:13
@ivan-ottinger ivan-ottinger changed the title Fix Featured Blueprint card grid styling Fix Featured Blueprints card grid styling Dec 17, 2025
@ivan-ottinger ivan-ottinger requested a review from a team December 17, 2025 15:14
Copy link
Contributor

@gcsecsey gcsecsey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ivan-ottinger for identifying and quickly fixing this! 🙌

I could reproduce the issue on trunk, and I can confirm that the fix makes sense and works great! 🚢

I also checked that we're not using the dataviews-view class anywhere else within the repo, so the change most likely affected only this page.

trunk branch
Image Image

Copy link
Contributor

@epeicher epeicher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this nasty bug @ivan-ottinger 🙌 ! I have reproduced it in trunk and confirmed it is fixed as part of these changes. LGTM! :shipit:

trunk This PR
Image Image

@ivan-ottinger ivan-ottinger merged commit 10f77e0 into trunk Dec 18, 2025
11 of 14 checks passed
@ivan-ottinger ivan-ottinger deleted the fix/featured-blueprints-styling branch December 18, 2025 08:40
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.

4 participants