WordPress.org

Make WordPress Core

Opened 3 months ago

Closed 3 months ago

#53487 closed defect (bug) (fixed)

Empty move to options in Widgets Customizer

Reported by: kevin940726 Owned by: hellofromTonya
Milestone: 5.8 Priority: normal
Severity: normal Version:
Component: Widgets Keywords: has-patch has-unit-tests commit
Focuses: Cc:

Description

When trying to move a block to another widget area in the new block based Widgets Customizer, the dropdown shows the correct number of options and clicking on them works as expected, but the label are all empty. (See the image below)

I believe it's a core merge issue, as Gutenberg is registering both label and description field,
https://github.com/WordPress/gutenberg/blob/57ae6eea5d18e57ffa72607dc21eacc90e1c7035/lib/widgets-customize.php#L47-L59

but Core is not registering those fields.
https://github.com/WordPress/wordpress-develop/blob/bdf7dd6299e4453f9d7438de0a0e138d950ec0bf/src/wp-includes/class-wp-customize-widgets.php#L479-L486

Testing instructions

  1. Enable a theme with multiple widget areas (TwentyTwenty for instance)
  2. Go to Appearance -> Customize -> Widgets
  3. Try to move a block from one to another

Attachments (2)

image (2).png (8.0 KB) - added by kevin940726 3 months ago.
Move to dropdown showing empty labels in Widgets Customizer
53487-testing.gif (3.9 MB) - added by hellofromTonya 3 months ago.
Testing before and after applying PR 1428 => Before shows problem; After resolves it.

Change History (10)

@kevin940726
3 months ago

Move to dropdown showing empty labels in Widgets Customizer

#1 @noisysocks
3 months ago

  • Milestone changed from Awaiting Review to 5.8

#2 @noisysocks
3 months ago

  • Keywords needs-patch added

#3 @hellofromTonya
3 months ago

  • Owner set to hellofromTonya
  • Status changed from new to accepted

I can reproduce the problem. Working on a patch and tests.

This ticket was mentioned in PR #1428 on WordPress/wordpress-develop by hellofromtonya.


3 months ago

  • Keywords has-patch has-unit-tests added; needs-patch removed

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

  • Added tests for both classic widgets and widgets block editor. Tests failed as expected ✅
  • Added the label and description
  • Reran tests. Tests pass ✅

Manually testing => works as expected. See notes in ticket.

@hellofromTonya
3 months ago

Testing before and after applying PR 1428 => Before shows problem; After resolves it.

#5 @hellofromTonya
3 months ago

  • Keywords commit added

Testing

Env:

  • OS: macOS Big Sur
  • Browser: Chrome, Safari, and Firefox
  • Plugins: none
  • Theme: Twenty Twenty
  • WordPress: current trunk

Before Patch

Able to reproduce the problem, i.e. the Move To dropdown is not populated. ✅

After Patch

The Move To dropdown is populated with the registered sidebars. Selecting correctly moves the widget to the selected sidebar. ✅

Works as expected ✅ .

Marking for review and commit.

Last edited 3 months ago by hellofromTonya (previous) (diff)

This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.


3 months ago

This ticket was mentioned in Slack in #core by desrosj. View the logs.


3 months ago

#8 @desrosj
3 months ago

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

In 51228:

Widgets: Add missing label and description to Customizer controls.

This fixes a bug where the “Move To” dialogue was empty when a theme has multiple widget areas.

Props kevin940726, noisysocks, hellofromTonya.
Fixes #53487.

Note: See TracTickets for help on using tickets.