From c388ff84ee27b4fc56125db2f34e69d656490b1a Mon Sep 17 00:00:00 2001 From: Tammie Lister Date: Tue, 25 Oct 2016 16:50:49 +0000 Subject: [PATCH] Twenty Seventeen: Display scroll down arrow when no menu This improves scrollability without a menu. Props nnaimov, melchoyce, davidakennedy Fixes #38392 git-svn-id: https://develop.svn.wordpress.org/trunk@38909 602fd350-edb4-49c9-b593-d223f7449a82 --- .../themes/twentyseventeen/assets/js/global.js | 4 ++-- .../themes/twentyseventeen/style.css | 18 +++++++++++++----- .../template-parts/header/site-branding.php | 4 ++++ 3 files changed, 19 insertions(+), 7 deletions(-) diff --git a/src/wp-content/themes/twentyseventeen/assets/js/global.js b/src/wp-content/themes/twentyseventeen/assets/js/global.js index 14863b09ee..1f707b3384 100644 --- a/src/wp-content/themes/twentyseventeen/assets/js/global.js +++ b/src/wp-content/themes/twentyseventeen/assets/js/global.js @@ -10,7 +10,7 @@ $navWrap = $navigation.find( '.wrap' ), $navMenuItem = $navigation.find( '.menu-item' ), $menuToggle = $navigation.find( '.menu-toggle' ), - $menuScrollDown = $navigation.find( '.menu-scroll-down' ), + $menuScrollDown = $body.find( '.menu-scroll-down' ), $sidebar = $body.find( '#secondary' ), $entryContent = $body.find( '.entry-content' ), $formatQuote = $body.find( '.format-quote blockquote' ), @@ -143,7 +143,7 @@ $( document ).ready( function() { // Let's fire some JavaScript! - if ( 'true' === twentyseventeenScreenReaderText.has_navigation ) { + if ( $menuScrollDown.length ) { /** * 'Scroll Down' arrow in menu area diff --git a/src/wp-content/themes/twentyseventeen/style.css b/src/wp-content/themes/twentyseventeen/style.css index 53dab0ec9d..16d69baea7 100644 --- a/src/wp-content/themes/twentyseventeen/style.css +++ b/src/wp-content/themes/twentyseventeen/style.css @@ -1464,7 +1464,7 @@ a:hover .nav-title, /* Scroll down arrow */ -.navigation-top .menu-scroll-down { +.site-header .menu-scroll-down { display: none; } @@ -3514,24 +3514,32 @@ article.panel-placeholder { display: none; } - .site-navigation-fixed.navigation-top .menu-scroll-down { + .site-navigation-fixed.site-header.menu-scroll-down { display: none; } /* Scroll down arrow */ - .navigation-top .menu-scroll-down { - color: #767676; + .site-header .menu-scroll-down { display: block; padding: 0.5em 0.5em 0.4em; position: absolute; right: 0; - top: 0.9em; -webkit-transform: rotate(90deg); /* Chrome, Safari, Opera */ -ms-transform: rotate(90deg); /* IE 9 */ transform: rotate(90deg); } + .site-header .menu-scroll-down { + color: #fff; + top: 2em; + } + + .site-header .navigation-top .menu-scroll-down { + color: #767676; + top: 0.9em; + } + .menu-scroll-down:focus { outline: thin dotted; } diff --git a/src/wp-content/themes/twentyseventeen/template-parts/header/site-branding.php b/src/wp-content/themes/twentyseventeen/template-parts/header/site-branding.php index ffa668d15e..5ba9117375 100644 --- a/src/wp-content/themes/twentyseventeen/template-parts/header/site-branding.php +++ b/src/wp-content/themes/twentyseventeen/template-parts/header/site-branding.php @@ -27,5 +27,9 @@ + + 'next' ) ); ?> + +