[wp-trac] [WordPress Trac] #32841: Partial refresh <div> in menu customizer breaks some theme designs

WordPress Trac noreply at wordpress.org
Wed Jul 8 21:29:54 UTC 2015


#32841: Partial refresh <div> in menu customizer breaks some theme designs
------------------------------+--------------------------
 Reporter:  greenshady        |       Owner:  westonruter
     Type:  defect (bug)      |      Status:  closed
 Priority:  normal            |   Milestone:  4.3
Component:  Customize         |     Version:  trunk
 Severity:  normal            |  Resolution:  fixed
 Keywords:  has-patch commit  |     Focuses:  javascript
------------------------------+--------------------------
Changes (by westonruter):

 * status:  accepted => closed
 * resolution:   => fixed


Comment:

 In [changeset:"33138"]:
 {{{
 #!CommitTicketReference repository="" revision="33138"
 Customizer: Remove additional wrapper element around `wp_nav_menu()` which
 broke some theme designs.

 Also includes these related changes:
 * Export `oldContainer` and `newContainer` among the `customize-preview-
 menu-refreshed` event params for themes to be able to more easily re-
 initialize the DOM elements.
 * Improve performance for partial refresh by only sending settings related
 to the menu being previewed.
 * Fix previewing of menu assigned to Custom Menu by exporting a menu
 `term_id` as opposed to an object, as the former is more stable for
 comparing in in args hashes.
 * Do full refresh of preview when nav menu unassigned so that the layout
 can be updated.
 * Harden conditions for when partial refresh is eligible for a
 `wp_nav_menu()` instance.

 Fixes #32841.
 }}}

--
Ticket URL: <https://core.trac.wordpress.org/ticket/32841#comment:21>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list