From 9cf75b8fc429a79984ad3c1fccf4f18dd90841e3 Mon Sep 17 00:00:00 2001
From: Aaron Ang <awz.ang@gmail.com>
Date: Mon, 12 Dec 2016 10:39:28 +0100
Subject: [PATCH] Use HTML navigation element

In this patch, the navigation list is changed such that the proper HTML
navigation element is used.
---
 lessons/03-navigating-with-link/README.md      | 10 ++++++----
 lessons/04-nested-routes/README.md             | 11 ++++++-----
 lessons/04-nested-routes/modules/App.js        | 10 ++++++----
 lessons/05-active-links/modules/App.js         | 10 ++++++----
 lessons/06-params/modules/App.js               | 10 ++++++----
 lessons/07-more-nesting/modules/App.js         | 10 ++++++----
 lessons/08-index-routes/modules/App.js         | 10 ++++++----
 lessons/09-index-links/modules/App.js          | 10 ++++++----
 lessons/10-clean-urls/modules/App.js           | 12 +++++++-----
 lessons/11-productionish-server/modules/App.js | 12 +++++++-----
 lessons/12-navigating/modules/App.js           | 12 +++++++-----
 lessons/13-server-rendering/modules/App.js     | 12 +++++++-----
 lessons/14-whats-next/modules/App.js           | 12 +++++++-----
 13 files changed, 83 insertions(+), 58 deletions(-)

diff --git a/lessons/03-navigating-with-link/README.md b/lessons/03-navigating-with-link/README.md
index fd1518d8..504fe446 100644
--- a/lessons/03-navigating-with-link/README.md
+++ b/lessons/03-navigating-with-link/README.md
@@ -16,10 +16,12 @@ export default React.createClass({
     return (
       <div>
         <h1>React Router Tutorial</h1>
-        <ul role="nav">
-          <li><Link to="/about">About</Link></li>
-          <li><Link to="/repos">Repos</Link></li>
-        </ul>
+        <nav>
+          <ul>
+            <li><Link to="/about">About</Link></li>
+            <li><Link to="/repos">Repos</Link></li>
+          </ul>
+        </nav>
       </div>
     )
   }
diff --git a/lessons/04-nested-routes/README.md b/lessons/04-nested-routes/README.md
index bac8f2cd..bce88115 100644
--- a/lessons/04-nested-routes/README.md
+++ b/lessons/04-nested-routes/README.md
@@ -76,11 +76,12 @@ Next, render children inside of `App`.
     return (
       <div>
         <h1>React Router Tutorial</h1>
-        <ul role="nav">
-          <li><Link to="/about">About</Link></li>
-          <li><Link to="/repos">Repos</Link></li>
-        </ul>
-
+        <nav>
+          <ul>
+            <li><Link to="/about">About</Link></li>
+            <li><Link to="/repos">Repos</Link></li>
+          </ul>
+        </nav>
         {/* add this */}
         {this.props.children}
 
diff --git a/lessons/04-nested-routes/modules/App.js b/lessons/04-nested-routes/modules/App.js
index 70d49928..afcd46f0 100644
--- a/lessons/04-nested-routes/modules/App.js
+++ b/lessons/04-nested-routes/modules/App.js
@@ -6,10 +6,12 @@ export default React.createClass({
     return (
       <div>
         <h1>React Router Tutorial</h1>
-        <ul role="nav">
-          <li><Link to="/about">About</Link></li>
-          <li><Link to="/repos">Repos</Link></li>
-        </ul>
+        <nav>
+          <ul>
+            <li><Link to="/about">About</Link></li>
+            <li><Link to="/repos">Repos</Link></li>
+          </ul>
+        </nav>
       </div>
     )
   }
diff --git a/lessons/05-active-links/modules/App.js b/lessons/05-active-links/modules/App.js
index f3867803..52b09a6a 100644
--- a/lessons/05-active-links/modules/App.js
+++ b/lessons/05-active-links/modules/App.js
@@ -6,10 +6,12 @@ export default React.createClass({
     return (
       <div>
         <h1>React Router Tutorial</h1>
-        <ul role="nav">
-          <li><Link to="/about">About</Link></li>
-          <li><Link to="/repos">Repos</Link></li>
-        </ul>
+        <nav>
+          <ul>
+            <li><Link to="/about">About</Link></li>
+            <li><Link to="/repos">Repos</Link></li>
+          </ul>
+        </nav>
         {this.props.children}
       </div>
     )
diff --git a/lessons/06-params/modules/App.js b/lessons/06-params/modules/App.js
index c644187d..8eb975bb 100644
--- a/lessons/06-params/modules/App.js
+++ b/lessons/06-params/modules/App.js
@@ -6,10 +6,12 @@ export default React.createClass({
     return (
       <div>
         <h1>React Router Tutorial</h1>
-        <ul role="nav">
-          <li><NavLink to="/about">About</NavLink></li>
-          <li><NavLink to="/repos">Repos</NavLink></li>
-        </ul>
+        <nav>
+          <ul>
+            <li><NavLink to="/about">About</NavLink></li>
+            <li><NavLink to="/repos">Repos</NavLink></li>
+          </ul>
+        </nav>
         {this.props.children}
       </div>
     )
diff --git a/lessons/07-more-nesting/modules/App.js b/lessons/07-more-nesting/modules/App.js
index c644187d..8eb975bb 100644
--- a/lessons/07-more-nesting/modules/App.js
+++ b/lessons/07-more-nesting/modules/App.js
@@ -6,10 +6,12 @@ export default React.createClass({
     return (
       <div>
         <h1>React Router Tutorial</h1>
-        <ul role="nav">
-          <li><NavLink to="/about">About</NavLink></li>
-          <li><NavLink to="/repos">Repos</NavLink></li>
-        </ul>
+        <nav>
+          <ul>
+            <li><NavLink to="/about">About</NavLink></li>
+            <li><NavLink to="/repos">Repos</NavLink></li>
+          </ul>
+        </nav>
         {this.props.children}
       </div>
     )
diff --git a/lessons/08-index-routes/modules/App.js b/lessons/08-index-routes/modules/App.js
index c644187d..8eb975bb 100644
--- a/lessons/08-index-routes/modules/App.js
+++ b/lessons/08-index-routes/modules/App.js
@@ -6,10 +6,12 @@ export default React.createClass({
     return (
       <div>
         <h1>React Router Tutorial</h1>
-        <ul role="nav">
-          <li><NavLink to="/about">About</NavLink></li>
-          <li><NavLink to="/repos">Repos</NavLink></li>
-        </ul>
+        <nav>
+          <ul>
+            <li><NavLink to="/about">About</NavLink></li>
+            <li><NavLink to="/repos">Repos</NavLink></li>
+          </ul>
+        </nav>
         {this.props.children}
       </div>
     )
diff --git a/lessons/09-index-links/modules/App.js b/lessons/09-index-links/modules/App.js
index c644187d..8eb975bb 100644
--- a/lessons/09-index-links/modules/App.js
+++ b/lessons/09-index-links/modules/App.js
@@ -6,10 +6,12 @@ export default React.createClass({
     return (
       <div>
         <h1>React Router Tutorial</h1>
-        <ul role="nav">
-          <li><NavLink to="/about">About</NavLink></li>
-          <li><NavLink to="/repos">Repos</NavLink></li>
-        </ul>
+        <nav>
+          <ul>
+            <li><NavLink to="/about">About</NavLink></li>
+            <li><NavLink to="/repos">Repos</NavLink></li>
+          </ul>
+        </nav>
         {this.props.children}
       </div>
     )
diff --git a/lessons/10-clean-urls/modules/App.js b/lessons/10-clean-urls/modules/App.js
index bbef4f0e..7fd38dc1 100644
--- a/lessons/10-clean-urls/modules/App.js
+++ b/lessons/10-clean-urls/modules/App.js
@@ -6,11 +6,13 @@ export default React.createClass({
     return (
       <div>
         <h1>React Router Tutorial</h1>
-        <ul role="nav">
-          <li><NavLink to="/" onlyActiveOnIndex>Home</NavLink></li>
-          <li><NavLink to="/about">About</NavLink></li>
-          <li><NavLink to="/repos">Repos</NavLink></li>
-        </ul>
+        <nav>
+          <ul>
+            <li><NavLink to="/" onlyActiveOnIndex>Home</NavLink></li>
+            <li><NavLink to="/about">About</NavLink></li>
+            <li><NavLink to="/repos">Repos</NavLink></li>
+          </ul>
+        </nav>
         {this.props.children}
       </div>
     )
diff --git a/lessons/11-productionish-server/modules/App.js b/lessons/11-productionish-server/modules/App.js
index bbef4f0e..7fd38dc1 100644
--- a/lessons/11-productionish-server/modules/App.js
+++ b/lessons/11-productionish-server/modules/App.js
@@ -6,11 +6,13 @@ export default React.createClass({
     return (
       <div>
         <h1>React Router Tutorial</h1>
-        <ul role="nav">
-          <li><NavLink to="/" onlyActiveOnIndex>Home</NavLink></li>
-          <li><NavLink to="/about">About</NavLink></li>
-          <li><NavLink to="/repos">Repos</NavLink></li>
-        </ul>
+        <nav>
+          <ul>
+            <li><NavLink to="/" onlyActiveOnIndex>Home</NavLink></li>
+            <li><NavLink to="/about">About</NavLink></li>
+            <li><NavLink to="/repos">Repos</NavLink></li>
+          </ul>
+        </nav>
         {this.props.children}
       </div>
     )
diff --git a/lessons/12-navigating/modules/App.js b/lessons/12-navigating/modules/App.js
index bbef4f0e..7fd38dc1 100644
--- a/lessons/12-navigating/modules/App.js
+++ b/lessons/12-navigating/modules/App.js
@@ -6,11 +6,13 @@ export default React.createClass({
     return (
       <div>
         <h1>React Router Tutorial</h1>
-        <ul role="nav">
-          <li><NavLink to="/" onlyActiveOnIndex>Home</NavLink></li>
-          <li><NavLink to="/about">About</NavLink></li>
-          <li><NavLink to="/repos">Repos</NavLink></li>
-        </ul>
+        <nav>
+          <ul>
+            <li><NavLink to="/" onlyActiveOnIndex>Home</NavLink></li>
+            <li><NavLink to="/about">About</NavLink></li>
+            <li><NavLink to="/repos">Repos</NavLink></li>
+          </ul>
+        </nav>
         {this.props.children}
       </div>
     )
diff --git a/lessons/13-server-rendering/modules/App.js b/lessons/13-server-rendering/modules/App.js
index bbef4f0e..7fd38dc1 100644
--- a/lessons/13-server-rendering/modules/App.js
+++ b/lessons/13-server-rendering/modules/App.js
@@ -6,11 +6,13 @@ export default React.createClass({
     return (
       <div>
         <h1>React Router Tutorial</h1>
-        <ul role="nav">
-          <li><NavLink to="/" onlyActiveOnIndex>Home</NavLink></li>
-          <li><NavLink to="/about">About</NavLink></li>
-          <li><NavLink to="/repos">Repos</NavLink></li>
-        </ul>
+        <nav>
+          <ul>
+            <li><NavLink to="/" onlyActiveOnIndex>Home</NavLink></li>
+            <li><NavLink to="/about">About</NavLink></li>
+            <li><NavLink to="/repos">Repos</NavLink></li>
+          </ul>
+        </nav>
         {this.props.children}
       </div>
     )
diff --git a/lessons/14-whats-next/modules/App.js b/lessons/14-whats-next/modules/App.js
index bbef4f0e..7fd38dc1 100644
--- a/lessons/14-whats-next/modules/App.js
+++ b/lessons/14-whats-next/modules/App.js
@@ -6,11 +6,13 @@ export default React.createClass({
     return (
       <div>
         <h1>React Router Tutorial</h1>
-        <ul role="nav">
-          <li><NavLink to="/" onlyActiveOnIndex>Home</NavLink></li>
-          <li><NavLink to="/about">About</NavLink></li>
-          <li><NavLink to="/repos">Repos</NavLink></li>
-        </ul>
+        <nav>
+          <ul>
+            <li><NavLink to="/" onlyActiveOnIndex>Home</NavLink></li>
+            <li><NavLink to="/about">About</NavLink></li>
+            <li><NavLink to="/repos">Repos</NavLink></li>
+          </ul>
+        </nav>
         {this.props.children}
       </div>
     )