N-archive/hacks-guide-minimal-mistakes-theme
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix typo in menu button class

Browse source 
- .navtoogle should be .navtoggle
- Fixes #175
This commit is contained in:
Michael Rose 2015-08-19 12:50:44 -04:00
parent e7489a139d
commit d5a0a2e585
3 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
_sass/page.scss
View file

@ -165,7 +165,7 @@ $button-size: 1.5rem;
} }
} }
// Style the toggle menu link and hide it // Style the toggle menu link and hide it
.nav .navtoogle { .nav .navtoggle {
@include font-rem(18); @include font-rem(18);
font-weight: normal; font-weight: normal;
background-color: $black; background-color: $black;
@ -180,7 +180,7 @@ $button-size: 1.5rem;
border: none; border: none;
background: none; background: none;
} }
.navtoogle i { .navtoggle i {
z-index:-1; z-index:-1;
} }
.icon-menu { .icon-menu {

6
assets/js/_main.js
View file

@ -13,11 +13,11 @@ var changeClass = function (r,className1,className2) {
}; };
// Creating our button in JS for smaller screens // Creating our button in JS for smaller screens
var menuElements = document.getElementById('site-nav'); var menuElements = document.getElementById('site-nav');
menuElements.insertAdjacentHTML('afterBegin','<button type="button" role="button" id="menutoggle" class="navtoogle navicon-lines-button x" aria-hidden="true"><span class="navicon-lines"></span>menu</button>'); menuElements.insertAdjacentHTML('afterBegin','<button type="button" role="button" id="menutoggle" class="navtoggle navicon-lines-button x" aria-hidden="true"><span class="navicon-lines"></span>menu</button>');
// Toggle the class on click to show / hide the menu // Toggle the class on click to show / hide the menu
document.getElementById('menutoggle').onclick = function() { document.getElementById('menutoggle').onclick = function() {
changeClass(this, 'navtoogle active', 'navtoogle'); changeClass(this, 'navtoggle active', 'navtoggle');
}; };
// http://tympanus.net/codrops/2013/05/08/responsive-retina-ready-menu/comment-page-2/#comment-438918 // http://tympanus.net/codrops/2013/05/08/responsive-retina-ready-menu/comment-page-2/#comment-438918
document.onclick = function(e) { document.onclick = function(e) {
@ -25,7 +25,7 @@ document.onclick = function(e) {
buttonStyle = mobileButton.currentStyle ? mobileButton.currentStyle.display : getComputedStyle(mobileButton, null).display; buttonStyle = mobileButton.currentStyle ? mobileButton.currentStyle.display : getComputedStyle(mobileButton, null).display;
if(buttonStyle === 'block' && e.target !== mobileButton && new RegExp(' ' + 'active' + ' ').test(' ' + mobileButton.className + ' ')) { if(buttonStyle === 'block' && e.target !== mobileButton && new RegExp(' ' + 'active' + ' ').test(' ' + mobileButton.className + ' ')) {
changeClass(mobileButton, 'navtoogle active', 'navtoogle'); changeClass(mobileButton, 'navtoggle active', 'navtoggle');
} }
}; };

2
assets/js/scripts.min.js vendored
View file

File diff suppressed because one or more lines are too long