[wp-trac] [WordPress Trac] #32814: Custom Menu Widget should show newly added menus in Customizer

WordPress Trac noreply at wordpress.org
Sun Jul 5 18:52:40 UTC 2015


#32814: Custom Menu Widget should show newly added menus in Customizer
------------------------------------+-------------------------
 Reporter:  adamsilverstein         |       Owner:
     Type:  defect (bug)            |      Status:  new
 Priority:  normal                  |   Milestone:  4.3
Component:  Customize               |     Version:  trunk
 Severity:  normal                  |  Resolution:
 Keywords:  dev-feedback has-patch  |     Focuses:  javascript
------------------------------------+-------------------------
Changes (by adamsilverstein):

 * keywords:  needs-patch dev-feedback => dev-feedback has-patch


Comment:

 Posting [attachment:32814.2.diff] (a work in progress) for feedback,
 review & testing:

 * Includes duplicate dom manipulation code removal from other related
 ticket
 * When adding a new (unsaved) menu to a custom menu widget, ensure that
 menu's data is sent as part of the widget-update ajax request
 * On the back end, when a new menu id is being used for a custom menu
 widget, use the passed 'customized' data to generate the widget html
 * Started working on code to refresh widgets when menu updated, not
 certain how to refresh control once I find a wigdet using the menu in
 question (I could find them, but not refresh); temp brute force workaround
 it to refresh all controls when menu changed - ''need some help fixing
 this''
 * Added code to refresh id when saved and published, so widgets using the
 saved menu get the new (real) id, needs testing
 * Slight update to wp_get_nav_menu_object was required to avoid a php
 warning where get_term was being called with menu object

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


More information about the wp-trac mailing list