first solution

README.md

@@ -28,4 +28,4 @@ implement the ability to filter and sort people in the table.
 - Implement a solution following the [React task guideline](https://github.com/mate-academy/react_task-guideline#react-tasks-guideline).
 - Use the [React TypeScript cheat sheet](https://mate-academy.github.io/fe-program/js/extra/react-typescript).
 - Open one more terminal and run tests with `npm test` to ensure your solution is correct.
-- Replace `<your_account>` with your Github username in the [DEMO LINK](https://<your_account>.github.io/react_people-table-advanced/) and add it to the PR description.
+- Replace `<your_account>` with your Github username in the [DEMO LINK](https://vensecue.github.io/react_people-table-advanced/) and add it to the PR description.

src/App.tsx

@@ -1,3 +1,4 @@
+import { Navigate, Route, Routes } from 'react-router-dom';
 import { PeoplePage } from './components/PeoplePage';
 import { Navbar } from './components/Navbar';
 
@@ -10,9 +11,18 @@ export const App = () => {
 
       <div className="section">
         <div className="container">
-          <h1 className="title">Home Page</h1>
-          <h1 className="title">Page not found</h1>
-          <PeoplePage />
+          <Routes>
+            <Route path="/" element={<h1 className="title">Home Page</h1>} />
+            <Route
+              path="*"
+              element={<h1 className="title">Page not found</h1>}
+            />
+            <Route path="/home" element={<Navigate to="/" replace />} />
+            <Route path="/people" element={<PeoplePage />}>
+              <Route path=":slug" element={<PeoplePage />} />
+            </Route>
+          </Routes>
+
         </div>
       </div>
     </div>

src/components/Navbar.tsx

@@ -1,3 +1,6 @@
+import cn from 'classnames';
+import { NavLink } from 'react-router-dom';
+
 export const Navbar = () => {
   return (
     <nav
@@ -8,15 +11,22 @@ export const Navbar = () => {
     >
       <div className="container">
         <div className="navbar-brand">
-          <a className="navbar-item" href="#/">Home</a>
+          <NavLink
+            className={({ isActive }) => cn('navbar-item',
+              { 'has-background-grey-lighter': isActive })}
+            to="/"
+          >
+            Home
+          </NavLink>
 
-          <a
+          <NavLink
             aria-current="page"
-            className="navbar-item has-background-grey-lighter"
-            href="#/people"
+            className={({ isActive }) => cn('navbar-item',
+              { 'has-background-grey-lighter': isActive })}
+            to="/people"
           >
             People
-          </a>
+          </NavLink>
         </div>
       </div>
     </nav>

src/components/PeopleFilters.tsx

@@ -1,92 +1,159 @@
+import { useSearchParams } from 'react-router-dom';
+import cn from 'classnames';
+import { SearchLink } from './SearchLink';
+
 export const PeopleFilters = () => {
+  const [searchParams, setSearchParams] = useSearchParams();
+  const selectedGender = searchParams.get('sex');
+  const allCenturies = searchParams.getAll('centuries');
+  const query = searchParams.get('query');
+
+  const centurySetter = (century: string) => {
+    if (allCenturies.includes(century)) {
+      return allCenturies.filter(age => age !== century);
+    }
+
+    return [...allCenturies, century];
+  };
+
+  const handleQueryChange = (newValue: string) => {
+    if (newValue === '') {
+      searchParams.delete('query');
+    } else {
+      searchParams.set('query', newValue);
+    }
+
+    setSearchParams(searchParams);
+  };
+
   return (
     <nav className="panel">
       <p className="panel-heading">Filters</p>
 
       <p className="panel-tabs" data-cy="SexFilter">
-        <a className="is-active" href="#/people">All</a>
-        <a className="" href="#/people?sex=m">Male</a>
-        <a className="" href="#/people?sex=f">Female</a>
+        <SearchLink
+          className={cn({ 'is-active': selectedGender === null })}
+          params={{ sex: null }}
+        >
+          All
+
+        </SearchLink>
+        <SearchLink
+          className={cn({ 'is-active': selectedGender === 'm' })}
+          params={{ sex: 'm' }}
+        >
+          Male
+
+        </SearchLink>
+        <SearchLink
+          className={cn({ 'is-active': selectedGender === 'f' })}
+          params={{ sex: 'f' }}
+        >
+          Female
+
+        </SearchLink>
       </p>
 
       <div className="panel-block">
         <p className="control has-icons-left">
           <input
+            value={query || ''}
+            onChange={(e) => handleQueryChange(e.target.value)}
             data-cy="NameFilter"
             type="search"
             className="input"
             placeholder="Search"
           />
-
           <span className="icon is-left">
             <i className="fas fa-search" aria-hidden="true" />
           </span>
         </p>
       </div>
-
       <div className="panel-block">
         <div className="level is-flex-grow-1 is-mobile" data-cy="CenturyFilter">
           <div className="level-left">
-            <a
+            <SearchLink
               data-cy="century"
-              className="button mr-1"
-              href="#/people?centuries=16"
+              className={cn('button mr-1',
+                { 'is-info': allCenturies.includes('16') })}
+              params={{
+                centuries: centurySetter('16'),
+              }}
             >
               16
-            </a>
+            </SearchLink>
 
-            <a
+            <SearchLink
               data-cy="century"
-              className="button mr-1 is-info"
-              href="#/people?centuries=17"
+              className={cn('button mr-1',
+                { 'is-info': allCenturies.includes('17') })}
+              params={{
+                centuries: centurySetter('17'),
+              }}
             >
               17
-            </a>
+            </SearchLink>
 
-            <a
+            <SearchLink
               data-cy="century"
-              className="button mr-1 is-info"
-              href="#/people?centuries=18"
+              className={cn('button mr-1',
+                { 'is-info': allCenturies.includes('18') })}
+              params={{
+                centuries: centurySetter('18'),
+              }}
             >
               18
-            </a>
+            </SearchLink>
 
-            <a
+            <SearchLink
               data-cy="century"
-              className="button mr-1 is-info"
-              href="#/people?centuries=19"
+              className={cn('button mr-1',
+                { 'is-info': allCenturies.includes('19') })}
+              params={{
+                centuries: centurySetter('19'),
+              }}
             >
               19
-            </a>
+            </SearchLink>
 
-            <a
+            <SearchLink
               data-cy="century"
-              className="button mr-1"
-              href="#/people?centuries=20"
+              className={cn('button mr-1',
+                { 'is-info': allCenturies.includes('20') })}
+              params={{
+                centuries: centurySetter('20'),
+              }}
             >
               20
-            </a>
+            </SearchLink>
           </div>
 
           <div className="level-right ml-4">
-            <a
+            <SearchLink
               data-cy="centuryALL"
-              className="button is-success is-outlined"
-              href="#/people"
+              className={cn('button is-success',
+                { 'is-outlined': allCenturies.length !== 0 })}
+              params={{
+                centuries: null,
+              }}
             >
               All
-            </a>
+            </SearchLink>
           </div>
         </div>
       </div>
 
       <div className="panel-block">
-        <a
+        <SearchLink
           className="button is-link is-outlined is-fullwidth"
-          href="#/people"
+          params={{
+            centuries: null,
+            query: null,
+            sex: null,
+          }}
         >
           Reset all filters
-        </a>
+        </SearchLink>
       </div>
     </nav>
   );

src/components/PeoplePage.tsx

@@ -1,31 +1,72 @@
+import { useEffect, useState } from 'react';
 import { PeopleFilters } from './PeopleFilters';
 import { Loader } from './Loader';
 import { PeopleTable } from './PeopleTable';
+import { Person } from '../types';
+import { getPeople } from '../api';
+
+const preparePeople = (peop: Person[]) => {
+  return peop.map(person => {
+    const mother = peop.find(human => human.name === person.motherName);
+    const father = peop.find(human => human.name === person.fatherName);
+
+    return ({
+      ...person,
+      mother,
+      father,
+    });
+  });
+};
 
 export const PeoplePage = () => {
+  const [people, setPeople] = useState<Person[] | null>(null);
+  const [loading, setLoading] = useState(true);
+  const [loadingError, setLoadingError] = useState(false);
+
+  const fetchData = () => {
+    getPeople()
+      .then((result) => {
+        setPeople(preparePeople(result));
+      })
+      .catch(() => {
+        setLoadingError(true);
+      })
+      .finally(() => {
+        setLoading(false);
+      });
+  };
+
+  useEffect(() => {
+    fetchData();
+  }, []);
+
   return (
     <>
       <h1 className="title">People Page</h1>
 
       <div className="block">
         <div className="columns is-desktop is-flex-direction-row-reverse">
           <div className="column is-7-tablet is-narrow-desktop">
-            <PeopleFilters />
+            {!loading && <PeopleFilters />}
           </div>
 
           <div className="column">
             <div className="box table-container">
-              <Loader />
-
-              <p data-cy="peopleLoadingError">Something went wrong</p>
+              {loading && <Loader />}
 
-              <p data-cy="noPeopleMessage">
-                There are no people on the server
-              </p>
+              {loadingError && (
+                <p data-cy="peopleLoadingError" className="has-text-danger">
+                  Something went wrong
+                </p>
+              )}
 
-              <p>There are no people matching the current search criteria</p>
+              {people?.length === 0 && (
+                <p data-cy="noPeopleMessage">
+                  There are no people on the server
+                </p>
+              ) }
 
-              <PeopleTable />
+              {people && <PeopleTable people={people} />}
             </div>
           </div>
         </div>