Make WordPress Core

Opened 8 months ago

Closed 8 months ago

Last modified 8 months ago

#60343 closed defect (bug) (fixed)

Fix: Theme.json application of custom root selector for styles

Reported by: aaronrobertshaw's profile aaronrobertshaw Owned by: youknowriad's profile youknowriad
Milestone: 6.5 Priority: normal
Severity: normal Version: 6.5
Component: Editor Keywords: gutenberg-merge has-patch has-unit-tests
Focuses: Cc:

Description

This ticket tracks the backporting of PHP files for the following Gutenberg changes:

https://github.com/WordPress/gutenberg/pull/58050

These changes fix the incorrect application of a custom root selector in theme.json.

Change History (5)

This ticket was mentioned in PR #5944 on WordPress/wordpress-develop by @aaronrobertshaw.


8 months ago
#1

  • Keywords has-patch has-unit-tests added

Syncs the changes from https://github.com/WordPress/gutenberg/pull/58050 to core.

These changes fix the incorrect application of a custom root selector in theme.json.

To test:

npm run test:php -- --filter Tests_Theme_wpThemeJson

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

#2 @get_dave
8 months ago

I tested this PR and the unit test passed successfully

OK (139 tests, 403 assertions)

Also verified file changes mirror those originally made in the PR being synced from Gutenberg to Core.

@youknowriad I think this can be committed.

#3 @youknowriad
8 months ago

  • Milestone changed from Awaiting Review to 6.5

#4 @youknowriad
8 months ago

  • Owner set to youknowriad
  • Resolution set to fixed
  • Status changed from new to closed

In 57352:

Editor: Fix Theme.json application of custom root selector for styles.

Theme.json stylesheets attempting to use a custom root selector are generated with in correct styles.

Props aaronrobertshaw, get_dave, mukesh27.
Fixes #60343.

Note: See TracTickets for help on using tickets.