Simple Responsive menu javascript

by ducrat

HTML

<div class='navmenu'>
    <button id="menu">Menu</button>
    <nav id="navbar">
    <ul class="navbar">
        <li><a href="index.html">Home</a></li>
        <li><a href="newsletter.html">abcd</a></li>
        <li><a href="courses.html">Courses</a></li>
        <li><a href="materials.html">Materials &amp; More</a></li>
         <li><a href="contactus.html">Contact Us</a></li>
 
    </ul>
</nav>

</div>

<h1>The Main Heading</h1>

<p>Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod
tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam,l
quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo
consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse
cillum dolore eu fugiat nulla pariatur.</p>

CSS

.navmenu{width:100%;box-shadow:0 1px 1px rgba(0,0,0,0.15);background:#303030}}
#menu {
    margin: 2px;
}
#menu a {
    text-decoration: none;
}
nav {
    display: none;
}
nav ul {
    list-style-type: none;
    padding-left: 0;
    font-size: 0;
    background-color: #303030;
}
nav li {
    display: block;
    font-size: 14px;
    color: white;
}
nav a {
    display: block;
    padding: 10px;
    text-decoration: none;
    color: inherit;
}
nav a:hover {Color:red
}

@media screen and (min-width: 800px) {
    #menu {
        display: none;
    }
    nav {
        display: block !important;
    }
    nav li {
        display: inline-block;
        border-right: 1px solid cornflowerblue;
        border-bottom: none;
    }
}

JavaScript

document.getElementById('menu').addEventListener('click', function () {
        var nav = document.getElementsByTagName('nav')[0];
        if (nav.style.display == 'block') {
            nav.style.display = 'none';
        } else {
            nav.style.display = 'block';
        }
    }, false);