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

Only load wp-editor-classic-layout-styles styles for themes without theme.json support #1290

Closed
wants to merge 5 commits into from
Closed

Only load wp-editor-classic-layout-styles styles for themes without theme.json support #1290

wants to merge 5 commits into from

Conversation

oandregal
Copy link
Member

@oandregal oandregal commented May 24, 2021

Part of https://core.trac.wordpress.org/ticket/53175
See WordPress/gutenberg#31322
Related WordPress/gutenberg#30375

How to test

Activate a theme without theme.json support (TwentyTwenty):

  • Load the post editor.
  • Verify that the wp-editor-classic-layout-styles-css external stylesheet is loaded.

Activate a theme with theme.json support (TT1-blocks from the github repo not from the WordPress theme's repo, or add a theme.json file to the TwentyTwentyOne theme):

  • Load the post editor.
  • Verify that the wp-editor-classic-layout-styles-css external stylesheet is not loaded.

@youknowriad
Copy link
Contributor

This is looking good.

@youknowriad
Copy link
Contributor

committed

@oandregal oandregal deleted the update/only-load-classic-when-no-theme-json branch May 25, 2021 07:43
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.

2 participants