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

Themes: inconsistent exit behavior exiting the Customizer after changing a theme #68

Closed
lancewillett opened this issue Nov 18, 2015 · 1 comment
Labels
[Feature] Customizer The site customizer for traditional, non-block themes. [Feature Group] Appearance & Themes Features related to the appearance of sites. [Type] Bug When a feature is broken and / or not performing as intended

Comments

@lancewillett
Copy link
Contributor

This issue is a series of related Customizer bugs that happen in Calypso, specifically when exiting the Customizer after changing a theme.

Issue 1:

1/ Go to Calypso single site view, click on 'Customize' button

screen shot 2015-08-27 at 8 33 32 pm

2/ Change the site theme & the site title:

screen shot 2015-08-27 at 8 36 49 pm

3/ Click on 'Save & Activate'

screen shot 2015-08-27 at 8 37 50 pm

Actual result: See the front page of the site
Expected result: To go back to Calypso Themes with a confirmation screen about changing the theme:

screen shot 2015-08-27 at 8 33 23 pm

The success message would be consistent with the rest of Themes behavior of confirming the theme change.

Issue 2:

1/ Go to Calypso > Themes, click on 'Customize' for the current theme:

screen shot 2015-08-27 at 8 45 44 pm

2/ Change theme to something else:

screen shot 2015-08-27 at 8 46 32 pm

3/ Click on 'Save & Activate'

screen shot 2015-08-27 at 8 47 03 pm

Actual result: Go to the front page of the site
Expected result: Go back to Calypso Themes and see a success message about changing the theme:

screen shot 2015-08-27 at 8 33 23 pm

Issue 3:

1/ Go to Calypso > Themes, click on 'Customize' for the current theme:

screen shot 2015-08-27 at 8 45 44 pm

2/ Change the site theme & the site title:

screen shot 2015-08-27 at 8 36 49 pm

3/ Click on 'Save & Activate'

screen shot 2015-08-27 at 8 52 28 pm

Actual result: Go to the front page of the site
Expected result: Go back to Calypso Themes and see a success message about changing the theme:

screen shot 2015-08-27 at 8 33 23 pm

Solution

The best possible outcome here is to redirect back to the Showcase if the Customizer has been accessed via a Showcase link.

Why? Because once the theme has been changed from within the Customizer, we should go back to our welcome message. It's not perfect but at least it gives the user a choice: Back to themes or Visit site.

@lancewillett lancewillett added [Type] Bug When a feature is broken and / or not performing as intended [Feature Group] Appearance & Themes Features related to the appearance of sites. [Feature] Customizer The site customizer for traditional, non-block themes. labels Nov 18, 2015
@mattwiebe
Copy link
Contributor

These instances should all be fixed now that #1227 is closed. Those cases all involved breaking out of Calypso due to the way theme previewing in the Customizer works.

ockham pushed a commit that referenced this issue Sep 28, 2018
* Update the parse options to support what calypso uses

Add exportDefaultFrom, exportNamespaceFrom, and dynamicImport.

* Alphabetize plugins
ockham pushed a commit that referenced this issue Oct 11, 2018
* Update the parse options to support what calypso uses

Add exportDefaultFrom, exportNamespaceFrom, and dynamicImport.

* Alphabetize plugins
sirreal pushed a commit that referenced this issue Dec 6, 2018
* Update the parse options to support what calypso uses

Add exportDefaultFrom, exportNamespaceFrom, and dynamicImport.

* Alphabetize plugins
sirreal pushed a commit that referenced this issue Dec 7, 2018
* Update the parse options to support what calypso uses

Add exportDefaultFrom, exportNamespaceFrom, and dynamicImport.

* Alphabetize plugins
sirreal pushed a commit that referenced this issue Dec 7, 2018
* Update the parse options to support what calypso uses

Add exportDefaultFrom, exportNamespaceFrom, and dynamicImport.

* Alphabetize plugins
sirreal pushed a commit that referenced this issue Dec 11, 2018
* Update the parse options to support what calypso uses

Add exportDefaultFrom, exportNamespaceFrom, and dynamicImport.

* Alphabetize plugins
sirreal pushed a commit that referenced this issue Dec 11, 2018
* Update the parse options to support what calypso uses

Add exportDefaultFrom, exportNamespaceFrom, and dynamicImport.

* Alphabetize plugins
jsnajdr pushed a commit that referenced this issue Jan 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Customizer The site customizer for traditional, non-block themes. [Feature Group] Appearance & Themes Features related to the appearance of sites. [Type] Bug When a feature is broken and / or not performing as intended
Projects
None yet
Development

No branches or pull requests

2 participants