Removed invisible home link from main navigiation. Fixed navigation k…

…eyboard preventDefault issue.
Office-of-Digital-Services · Mar 13, 2019 · 9188764 · 9188764
1 parent d06b623
commit 9188764
Show file tree

Hide file tree

Showing 8 changed files with 7 additions and 18 deletions.
diff --git a/js/cagov.core.js b/js/cagov.core.js
@@ -1,5 +1,5 @@
 /**
- * CA State Template v5.5 -  @version v5.5.2 -  3/5/2019 
+ * CA State Template v5.5 -  @version v5.5.2 -  3/13/2019 
   STYLES COMPILED FROM SOURCE (source/js) DO NOT MODIFY */
 /*! modernizr (Custom Build) | MIT *
  * https://modernizr.com/download/?-flexbox-setclasses !*/
@@ -15214,7 +15214,7 @@ limitations under the License.
                 case Keyboard.SPACE:
                 case Keyboard.ENTER:
                     if (isTopNavItem) {
-                        event.preventDefault();
+                       // event.preventDefault();
                         _clickHandler.call(that, event);
                     } else {
                         return true;

diff --git a/js/cagov.core.min.js b/js/cagov.core.min.js
diff --git a/sample/modules/nav-dev.html b/sample/modules/nav-dev.html
@@ -1,9 +1,7 @@
 <!-- Megamenu dropdown -->
     <nav id="navigation" class="main-navigation megadropdown off-canvas">
 		<ul id="nav_list" class="top-level-nav">
-        	<li class="home-link nav-item">
-            	<a href="/" class="first-level-link"><span id="nav_home_container" class="ca-gov-icon-home" aria-hidden="true"></span><span class="sr-only">Home</span></a>
-            </li>
+
 			<li class="nav-item">
             	<a href="/" class="first-level-link">Link 1</a>
             	<div class="sub-nav">

diff --git a/sample/modules/nav-megamenu-orig.html b/sample/modules/nav-megamenu-orig.html
@@ -7,9 +7,7 @@
 
 <nav id="navigation" class="main-navigation megadropdown original">
     <ul id="nav_list" class="top-level-nav">
-        <li class="home-link nav-item">
-            <a href="/" class="first-level-link"><span id="nav_home_container" class="ca-gov-icon-home" aria-hidden="true"></span><span class="sr-only">Home</span></a>
-        </li>
+
         <li class="nav-item active">
             <a href="/sample/" class="first-level-link has-sub"><span class="ca-gov-icon-gears"></span>Examples</a>
             <span class="mobile-control toggle-sub-nav closed"><span class="ca-gov-icon-menu-toggle-open" aria-hidden="true"></span><span class="sr-only">Sub Menu Toggle</span></span><div class="sub-nav">

diff --git a/sample/modules/nav-singlelevel-standard-mobile.html b/sample/modules/nav-singlelevel-standard-mobile.html
@@ -1,7 +1,6 @@
 <!-- Top Level Navigation Only -->
 <nav id="navigation" class="main-navigation singlelevelnav auto-highlight no-icons">
     <ul id="nav_list" class="top-level-nav">
-        <li class="home-link nav-item"><a href="/" class="first-level-link"><span id="nav_home_container" class="ca-gov-icon-home" aria-hidden="true"></span><span class="sr-only">Home</span></a></li>
         <li class="nav-item"><a href="/sample/" class="first-level-link">Examples</a></li>
         <li class="nav-item"><a href="/about.html" class="first-level-link">About</a></li>
         <li class="nav-item"><a href="https://github.com/Office-of-Digital-Innovation" target="_blank" class="first-level-link">Download</a></li>

diff --git a/sample/modules/nav-singlelevel.html b/sample/modules/nav-singlelevel.html
@@ -1,9 +1,6 @@
 <!-- Top Level Navigation Only -->
 <nav id="navigation" class="main-navigation singlelevelnav auto-highlight">
     <ul id="nav_list" class="top-level-nav">
-         	<li class="home-link nav-item">
-            	<a href="/" class="first-level-link"><span id="nav_home_container" class="ca-gov-icon-home" aria-hidden="true"></span>Home</a>
-        	</li>
         	<li class="nav-item">
             	<a href="/sample/" class="first-level-link"><span class="ca-gov-icon-gears"></span>Examples</a>
             </li>

diff --git a/source/js/cagov/navigation.js b/source/js/cagov/navigation.js
@@ -594,7 +594,7 @@ limitations under the License.
                 case Keyboard.SPACE:
                 case Keyboard.ENTER:
                     if (isTopNavItem) {
-                        event.preventDefault();
+                       // event.preventDefault();
                         _clickHandler.call(that, event);
                     } else {
                         return true;

diff --git a/ssi/navigation.html b/ssi/navigation.html
@@ -7,9 +7,6 @@
 
 <nav id="navigation" class="main-navigation megadropdown auto-highlight">
     <ul id="nav_list" class="top-level-nav">
-        <li class="home-link nav-item">
-            <a href="/" class="first-level-link"><span id="nav_home_container" class="ca-gov-icon-home" aria-hidden="true"></span>Home</a>
-        </li>
         <li class="nav-item">
             <a href="/sample/" class="first-level-link"><span class="ca-gov-icon-gears"></span>Examples</a>
             <div class="sub-nav">