[wp-trac] [WordPress Trac] #32814: Custom Menu Widget should show newly added menus in Customizer
WordPress Trac
noreply at wordpress.org
Thu Jul 2 19:39:50 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: needs-patch dev-feedback | Focuses: javascript
--------------------------------------+-------------------------
Comment (by adamsilverstein):
Still working on this item.
Looks like the best/most straightforward approach will be to save the new
unpublished menu items to the server via ajax so that rendering can happen
correctly for the custom menu widget (and any other customizer element
referencing menus).
Since we previously had code that was saving the menus via ajax, I should
be able to leverage this old code - working on that now. We will need to
fire events to add, update or delete menus, using the temporary menu id.
Once this is in place the existing widget rendering functionality should
work as expected. Once the menu is 'saved and published' we will need to
remove the temporary php side data and update any widgets to use the new
ID (partially in place already), while also maintaining the correctly
selected (potentially new) menus.
Replying to [comment:7 adamsilverstein]:
> Replying to [comment:6 samuelsidler]:
> > Replying to [comment:5 tywayne]:
> > > Though, maybe I'm dramatizing that assumption? Is there precedent
anywhere else in the customizer where we tell a user they have to save
before they can continue?
> >
> > No, there's not. I would hope there's some way we can work around
this.
>
> There is a way forward, it is just complicated. I will continue working
on this.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/32814#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list