Bundled Themes: Twenty Twenty anchor links don't work in mobile menu.

Modifies the mobile modal menu javascript, so that anchor links will close the modal and scroll to the anchor within the page.

This brings the changes from [47784] to the 5.4 branch.

Props Giorgio25b, suzylah, yuhin, samful, bdcstr.
Fixes #48916.

Built from https://develop.svn.wordpress.org/branches/5.4@47829


git-svn-id: http://core.svn.wordpress.org/branches/5.4@47605 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
whyisjake 2020-05-19 22:50:07 +00:00
parent c9bf01c0a4
commit 8e5980a770
2 changed files with 12 additions and 1 deletions

View File

@ -137,6 +137,17 @@ twentytwenty.coverModals = {
var target = event.target;
var modal = document.querySelector( '.cover-modal.active' );
// if target onclick is <a> with # within the href attribute
if ( event.target.tagName.toLowerCase() === 'a' && event.target.hash.includes( '#' ) && modal !== null ) {
// untoggle the modal
this.untoggleModal( modal );
// wait 550 and scroll to the anchor
setTimeout( function() {
var anchor = document.getElementById( event.target.hash.slice( 1 ) );
anchor.scrollIntoView();
}, 550 );
}
if ( target === modal ) {
this.untoggleModal( target );
}

View File

@ -13,7 +13,7 @@
*
* @global string $wp_version
*/
$wp_version = '5.4.2-alpha-47828';
$wp_version = '5.4.2-alpha-47829';
/**
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.