feat(ui): use React Router NavLinks for the list filters

redis-developer · Feb 23, 2021 · 23c95c0 · 23c95c0
1 parent bcc71ea
commit 23c95c0
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/src/main/webapp/src/components/todo_list_controls.js b/src/main/webapp/src/components/todo_list_controls.js
@@ -1,5 +1,6 @@
 import React, { useMemo, useContext } from 'react';
 import { TodosContext } from "../context/todos_context";
+import { NavLink } from "react-router-dom";
 
 const TodoListControls = function (props) {
   const { todos, clearCompleted } = useContext(TodosContext);
@@ -19,14 +20,14 @@ const TodoListControls = function (props) {
       {/*  Remove this if you don't implement routing  */}
       <ul className="filters">
         <li>
-				  <a className="selected" href="#/">All</a>
-			  </li>
-				<li>
-					<a href="#/active">Active</a>
-				</li>
-				<li>
-					<a href="#/completed">Completed</a>
-				</li>
+          <NavLink to="/" id="todos-all" exact={true} activeClassName="selected">All</NavLink>
+        </li>
+        <li>
+          <NavLink to="/active" id="todos-active" activeClassName="selected">Active</NavLink>
+        </li>
+        <li>
+          <NavLink to="/completed" id="todos-completed" activeClassName="selected">Completed</NavLink>
+        </li>
       </ul>
       {/*  Hidden if no completed items are left ↓  */}
       {itemsCompleted > 0 &&