From: Dan Allen <dan@opendevise.com>
Date: Sun, 20 Dec 2020 23:33:00 +0000 (-0700)
Subject: minor refactoring
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=805c1f61728186c226caf4bee43458fabf53c111;p=working%2Feg-antora.git

minor refactoring
---

diff --git a/src/js/01-nav.js b/src/js/01-nav.js
index ab68291..0b4c5bf 100644
--- a/src/js/01-nav.js
+++ b/src/js/01-nav.js
@@ -105,24 +105,24 @@
 
   function showNav (e) {
     if (navToggle.classList.contains('is-active')) return hideNav(e)
-    var navBounds = nav.getBoundingClientRect()
-    var expectedNavHeight = window.innerHeight - Math.round(navBounds.top)
-    if (Math.round(navBounds.height) !== expectedNavHeight) nav.style.height = expectedNavHeight + 'px'
+    trapEvent(e)
     var html = document.documentElement
     html.classList.add('is-clipped--nav')
     navToggle.classList.add('is-active')
     navContainer.classList.add('is-active')
+    var bounds = nav.getBoundingClientRect()
+    var expectedHeight = window.innerHeight - Math.round(bounds.top)
+    if (Math.round(bounds.height) !== expectedHeight) nav.style.height = expectedHeight + 'px'
     html.addEventListener('click', hideNav)
-    trapEvent(e)
   }
 
   function hideNav (e) {
+    trapEvent(e)
     var html = document.documentElement
     html.classList.remove('is-clipped--nav')
     navToggle.classList.remove('is-active')
     navContainer.classList.remove('is-active')
     html.removeEventListener('click', hideNav)
-    trapEvent(e)
   }
 
   function trapEvent (e) {