From d56d5573b332ac0ce7f569602f0345040b8787f5 Mon Sep 17 00:00:00 2001
From: Sachin Chaurasiya <sachinchaurasiyachotey87@gmail.com>
Date: Sun, 24 Oct 2021 16:22:06 +0530
Subject: [PATCH] feat: add grid layout for user-list and refactor searchbar
 (#596)

* add Sachin-chaurasiya.json

* Add React router component for links

* Add grid layout for user-list

closes #583
---
 src/App.js                       | 20 +++++++--------
 src/Components/Home/Home.css     | 37 +++++++++++++++++++++++++---
 src/Components/Home/Home.js      |  2 +-
 src/Components/Home/Searchbar.js | 27 ++++++++++++++++++++
 src/Components/Home/Users.js     | 42 ++++++++++++++------------------
 5 files changed, 89 insertions(+), 39 deletions(-)
 create mode 100644 src/Components/Home/Searchbar.js

diff --git a/src/App.js b/src/App.js
index 7b9769de6b0..8ccf03eff68 100644
--- a/src/App.js
+++ b/src/App.js
@@ -13,17 +13,15 @@ import Home from './Components/Home/Home'
 function App() {
   return (
     <Router>
-      <div className="p-m-4">
-        <Switch>
-          <Route path="/:username">
-            <Socials />
-          </Route>
-          <Route path="/">
-            <Home />
-          </Route>
-        </Switch>
-        <Footer />
-      </div>
+      <Switch>
+        <Route path="/:username">
+          <Socials />
+        </Route>
+        <Route path="/">
+          <Home />
+        </Route>
+      </Switch>
+      <Footer />
     </Router>
   )
 }
diff --git a/src/Components/Home/Home.css b/src/Components/Home/Home.css
index 86fafec77f0..5b8c2fd0656 100644
--- a/src/Components/Home/Home.css
+++ b/src/Components/Home/Home.css
@@ -1,3 +1,17 @@
+.container {
+  width: 100%;
+  margin-left: auto;
+  margin-right: auto;
+  padding-left: 24px;
+  padding-right: 24px;
+}
+
+.user-list {
+  display: grid;
+  grid-template-columns: repeat(4, 1fr);
+  place-items: center;
+}
+
 a {
   text-decoration: none;
 }
@@ -27,13 +41,30 @@ a {
 .search-section {
   display: flex;
   justify-content: center;
-  margin-bottom: 10px;
+  margin: 10px 0px;
 }
 
 label {
-  text-decoration:underline;
+  text-decoration: underline;
   text-align: center;
   display: flex;
   justify-content: center;
-  margin-bottom: 10px;  
+  margin-bottom: 10px;
+}
+
+@media screen and (max-width: 1024px) {
+  .user-list {
+    grid-template-columns: repeat(3, 1fr);
+  }
+}
+@media screen and (max-width: 768px) {
+  .user-list {
+    grid-template-columns: repeat(2, 1fr);
+  }
+}
+@media screen and (max-width: 480px) {
+  .user-list {
+    place-items: left;
+    grid-template-columns: repeat(1, 1fr);
+  }
 }
diff --git a/src/Components/Home/Home.js b/src/Components/Home/Home.js
index b5687c14722..09d021bd458 100644
--- a/src/Components/Home/Home.js
+++ b/src/Components/Home/Home.js
@@ -29,7 +29,7 @@ function Home() {
   }, [])
 
   return (
-    <main>
+    <main className="container">
       {showProgress && <ProgressBar mode="indeterminate" />}
       {skeleton ? <Placeholders list={list} /> : <Users list={list} />}
     </main>
diff --git a/src/Components/Home/Searchbar.js b/src/Components/Home/Searchbar.js
new file mode 100644
index 00000000000..9436f84b675
--- /dev/null
+++ b/src/Components/Home/Searchbar.js
@@ -0,0 +1,27 @@
+import React from 'react'
+import PropTypes from 'prop-types'
+import { InputText } from 'primereact/inputtext'
+
+const Searchbar = ({ searchHandler, searchTerm }) => {
+  return (
+    <div className="search-section">
+      <span className="p-input-icon-left">
+        <i className="pi pi-search" />
+        <InputText
+          type="search"
+          value={searchTerm}
+          onChange={({ target }) => searchHandler(target.value)}
+          name="user"
+          id="search-input"
+          placeholder="Search user..."
+        />
+      </span>
+    </div>
+  )
+}
+Searchbar.propTypes = {
+  searchHandler: PropTypes.func.isRequired,
+  searchTerm: PropTypes.string.isRequired,
+}
+
+export default Searchbar
diff --git a/src/Components/Home/Users.js b/src/Components/Home/Users.js
index 46ec58fa33a..6fe2d31cb0d 100644
--- a/src/Components/Home/Users.js
+++ b/src/Components/Home/Users.js
@@ -1,40 +1,34 @@
 import React, { useState } from 'react'
 import PropTypes from 'prop-types'
 import { Chip } from 'primereact/chip'
-import { InputText } from 'primereact/inputtext'
 import { Link } from 'react-router-dom'
+import Searchbar from './Searchbar'
 
-function User({ list }) {
+function Users({ list }) {
   const [searchTerm, setSearchTerm] = useState('')
+  const searchHandler = (value) => {
+    setSearchTerm(value || '')
+  }
   return (
     <>
-      <div className="search-section">
-        <span className="p-input-icon-left">
-          <i className="pi pi-search" />
-          <InputText
-            value={searchTerm}
-            onChange={({ target }) => setSearchTerm(target.value)}
-            name="user"
-            id="search-input"
-            placeholder="Search..."
-          />
-        </span>
+      <Searchbar searchTerm={searchTerm} searchHandler={searchHandler} />
+      <div className="user-list">
+        {list
+          .filter((User) =>
+            User.name.toLowerCase().includes(searchTerm.toLowerCase()),
+          )
+          .map((user, key) => (
+            <Link to={`${user.username}`} key={`avatar-${key}`}>
+              <Chip image={user.avatar} className="p-m-2" label={user.name} />
+            </Link>
+          ))}
       </div>
-      {list
-        .filter((User) =>
-          User.name.toLowerCase().includes(searchTerm.toLowerCase()),
-        )
-        .map((user, key) => (
-          <Link to={`${user.username}`} key={`avatar-${key}`}>
-            <Chip image={user.avatar} className="p-m-2" label={user.name} />
-          </Link>
-        ))}
     </>
   )
 }
 
-User.propTypes = {
+Users.propTypes = {
   list: PropTypes.array.isRequired,
 }
 
-export default User
+export default Users