Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Theme Showcase: Style variation incorrectly set #93826

Closed
dsas opened this issue Aug 22, 2024 · 2 comments
Closed

Theme Showcase: Style variation incorrectly set #93826

dsas opened this issue Aug 22, 2024 · 2 comments
Assignees
Labels
[Feature] Global Styles The Global Styles tools in the site editor and theme style variations. [Feature Group] Appearance & Themes Features related to the appearance of sites. [Feature] Theme Showcase The theme showcase screen in Calypso in Appearance > Themes. [Platform] Atomic [Platform] Simple [Pri] Low Address when resources are available. [Product] WordPress.com All features accessible on and related to WordPress.com. [Type] Bug [Type] Fit and Finish Issues related to the fit, finish and overall polish of WordPress.com

Comments

@dsas
Copy link
Contributor

dsas commented Aug 22, 2024

Quick summary

It's possible to choose the default (free) style variation, make no global styles changes and still have global styles upgrade nudges everywhere.

h/t @JanaMW27 for the original report (p1724264534878539-slack-C048CUFRGFQ)

Steps to reproduce

  1. Be on a free site
  2. Go to /themes
  3. Click a style variation disc on the theme showcase grid
  4. Click the default variation disc
  5. Click into the theme
  6. Activate it
  7. Look at the front-end of the site, or the site editor
Screen.Recording.2024-08-22.at.18.33.30.mov

What you expected to happen

I should see my site using the default style variation and no global styles upgrade nudges.

What actually happened

I see global styles upgrades nudges in all the usual places.

Impact

Some (< 50%)

Available workarounds?

Yes, easy to implement

If the above answer is "Yes...", outline the workaround.

There are two workarounds:

  1. Ignore the misleading upgrade nudges
  2. Use the site editor, open the gs panel and reset global styles

Platform (Simple and/or Atomic)

Simple, Atomic

Logs or notes

Presumably selecting a style variation disc, even if it's the default means that the style variation is copied into the global styles CPT and so the upgrade nudge thinks there's a need to ugprade. Probably somewhere we need to check whether the selected style variation slug is default and not do this.

@dsas dsas added [Type] Bug [Feature Group] Appearance & Themes Features related to the appearance of sites. Needs triage Ticket needs to be triaged [Feature] Theme Showcase The theme showcase screen in Calypso in Appearance > Themes. [Product] WordPress.com All features accessible on and related to WordPress.com. labels Aug 22, 2024
@github-actions github-actions bot added [Platform] Atomic [Platform] Simple [Pri] Low Address when resources are available. labels Aug 22, 2024
@dsas dsas added the [Type] Fit and Finish Issues related to the fit, finish and overall polish of WordPress.com label Aug 22, 2024
@retnonindya
Copy link

📌 REPRODUCTION RESULTS

  • Tested on Simple – Replicated

📌 FINDINGS/SCREENSHOTS/VIDEO
The live site displays upgrade nudge despite the style being used is the default style

Screenshot 2024-08-28 at 4 07 37 PM

📌 ACTIONS

  • Triaged

@retnonindya retnonindya removed the Needs triage Ticket needs to be triaged label Aug 28, 2024
@retnonindya retnonindya moved this from Needs Triage to Triaged in Automattic Prioritization: The One Board ™ Aug 28, 2024
@dsas dsas added the [Feature] Global Styles The Global Styles tools in the site editor and theme style variations. label Aug 28, 2024
@arthur791004 arthur791004 moved this from Triaged to Needs Review in Automattic Prioritization: The One Board ™ Oct 21, 2024
@arthur791004 arthur791004 moved this from Needs Review to Needs Triage in Automattic Prioritization: The One Board ™ Oct 21, 2024
@arthur791004 arthur791004 self-assigned this Oct 21, 2024
@arthur791004 arthur791004 moved this from Needs Triage to Needs Review in Automattic Prioritization: The One Board ™ Oct 21, 2024
@arthur791004
Copy link
Contributor

Closing it by #95542

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Global Styles The Global Styles tools in the site editor and theme style variations. [Feature Group] Appearance & Themes Features related to the appearance of sites. [Feature] Theme Showcase The theme showcase screen in Calypso in Appearance > Themes. [Platform] Atomic [Platform] Simple [Pri] Low Address when resources are available. [Product] WordPress.com All features accessible on and related to WordPress.com. [Type] Bug [Type] Fit and Finish Issues related to the fit, finish and overall polish of WordPress.com
Development

No branches or pull requests

3 participants