Skip to content

Commit

Permalink
turn the nav-toggle-label into a button
Browse files Browse the repository at this point in the history
  • Loading branch information
stephengold committed Jul 14, 2020
1 parent 8950d94 commit 5dd997f
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
3 changes: 2 additions & 1 deletion src/css/toolbar.css
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,8 @@
}

.nav-toggle-label {
padding: 0.5rem;
background-color: var(--toolbar-background);
border: none;
}

.nav-toggle {
Expand Down
2 changes: 2 additions & 0 deletions src/js/01-nav.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,10 @@

var navContainer = document.querySelector('.nav-container')
var navToggle = document.querySelector('.nav-toggle')
var navToggleLabel = document.querySelector('.nav-toggle-label')

navToggle.addEventListener('click', showNav)
navToggleLabel.addEventListener('click', showNav)
// NOTE don't let click events propagate outside of nav container
navContainer.addEventListener('click', concealEvent)

Expand Down
2 changes: 1 addition & 1 deletion src/partials/nav-toggle.hbs
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
<span class="nav-toggle-label">Project links</span>
<button class="nav-toggle-label">Project links</span>
<button class="nav-toggle"></button>

0 comments on commit 5dd997f

Please sign in to comment.