From 052f23aece5310cb2762a69b9fd282bbfa04e0af Mon Sep 17 00:00:00 2001
From: Chris George <chris.george@red-gate.com>
Date: Tue, 22 Dec 2020 12:12:44 +0000
Subject: [PATCH] Fixed chevron not toggling, and used add/remove class rather
 than string operations.

---
 src/navigation/js/honeycomb.navigation.vertical.js | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/navigation/js/honeycomb.navigation.vertical.js b/src/navigation/js/honeycomb.navigation.vertical.js
index 2ff1a925..d27e6da6 100644
--- a/src/navigation/js/honeycomb.navigation.vertical.js
+++ b/src/navigation/js/honeycomb.navigation.vertical.js
@@ -42,9 +42,9 @@ const toggle = ( e, a ) => {
     e.preventDefault();
     const parent = a.parentElement;
     if (parent.className.match(activeClass) !== null) {
-        parent.className = parent.className.replace(parentActiveClass, '').replace(activeClass, '');
+        $(parent).removeClass(parentActiveClass + " " + activeClass);
     } else {
-        parent.className = parent.className + ` ${parentActiveClass}`;
+        $(parent).addClass(parentActiveClass + " " + activeClass);
     }
 };
 
@@ -54,18 +54,18 @@ const update = ( e, nav, a ) => {
     // Remove all active classes.
     let items = nav.querySelectorAll(`.${activeClass}`);
     for (let i=0; i<items.length; i++) {
-        const re = new RegExp(activeClass, 'g');
-        items[i].className = items[i].className.replace(re, '');
+        $(items[i]).removeClass(parentActiveClass + " " + activeClass);
     }
 
+    var parent = a.parentElement;
     // Add active class to parent.
-    a.parentElement.className = a.parentElement.className + ` ${activeClass}`;
+    $(parent).addClass (activeClass);
 
     // Add parent active class to parent list items.
     let el = a.parentElement.parentElement;
     while (el.className.match('nav--vertical') === null) {
         if (el.nodeName === 'LI') {
-            el.className = el.className + ` ${parentActiveClass}`;
+            $(el).addClass(parentActiveClass);
         }
 
         el = el.parentElement;