WordPress.org
Get WordPress

Make WordPress Core

Opened 2 years ago

Closed 23 months ago

Last modified 23 months ago

#57550 closed defect (bug) (fixed)

Block editor: separate content styles for the iframe

Reported by: ellatrix's profile ellatrix Owned by: hellofromtonya's profile hellofromTonya
Milestone: 6.2 Priority: normal
Severity: normal Version:
Component: Editor Keywords: has-patch gutenberg-merge commit
Focuses: Cc:

Description (last modified by hellofromTonya)

This ticket is the PHP changes for the initiative to:

  • Separate the content styles contained in the block editor package into a separate stylesheet.
  • Avoid loading all block editor styles into the iframe.

Changes:

  • Remove the 'wp-block-editor' and 'wp-block-library' from the iframe's stylesheet collection (i.e. _wp_get_iframed_editor_assets()).
  • Add the new 'wp-block-editor-content' for the separate stylesheet to the list of editor ('wp-edit-blocks') dependencies.

This ticket is for the PHP changes in ​https://github.com/WordPress/gutenberg/pull/44298. The changes within the editor and block library will be included in a @wordpress packages update (see #57471).

Change History (7)

This ticket was mentioned in ​PR #3903 on ​WordPress/wordpress-develop by ​@ellatrix.
2 years ago #1

  • Keywords has-patch added

Trac ticket: https://core.trac.wordpress.org/ticket/57550

#2 @hellofromTonya
2 years ago

  • Description modified (diff)
  • Keywords gutenberg-merge added
  • Milestone changed from Awaiting Review to 6.2
  • Owner set to hellofromTonya
  • Status changed from new to reviewing
  • Summary changed from Block editor: separate content styles for the iframe (PHP backport) to Block editor: separate content styles for the iframe
  • Version trunk deleted

​@hellofromTonya commented on ​PR #3903:
23 months ago #3

Currently reviewing for commit.

#4 @hellofromTonya
23 months ago

  • Keywords commit added

​PR 3903 is ready for commit.

​@hellofromTonya commented on ​PR #3903:
23 months ago #5

Good. The CI jobs are all green. The failures were unrelated to this PR.

#6 @hellofromTonya
23 months ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 55179:

Editor: Separate content styles for iframe in wp_get_iframed_editor_assets() and wp_default_styles().

This changeset:

  • Removes the 'wp-block-editor' and 'wp-block-library' from the iframe's stylesheet collection (i.e. _wp_get_iframed_editor_assets()).
  • Adds the new 'wp-block-editor-content' for the separate stylesheet to the list of editor ('wp-edit-blocks') dependencies.

Why?

These PHP changes are part of the initiative to:

  • Separate the content styles contained in the block editor package into a separate stylesheet.
  • Avoid loading all block editor styles into the iframe.

References:

Follow-up to [53160], [50761].

Props ellatrix, youknowriad.
Fixes #57550.

Note: See TracTickets for help on using tickets.