Skip to content

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

Closed
oandregal wants to merge 5 commits into
WordPress:masterfrom
oandregal:update/only-load-classic-when-no-theme-json
Closed

Only load wp-editor-classic-layout-styles styles for themes without theme.json support#1290
oandregal wants to merge 5 commits into
WordPress:masterfrom
oandregal:update/only-load-classic-when-no-theme-json

Conversation

@oandregal

@oandregal oandregal commented May 24, 2021

Copy link
Copy Markdown
Member

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.

Comment thread src/wp-includes/script-loader.php
@youknowriad

Copy link
Copy Markdown
Contributor

This is looking good.

@youknowriad

Copy link
Copy Markdown
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