Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#37390 closed defect (bug) (fixed)

Twenty Thirteen: masonry widgets layout is not updated in the customizer

Reported by: celloexpressions's profile celloexpressions Owned by: westonruter's profile westonruter
Milestone: 4.6 Priority: normal
Severity: normal Version: 4.5
Component: Bundled Theme Keywords: has-patch commit has-screenshots
Focuses: javascript Cc:

Description

wp.customize.selectiveRefresh is not defined where the theme's functions.js checks for it, preventing the masonry footer widgets layout from updating properly when widgets are added, deleted, or reordered. This broke sometime between when the change was made to the theme and 4.5.3, and is likely due to the sequence of scripts loading.

See https://wordpress.slack.com/archives/core-customize/p1468779925000003, which has a fix from @westonruter. We also need to update the code in https://make.wordpress.org/core/2016/03/22/implementing-selective-refresh-support-for-widgets/ as themes may still be copying that.

Attachments (3)

37390.0.diff (842 bytes) - added by westonruter 9 years ago.
37390.1.diff (1.7 KB) - added by celloexpressions 9 years ago.
Bump script version
37390.after.gif (2.0 MB) - added by celloexpressions 9 years ago.
Intended functionality resolved.

Change History (7)

@westonruter
9 years ago

#1 @westonruter
9 years ago

  • Keywords has-patch added; needs-patch removed
  • Owner set to westonruter
  • Status changed from new to accepted

@celloexpressions
9 years ago

Bump script version

#2 @celloexpressions
9 years ago

  • Keywords commit added

37390.1.diff looks good to me, with the addition of a version bump on the script to ensure that it isn't cached.

#3 @westonruter
9 years ago

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

In 38083:

Twenty Thirteen: Fix selective refresh of Masonry-laid out widgets by deferring initialization until DOM ready.

This ensures that the wp.customize.selectiveRefresh object will be available when in the Customizer preview.

Fixes #37390.
Props westonruter, celloexpressions.

@celloexpressions
9 years ago

Intended functionality resolved.

#4 @celloexpressions
9 years ago

  • Keywords has-screenshots added
Note: See TracTickets for help on using tickets.