fix: parent link of collapsing menu in mobile nav doesn't load new page

inc/class-wp-bootstrap-navwalker.php

@@ -255,7 +255,6 @@ public function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0
 					$atts['class'][]     = 'dropdown-toggle';
 					if ( ! $this->hover ) {
 						$atts['data-toggle'] = 'dropdown';
-						$atts['href']        = '#';
 					}
 				} else {
 					$collpse_as_button = apply_filters( 'cpschool_nav_collapse_as_button', false, $item, $args, $depth );
@@ -264,7 +263,7 @@ public function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0
 						if ( ! $collpse_as_button ) {
 							$atts['class'][] = 'collapse-toggle';
 							$atts['data-toggle'] = 'collapse';
-							$atts['href']        = '#' . $data_target;
+							$atts['data-target'] = '#' . $data_target;
 						}
 					} else {
 						if ( ! $collpse_as_button ) {

js/theme.js

@@ -7468,8 +7468,20 @@ var cpSchoolThemeHelpers = function ($) {
         }
       }
     });
-  } // This will make vars in css work in IE11.
+  }
+
+  $('a.collapse-toggle').on('touchend', function (e) {
+    var isExpanded = $(this).attr('aria-expanded');
+
+    if (isExpanded === 'true') {
+      e.preventDefault();
+      var link = $(this).attr('href');
 
+      if (!link.startsWith('#')) {
+        window.location = link;
+      }
+    }
+  }); // This will make vars in css work in IE11.
 
   if (typeof cssVars === "function") {
     cssVars({

src/js/custom-javascript.js

@@ -95,10 +95,21 @@
 		});
 	}
 
+	$('nav.menu-mobile a.collapse-toggle').on('touchend', function(e) {
+		var isExpanded = $(this).attr('aria-expanded');
+		if(isExpanded === 'true') {
+			e.preventDefault();
+			var link = $(this).attr('href');
+			if (!link.startsWith('#')) {
+				window.location = link;
+			}
+		}
+	})
+
 	// This will make vars in css work in IE11.
 	if (typeof cssVars === "function") {
 		cssVars({
 			preserveStatic: false
 		});
 	}
-})(jQuery);
+})(jQuery);