[wp-trac] [WordPress Trac] #46830: When menu item removed form Customizer menu uncheck it's source item

WordPress Trac noreply at wordpress.org
Fri Sep 6 08:12:42 UTC 2019


#46830: When menu item removed form Customizer menu uncheck it's source item
---------------------------------------+-----------------------
 Reporter:  garrett-eclipse            |       Owner:  audrasjb
     Type:  defect (bug)               |      Status:  accepted
 Priority:  normal                     |   Milestone:  5.3
Component:  Menus                      |     Version:
 Severity:  normal                     |  Resolution:
 Keywords:  has-patch has-screenshots  |     Focuses:  ui
---------------------------------------+-----------------------
Changes (by garrett-eclipse):

 * keywords:  has-patch => has-patch has-screenshots


Comment:

 Thanks @donmhico this worked really nicely, testing both Posts and Pages
 the check indicator reverts to the + add icon once all items are removed
 as desired.

 Reviewing the code everything looked very well written thank you. In php
 unit tests I've seen ticket references as `@ticket XXXXX` but I'm not sure
 if that's also the standard with JS (referring to your `@see` reference).

 @audrasjb would you mind taking a final review this works nicely and
 applies cleanly.

 P.S. While testing I found the check icon is lost if you click 'Add Items'
 to open/close the drawer. I've opened
 [https://core.trac.wordpress.org/ticket/47990 47990] to look into keep
 that persistent.

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


More information about the wp-trac mailing list