Rework Navigation Menu Part 2

* Separate scoping and assignment so that a decision does not need to be made as to whether we should intermix spaces and tabs on the left
 * Restore the previously removed nav-menu class addition to the first unordered list in `#site-navigation`.
 * Restore the `.navigation-main.toggled-on .nav-menu` selector.
 * Remove rules for the `.main-small-navigation ul` selector.
This commit is contained in:
Michael Fields 2013-06-25 01:45:52 -07:00
parent 4c396f6f9b
commit ac3cda4543
2 changed files with 6 additions and 9 deletions

View File

@ -4,10 +4,9 @@
* Handles toggling the navigation menu for small screens. * Handles toggling the navigation menu for small screens.
*/ */
( function() { ( function() {
var container = document.getElementById( 'site-navigation' ), var container, button, menu;
button,
menu;
container = document.getElementById( 'site-navigation' )
if ( ! container ) if ( ! container )
return; return;
@ -23,6 +22,9 @@
return; return;
} }
if ( -1 == menu.className.indexOf( 'nav-menu' ) )
menu.className += ' nav-menu';
button.onclick = function() { button.onclick = function() {
if ( -1 != container.className.indexOf( 'toggled' ) ) if ( -1 != container.className.indexOf( 'toggled' ) )
container.className = container.className.replace( ' toggled', '' ); container.className = container.className.replace( ' toggled', '' );

View File

@ -441,14 +441,9 @@ a:active {
cursor: pointer; cursor: pointer;
} }
.main-small-navigation ul {
display: none;
}
@media screen and (max-width: 600px) { @media screen and (max-width: 600px) {
.menu-toggle, .menu-toggle,
.navigation-main.toggled ul.menu, .navigation-main.toggled-on .nav-menu {
.navigation-main.toggled div.menu > ul {
display: block; display: block;
} }