HSD8-744 fix menu keyboard (#446)

* HSD8-744 Fix main menu keyboard navigation * jquery only select the immediate children Co-authored-by: Sherakama <[email protected]>
SU-HSDO · Jan 17, 2020 · 5d04636 · 5d04636
1 parent 5de16aa
commit 5d04636
Show file tree

Hide file tree

Showing 8 changed files with 11 additions and 9 deletions.
diff --git a/docroot/themes/humsci/humsci_basic/src/scss/_base.scss b/docroot/themes/humsci/humsci_basic/src/scss/_base.scss
@@ -4,12 +4,12 @@ html {
   color: color(brand);
 
   @include colorful {
-    color: purple;
+    color: #800080;
     background-color: theme-color(color-purple);
   }
 
   @include airy {
-    color: blue;
+    color: #0000ff;
     font-family: $colorful-font--sans;
   }
 

diff --git a/docroot/themes/humsci/humsci_basic/src/scss/_main.scss b/docroot/themes/humsci/humsci_basic/src/scss/_main.scss
@@ -86,4 +86,4 @@
   // 8. Utilities
   //    Utilities and helper classes with ability to override anything
   //    which goes before in the triangle, eg. hide helper class
-  "base"; // TODO test file
+  'base'; // TODO test file
diff --git a/docroot/themes/humsci/humsci_basic/src/scss/humsci_airy.scss b/docroot/themes/humsci/humsci_basic/src/scss/humsci_airy.scss
@@ -1,3 +1,3 @@
 $current-theme: 'airy';
 
-@import "main";
+@import 'main';
diff --git a/docroot/themes/humsci/humsci_basic/src/scss/humsci_colorful.scss b/docroot/themes/humsci/humsci_basic/src/scss/humsci_colorful.scss
@@ -1,3 +1,3 @@
 $current-theme: 'colorful';
 
-@import "main";
+@import 'main';
diff --git a/docroot/themes/humsci/humsci_basic/src/scss/humsci_traditional.scss b/docroot/themes/humsci/humsci_basic/src/scss/humsci_traditional.scss
@@ -1,3 +1,3 @@
 $current-theme: 'traditional';
 
-@import "main";
+@import 'main';
diff --git a/docroot/themes/humsci/su_humsci_theme/css/components/molecules/index.css b/docroot/themes/humsci/su_humsci_theme/css/components/molecules/index.css
diff --git a/docroot/themes/humsci/su_humsci_theme/js/hover_menu.js b/docroot/themes/humsci/su_humsci_theme/js/hover_menu.js
@@ -52,7 +52,10 @@
       // Desktop, apply the jquery ui menu and change any mobile classes.
       if (window.innerWidth >= 1201) {
         $menu.menu({
-          position: {at: "left bottom"}
+          position: {at: "left bottom"},
+          select: function(e, item){
+           window.location.href = $(item.item).children('a:first-child').attr("href");
+          }
         });
         $menu.removeClass('expanded');
         $menu.find('.fa-minus').addClass('fa-plus').removeClass('fa-minus');

diff --git a/docroot/themes/humsci/su_humsci_theme/scss/components/molecules/_hover-menu.scss b/docroot/themes/humsci/su_humsci_theme/scss/components/molecules/_hover-menu.scss
@@ -157,7 +157,6 @@
 
         .decanter-nav-submenu .menu-item a,
         .decanter-nav-submenu .menu-item span {
-          border-bottom: 0;
           color: $color-bright-red;
 
           @include margin(0 1.5em 10px 0);