Changeset 5875

Timestamp:

09/04/2017 03:03:14 PM (8 years ago)

Author:

SergeyBiryukov

Message:

Breathe: Properly set the .active class and aria-expanded attribute for the current menu item in handbook Chapters menu.

Fixes #3048.

Location:

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-breathe

Files:

• functions.php (modified) (1 diff)
• js/chapters.js (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-breathe/functions.php

@@ -47 +47 @@
 
     // Cacheing hack
-    wp_enqueue_style( 'wporg-breathe', get_stylesheet_uri(), array( 'p2-breathe' ), '20170904' );
+    wp_enqueue_style( 'wporg-breathe', get_stylesheet_uri(), array( 'p2-breathe' ), '20170904a' );
 }
 add_action( 'wp_enqueue_scripts', __NAMESPACE__ . '\styles', 11 );

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-breathe/js/chapters.js

@@ -11 +11 @@
 
     // Open the current menu
-    jQuery( '.current-menu-item a' ).first()
+    jQuery( '.menu-table-of-contents-container .current-menu-item a' ).first()
         .addClass( 'active' )
         .parents( '.menu-item-has-children' )
-            .toggleClass( 'open' );
+            .toggleClass( 'open' )
+        .find( '> div > .dashicons' )
+            .attr( 'aria-expanded', true );
 
     // Or if wrapped in a div.expandable
@@ -23 +25 @@
 
         menuToggle.toggleClass( 'open' );
-        menuToggle.attr( 'aria-expanded', menuToggle.hasClass( 'open' ) );
+        jQuery( this ).attr( 'aria-expanded', menuToggle.hasClass( 'open' ) );
     } );
 } );