Twenty Nineteen: Make sure links are followed on touchend, not touchstart.

The theme's original navigation JavaScript was making it so all links on a site were immediately followed on touchstart when using a touch-enabled device. This update makes sure links are followed at touchend, to improve usability and menu behavior.

Props anevins, panchen, kjellr.
Fixes #45510.



git-svn-id: https://develop.svn.wordpress.org/branches/5.0@44357 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
laurelfulford 2018-12-21 21:08:56 +00:00
parent eb965c27c0
commit 0acb4ebc35

View File

@ -219,19 +219,9 @@
var url = event.target.getAttribute( 'href' ) ? event.target.getAttribute( 'href' ) : '';
// If theres a link, go to it on touchend
if ( '#' !== url && '' !== url ) {
window.location = url;
// Open submenu if url is #
} else if ( '#' === url && event.target.nextSibling.matches('.submenu-expand') ) {
if ( '#' === url && event.target.nextSibling.matches('.submenu-expand') ) {
openSubMenu( event.target );
// Prevent default touch events
} else {
event.preventDefault();
}
}