Skip to content

Pricing grid redesign: implement variant1#112375

Merged
dlind1 merged 2 commits into
trunkfrom
add/plans-grid-redesign-variant1
Jul 7, 2026
Merged

Pricing grid redesign: implement variant1#112375
dlind1 merged 2 commits into
trunkfrom
add/plans-grid-redesign-variant1

Conversation

@dlind1

@dlind1 dlind1 commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Fixes DOTCOM-17769

Proposed Changes

  • Show the new design in onboarding and /plans page for six_plan_new_design variant.

Onboarding flow
Screenshot 2026-07-07 at 13 38 01

Ineligible flow
Screenshot 2026-07-07 at 13 39 20

Why are these changes being made?

  • Part of the pricing grid redesign

Testing Instructions

  • Assign yourself to six_plan_new_design variant and navigate to /setup/onboarding.
  • Create a new site and confirm that the new design is shown during onboarding (without interval selector) and on /plans.
  • Create a new site from /setup/reblogging flow and confirm that the old design is shown in onboarding and on /plans.

Pre-merge Checklist

  • Has the general commit checklist been followed? (PCYsg-hS-p2)
  • Have you written new tests for your changes?
  • Have you tested the feature in Simple (P9HQHe-k8-p2), Atomic (P9HQHe-jW-p2), and self-hosted Jetpack sites (PCYsg-g6b-p2)?
  • Have you checked for TypeScript, React or other console errors?
  • For UI changes, have you tested the affected components in dark mode?
  • Have you tested accessibility for your changes? Ensure the feature remains usable with various user agents (e.g., browsers), interfaces (e.g., keyboard navigation), and assistive technologies (e.g., screen readers) (PCYsg-S3g-p2).
  • Have you used memoizing on expensive computations? More info in Memoizing with create-selector and Using memoizing selectors and Our Approach to Data
  • Have we added the "[Status] String Freeze" label as soon as any new strings were ready for translation (p4TIVU-5Jq-p2)?
    • For UI changes, have we tested the change in various languages (for example, ES, PT, FR, or DE)? The length of text and words vary significantly between languages.
  • For changes affecting Jetpack: Have we added the "[Status] Needs Privacy Updates" label if this pull request changes what data or activity we track or use (p4TIVU-aUh-p2)?

@dlind1 dlind1 marked this pull request as ready for review July 7, 2026 11:44
@dlind1 dlind1 requested a review from Copilot July 7, 2026 11:44
@matticbot matticbot added the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Jul 7, 2026

Copilot AI 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.

Pull request overview

Implements the “variant1” pricing grid redesign behavior behind the six_plan_new_design ExPlat experiment, ensuring the redesigned experience is shown during new-site onboarding while keeping the existing plan type selector behavior on non-signup (/plans) pages.

Changes:

  • Hide the plan type selector during signup when the plans grid redesign experiment is active.
  • Refine experiment eligibility for logged-in /plans pages to require both the gating flag and that the site was created via the onboarding flow.
  • Add/adjust unit tests to cover the new experiment gating and plan type selector visibility rules.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
client/my-sites/plans-features-main/test/index.jsx Adds tests ensuring the plan type selector is hidden only for redesign variants during signup.
client/my-sites/plans-features-main/index.tsx Hides the plan selector in signup when usePlansGridRedesign is enabled.
client/my-sites/plans-features-main/hooks/use-plans-grid-redesign-experiment.ts Tightens eligibility to require onboarding-created sites (plus gating flag) for logged-in flows.
client/my-sites/plans-features-main/hooks/test/use-plans-grid-redesign-experiment.test.ts Updates and expands tests to validate the new eligibility logic.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dlind1 dlind1 added this pull request to the merge queue Jul 7, 2026
Merged via the queue into trunk with commit b40dc5b Jul 7, 2026
17 checks passed
@dlind1 dlind1 deleted the add/plans-grid-redesign-variant1 branch July 7, 2026 12:06
@github-actions github-actions Bot removed the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Jul 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