From bffa48a6ee28b091d2beaa5cdfdf4b439ab9c9b8 Mon Sep 17 00:00:00 2001 From: Tammie Lister Date: Mon, 14 Nov 2016 17:00:57 +0000 Subject: [PATCH] Twenty Seventeen: Resolves focused controls hidden by the top menu. When a page is scrolled, the top nav menu become fixed. Depending on the page content this caused issue where focused links or buttons may be hidden behind the nav menu. Props afercia, davidakennedy, Fencer04 Fixes #38476 git-svn-id: https://develop.svn.wordpress.org/trunk@39225 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-content/themes/twentyseventeen/assets/js/global.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/wp-content/themes/twentyseventeen/assets/js/global.js b/src/wp-content/themes/twentyseventeen/assets/js/global.js index 3d893e17cd..fa29a19b79 100644 --- a/src/wp-content/themes/twentyseventeen/assets/js/global.js +++ b/src/wp-content/themes/twentyseventeen/assets/js/global.js @@ -27,7 +27,7 @@ resizeTimer; // Ensure the sticky navigation doesn't cover current focused links. - $( '#content a, #colophon a' ).focus( function() { + $( 'a[href], area[href], input:not([disabled]), select:not([disabled]), textarea:not([disabled]), button:not([disabled]), iframe, object, embed, [tabindex], [contenteditable]', '.site-content-contain' ).filter( ':visible' ).focus( function() { if ( $navigation.hasClass( 'site-navigation-fixed' ) ) { var windowScrollTop = $( window ).scrollTop(), fixedNavHeight = $navigation.height(), @@ -40,7 +40,7 @@ } if ( offsetDiff < fixedNavHeight ) { - $( window ).scrollTo( itemScrollTop - ( fixedNavHeight + 50 ), 600 ); + $( window ).scrollTo( itemScrollTop - ( fixedNavHeight + 50 ), 0 ); } } });