added an active class for the toggle button currently in use

Author: MelSumner

addon/components/es-navbar/link/component.js

@@ -31,8 +31,6 @@ export default Component.extend({
           // move focus to the first item in the dropdown
           this.processFirstElementFocus();
           this.processKeyPress();
-
-
         });
       }
     }
@@ -49,7 +47,7 @@ export default Component.extend({
     this.processFirstElementFocus();
   },
 
-  processBlur() { //TODO is this working?
+  processBlur() {
     next(this, function() {
       let subItems = Array.from(this.element.querySelectorAll('.navbar-dropdown-list li'));
       let focused = subItems.find(item => document.activeElement === item.querySelector('a'));
@@ -62,7 +60,7 @@ export default Component.extend({
   },
 
   processClick() {
-
+    // TODO handle mouseclick outside the current dropdown
   },
 
   processFirstElementFocus() {
@@ -103,6 +101,6 @@ export default Component.extend({
 
   willDestroyElement() {
     document.removeEventListener('keydown', this.triggerDropdown);
-    document.removeEventListener('click', this.triggerDropdown);
+    // document.removeEventListener('click', this.triggerDropdown);
   }
 });

addon/components/es-navbar/link/template.hbs

@@ -9,7 +9,7 @@
 {{#if (eq link.type "dropdown")}}
   <button
     onclick={{action "toggleDropdown"}}
-    class="navbar-list-item-dropdown-toggle"
+    class="navbar-list-item-dropdown-toggle {{if isDropdownOpen "active"}}"
     aria-expanded={{isExpanded}}
   >
     {{link.name}}