-
Notifications
You must be signed in to change notification settings - Fork 2k
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
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
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
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
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
2/ Change the site theme & the site title:
3/ Click on 'Save & Activate'
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:
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:
2/ Change theme to something else:
3/ Click on 'Save & Activate'
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:
Issue 3:
1/ Go to Calypso > Themes, click on 'Customize' for the current theme:
2/ Change the site theme & the site title:
3/ Click on 'Save & Activate'
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:
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.
The text was updated successfully, but these errors were encountered: