Skip to content

Conversation

@Copons
Copy link
Contributor

@Copons Copons commented Nov 4, 2025

Fixes DOTTHEM-115

Proposed Changes

  • Remove an unnecessary downgraded Atomic workaround from the Calypso My Themes screen.
Before After
Screenshot 2025-11-04 at 17 08 00 Screenshot 2025-11-04 at 17 08 11

Why are these changes being made?

The My Themes tab in the Calypso Theme Showcase has different behaviors on Simple and Atomic.

The idea is to show all the locally installed themes, mimicking the wp-admin counterpart. This works fine on Atomic, but on Simple, all Dotcom themes are locally installed.

For this reason, for a long time, the My Themes tab was only available on Atomic, until we started displaying it on all sites in January 2024. On Simple, instead of showing all Dotcom themes, we only show those that were previously activated on the site.

In that PR, we missed a workaround from February 2022, intended to prevent showing all themes on downgraded Atomic sites during the brief transitional period where a site was on Atomic but not on Business (when only Business was supposed to have access to Atomic features).

With our My Themes changes, that workaround was practically unreachable.

Alas, it came into play again with Summer Special opening Atomic features to all plans, creating an unexpected inconsistency.

Summer Special sites on Personal would hit that condition, accidentally showing all Dotcom themes even though the site has access to external themes.

Testing Instructions

  • Grab one Summer Special site on Personal and one on Premium.
  • Install an external theme by uploading the theme's zip.
  • Navigate to My Themes (/themes/my-themes/:SITE).
  • Ensure the screen only contains themes that are locally installed on the site, including the newly uploaded one.
  • Test for regression on an Atomic site.

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?
  • 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)?

@Copons Copons requested a review from a team November 4, 2025 17:15
@Copons Copons self-assigned this Nov 4, 2025
@Copons Copons added the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Nov 4, 2025
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

@matticbot
Copy link
Contributor

matticbot commented Nov 4, 2025

This PR modifies the release build for the following Calypso Apps:

For info about this notification, see here: PCYsg-OT6-p2

  • notifications
  • wpcom-block-editor

To test WordPress.com changes, run install-plugin.sh $pluginSlug dotthem-115-theme-showcase-my-themes-shows-all-themes-on-summer-special on your sandbox.

Copy link
Member

@mmtr mmtr left a comment

Choose a reason for hiding this comment

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

Works well for me, tested on Atomic + Personal and Atomic + Free

Installed themes Before After
Screenshot 2025-11-05 at 11 59 08 Screenshot 2025-11-05 at 11 59 10 Screenshot 2025-11-05 at 12 01 09

@Copons Copons force-pushed the dotthem-115-theme-showcase-my-themes-shows-all-themes-on-summer-special branch from 7c9e837 to e687f62 Compare November 5, 2025 12:08
@Copons Copons merged commit 56dc890 into trunk Nov 5, 2025
12 checks passed
@Copons Copons deleted the dotthem-115-theme-showcase-my-themes-shows-all-themes-on-summer-special branch November 5, 2025 13:09
@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 Nov 5, 2025
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