Make WordPress Core

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#53378 closed defect (bug) (fixed)

Two regressions in the UI color control

Reported by: oandregal's profile oandregal Owned by:
Milestone: 5.8 Priority: normal
Severity: normal Version: 5.8
Component: Editor Keywords: has-patch
Focuses: Cc:

Description

There's a couple of regressions related to the UI control control in the block editor:

  • themes can no longer pass an empty array to disable the feature
  • the legacy settings.color setting in the core/block-editor client store didn't have the proper value

Related https://core.trac.wordpress.org/ticket/53175

Change History (7)

This ticket was mentioned in PR #1363 on WordPress/wordpress-develop by nosolosw.


3 years ago
#1

Fixes https://core.trac.wordpress.org/ticket/53378

We found a couple of regressions with colors:

  • themes could no longer pass an empty array to disable the feature
  • the legacy settings.color setting in the core/block-editor store didn't have the proper value

These have been fixed in the plugin with this PR https://github.com/WordPress/gutenberg/pull/32358

props @jorgefilipecosta

#2 @desrosj
3 years ago

  • Milestone changed from Awaiting Review to 5.8

Moving to 5.8 for visibility.

nosolosw commented on PR #1363:


3 years ago
#3

Noting this one may need updating with the changes at https://github.com/WordPress/gutenberg/pull/32622

#5 @oandregal
3 years ago

This is going to be closed when https://core.trac.wordpress.org/ticket/53397 lands, as the changes necessary to fix it are part of that backport.

#6 @desrosj
3 years ago

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

#53397 is now fixed. @nosolosw if you could, please just double check trunk when you have a chance to confirm.

#7 @oandregal
3 years ago

Just tested it and can confirm this is fixed.

Note: See TracTickets for help on using tickets.