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

Unable to Activate Themes from Live Preview without Changes #60650

Closed
okmttdhr opened this issue Apr 11, 2024 · 3 comments · Fixed by #60699
Closed

Unable to Activate Themes from Live Preview without Changes #60650

okmttdhr opened this issue Apr 11, 2024 · 3 comments · Fixed by #60699
Assignees
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended

Comments

@okmttdhr
Copy link
Contributor

okmttdhr commented Apr 11, 2024

Description

When attempting to activate a theme from the Live Preview within the site editor, the activation fails if there are no other changes in the editor. Clicking on "Activate {theme}" triggers the pre-save panel (implying that there are unsaved changes, even though no changes are listed). Subsequently, attempting to confirm the activation by clicking on “Activate” has no effect, with no console errors or network requests being generated to indicate a problem.

Expected Behavior: The theme should be activated successfully, reflecting immediately in the site's appearance.

Actual Behavior: The activation process does not proceed. The pre-save panel opens without listing any changes, and clicking on “Activate” does nothing. No errors are logged to the console, and no network requests are made.

Step-by-step reproduction instructions

  • Install a Block theme and navigate to the Live Preview by clicking Live Preview button
  • Without making any other changes in the editor, click on “Activate {theme}”
  • Observe the pre-save panel opening
  • Click on “Activate” in the pre-save panel

Screenshots, screen recording, code snippet

Screen.Recording.2024-04-11.at.9.42.43.mov

Environment info

  • Gutenberg > 18.1.0

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@okmttdhr okmttdhr added the [Type] Bug An existing feature does not function as intended label Apr 11, 2024
@andrewserong
Copy link
Contributor

I'm not too familiar with the changed parts of the code, but I'm wondering if #60077 could have caused the regression? @ntsekouras I see the EntitiesSavedStatesExtensible component in that PR was updated to use saveDirtyEntities instead of saveCheckedEntities, so I'm wondering if that could have resulted in the activation button no longer working if the state otherwise has no changes in it?

@andrewserong andrewserong added the [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") label Apr 12, 2024
@annezazu
Copy link
Contributor

Just for triaging sake, noting that I can't replicate this with a clean 6.5.2 install (wanted to rule out a 6.5.x issue).

@ntsekouras
Copy link
Contributor

Thanks for reporting! I'll take a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended
4 participants