[wp-trac] [WordPress Trac] #34923: Allow users to more seamlessly create page-based nav menus during customization

WordPress Trac noreply at wordpress.org
Wed Aug 24 21:55:48 UTC 2016


#34923: Allow users to more seamlessly create page-based nav menus during
customization
-------------------------------------+-------------------------------------
 Reporter:  westonruter              |       Owner:  westonruter
     Type:  enhancement              |      Status:  accepted
 Priority:  high                     |   Milestone:  4.7
Component:  Customize                |     Version:  4.3
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch ux-feedback    |     Focuses:  ui, accessibility,
  needs-testing has-screenshots      |  javascript
  needs-unit-tests                   |
-------------------------------------+-------------------------------------
Changes (by westonruter):

 * keywords:  has-patch ux-feedback needs-testing has-screenshots => has-
     patch ux-feedback needs-testing has-screenshots needs-unit-tests


Comment:

 In [attachment:34923.6.diff]:

 * [https://github.com/xwp/wordpress-develop/pull/152/commits/6a51eca
 6a51eca] Fix JSHint issues
 * [https://github.com/xwp/wordpress-develop/pull/152/commits/120ac63
 120ac63] Fix JSCS issues
 * [https://github.com/xwp/wordpress-develop/pull/152/commits/93eeb23
 93eeb23] Fix PHPCS issues
 * [https://github.com/xwp/wordpress-develop/pull/152/commits/eeef76d
 eeef76d] Add sanitization for nav_menus_created_posts setting
 * [https://github.com/xwp/wordpress-develop/pull/152/commits/1eaa45b
 1eaa45b] Ensure that nav_menus_created_posts is set with clone since
 arrays passed by reference
 * [https://github.com/xwp/wordpress-develop/pull/152/commits/c81e576
 c81e576] Eliminate edit post links in notifications per discussion
 * [https://github.com/xwp/wordpress-develop/pull/152/commits/ee187ad
 ee187ad]  Move auto-draft post insertion into nav_menus component
 * [https://github.com/xwp/wordpress-develop/pull/152/commits/5bb985e
 5bb985e] Use flexbox to lay out new-content-item

 Todo:

 * Insert latest version of `ajax_insert_auto_draft_post` from Customize
 Posts: https://github.com/xwp/wp-customize-
 posts/blob/6c52c8a8dc3dca21dc6984b7a5ae274b8e079c89/php/class-wp-
 customize-posts.php#L1064-L1131
 * Integrate latest version of `insert_auto_draft_post` from Customize
 Posts: https://github.com/xwp/wp-customize-
 posts/blob/6c52c8a8dc3dca21dc6984b7a5ae274b8e079c89/php/class-wp-
 customize-posts.php#L930-L968
 * Add unit tests (tests can likely be copied from Customize Posts, which
 has coverage for these functions).

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


More information about the wp-trac mailing list