[wp-trac] [WordPress Trac] #35273: Menus Screen Options: various fixes for the Menus Screen and the Customizer Menus Panel

WordPress Trac noreply at wordpress.org
Sat Mar 5 00:26:39 UTC 2016


#35273: Menus Screen Options: various fixes for the Menus Screen and the Customizer
Menus Panel
-----------------------------------------+-----------------------------
 Reporter:  afercia                      |       Owner:  westonruter
     Type:  defect (bug)                 |      Status:  accepted
 Priority:  normal                       |   Milestone:  4.5
Component:  Menus                        |     Version:  4.4
 Severity:  normal                       |  Resolution:
 Keywords:  has-screenshots needs-patch  |     Focuses:  ui, javascript
-----------------------------------------+-----------------------------
Changes (by westonruter):

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


Comment:

 Replying to [comment:13 afercia]:
 > - should something similar be done also for the Screen Options in `nav-
 menus.php` (or for all the Screen Options auto-saving?)

 Yes. I'll have to let someone else come up with the patch for that.

 > - please consider I really don't know anything about best practices to
 follow in WP for file inclusions, would including a whole file for just a
 single function be OK?

 I would say yes. Better to eliminate redundancy than to load a bit more
 code.

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


More information about the wp-trac mailing list