Pricing grid redesign: implement variant5 - #112492
Conversation
Jetpack Cloud Live (direct link)
Automattic for Agencies Live (direct link)
Dashboard Live (dotcom) (direct link)
|
There was a problem hiding this comment.
Pull request overview
Implements the “variant5” pricing grid redesign for the WooCommerce/Commerce bottom plan card shown during onboarding (four_plan_new_description variant), including updated visuals and layout sizing logic in the plans-grid-next package.
Changes:
- Adds a redesigned WooCommerce bottom plan card with new imagery, copy, and CTA override.
- Updates features grid layout/classes and SCSS to support the new bottom card sizing and visuals across breakpoints.
- Introduces
.webpTypeScript module typing for the plans-grid-next package and removes the old WooCommerce title override fromPlansFeaturesMain.
Reviewed changes
Copilot reviewed 5 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| packages/plans-grid-next/src/components/shared/action-button/index.tsx | Adds support for overriding button label/aria label (used by the new Commerce CTA). |
| packages/plans-grid-next/src/components/features-grid/style.scss | Styles the redesigned WooCommerce bottom card and adjusts layout sizing when a bottom card is present. |
| packages/plans-grid-next/src/components/features-grid/index.tsx | Adds a column-count class to support bottom-card max-width logic. |
| packages/plans-grid-next/src/components/features-grid/bottom-plan-card.tsx | Implements the new WooCommerce bottom card markup, copy, CTA text override, and visuals. |
| packages/plans-grid-next/src/assets.d.ts | Adds TS module declaration for importing .webp assets. |
| client/my-sites/plans-features-main/index.tsx | Removes the old WooCommerce bottom-card title override (now handled in the card component). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| const shouldUseButtonText = !! buttonText && ! current && availableForPurchase; | ||
| const primaryText = shouldUseButtonText ? buttonText : text; | ||
| const primaryAriaLabel = shouldUseButtonText ? buttonText : ariaLabel; |
|
This Pull Request is now available for translation here: https://translate.wordpress.com/deliverables/34260153 Some locales (Hebrew) have been temporarily machine-translated due to translator availability. All other translations are usually ready within a few days. Untranslated and machine-translated strings will be sent for translation next Monday and are expected to be completed by the following Friday. Thank you @dlind1 for including a screenshot in the description! This is really helpful for our translators. |
Fixes DOTCOM-17773
Proposed Changes
Why are these changes being made?
Testing Instructions
Pre-merge Checklist