woocommerce/storefront
2
0
Fork 0
mirror of https://github.com/woocommerce/storefront.git synced 2025-08-20 04:00:29 +08:00
Code Issues Packages Wiki Activity Actions

Fix mobile nav menu in AMP Standard by using nav_menu_toggle (#1992)

Browse source
This commit is contained in:
Daniel Bachhuber 2022-06-28 04:03:17 -07:00 • committed by GitHub
parent 143f4a570e
commit 32bb2c33ae
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

12
inc/class-storefront.php
View file

@ -235,6 +235,18 @@ if ( ! class_exists( 'Storefront' ) ) :
* Add support for responsive embedded content.
*/
add_theme_support( 'responsive-embeds' );
add_theme_support(
'amp',
array(
'nav_menu_toggle' => array(
'nav_container_id' => 'site-navigation',
'nav_container_toggle_class' => 'toggled',
'menu_button_id' => 'site-navigation-menu-toggle',
'menu_button_toggle_class' => 'toggled',
),
)
);
}
/**

2
inc/storefront-template-functions.php
View file

@ -239,7 +239,7 @@ if ( ! function_exists( 'storefront_primary_navigation' ) ) {
function storefront_primary_navigation() {
?>
<nav id="site-navigation" class="main-navigation" role="navigation" aria-label="<?php esc_attr_e( 'Primary Navigation', 'storefront' ); ?>">
<button class="menu-toggle" aria-controls="site-navigation" aria-expanded="false"><span><?php echo esc_html( apply_filters( 'storefront_menu_toggle_text', __( 'Menu', 'storefront' ) ) ); ?></span></button>
<button id="site-navigation-menu-toggle" class="menu-toggle" aria-controls="site-navigation" aria-expanded="false"><span><?php echo esc_html( apply_filters( 'storefront_menu_toggle_text', __( 'Menu', 'storefront' ) ) ); ?></span></button>
<?php
wp_nav_menu(
array(