[wp-trac] [WordPress Trac] #38992: Twenty Seventeen: Menu toggle doesn't work after adding a new menu item in customizer

WordPress Trac noreply at wordpress.org
Fri Dec 2 06:33:27 UTC 2016


#38992: Twenty Seventeen: Menu toggle doesn't work after adding a new menu item in
customizer
-------------------------------------------+----------------------------
 Reporter:  ocean90                        |       Owner:  davidakennedy
     Type:  defect (bug)                   |      Status:  closed
 Priority:  normal                         |   Milestone:  4.7
Component:  Bundled Theme                  |     Version:  trunk
 Severity:  normal                         |  Resolution:  fixed
 Keywords:  has-patch commit dev-reviewed  |     Focuses:  javascript
-------------------------------------------+----------------------------

Comment (by helen):

 In [changeset:"39423"]:
 {{{
 #!CommitTicketReference repository="" revision="39423"
 Twenty Seventeen: Fix broken menu toggle in Customizer after menu items
 are added

 This simplifies the line `$( siteNavigation.closest( '.main-navigation' ),
 this ).toggleClass( 'toggled-on' );` to `$( siteNavContain ).toggleClass(
 'toggled-on' );`, since `this` is the clicked button, so the extra context
 isn't needed.

 Props afercia, laurelfulford.

 Fixes #38992 for the 4.7 branch.
 }}}

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


More information about the wp-trac mailing list