From ea8cd21bb197e4b81733d0b4edd8a03fe99b8a10 Mon Sep 17 00:00:00 2001
From: Gabriel Borges <gabrielborgesdemoraes@gmail.com>
Date: Thu, 23 May 2024 21:28:52 -0300
Subject: [PATCH] feat: define a height limit for tables

---
 frontend/src/pages/Authors/Authors.tsx | 76 ++++++++++++++------------
 frontend/src/pages/Books/Books.tsx     | 67 ++++++++++++-----------
 2 files changed, 75 insertions(+), 68 deletions(-)

diff --git a/frontend/src/pages/Authors/Authors.tsx b/frontend/src/pages/Authors/Authors.tsx
index 7990143..347de94 100644
--- a/frontend/src/pages/Authors/Authors.tsx
+++ b/frontend/src/pages/Authors/Authors.tsx
@@ -38,45 +38,49 @@ const Authors: React.FC = () => {
                 </button>
             </div>
             <hr />
-            <table className="w-full text-sm text-left rtl:text-right text-gray-500">
-                <thead className="text-xs uppercase ">
-                    <tr>
+            <div className='block max-h-[78dvh] overflow-y-auto'>
+                <table className="w-full text-sm text-left rtl:text-right text-gray-500">
+                    <thead className="text-xs uppercase ">
+                        <tr>
 
-                        <th scope="col" className="px-1 py-3">
-                            Author
-                        </th>
-                        <th scope="col" className="px-1 py-3">
-                            Nationality
-                        </th>
-                        <th scope="col" className="px-1 py-3">
-                            Birth date
-                        </th>
-                        <th scope="col" className="px-1 py-3">
-                            E-mail
-                        </th>
-                    </tr>
-                </thead>
-                <tbody>{authors?.length > 0 && authors.map(({ name, nationality, birthDate, email, id }: Author) => (
-                    shouldFilterInWith(name, nationality, birthDate, email, id)
-                    && (
-                        <tr key={id} className="border-b hover:bg-gray-50">
-                            <th scope="row" className="px-1 py-4 font-medium text-gray-900 whitespace-nowrap">
-                                {name}
+                            <th scope="col" className="px-1 py-3">
+                                Author
+                            </th>
+                            <th scope="col" className="px-1 py-3">
+                                Nationality
+                            </th>
+                            <th scope="col" className="px-1 py-3">
+                                Birth date
+                            </th>
+                            <th scope="col" className="px-1 py-3">
+                                E-mail
                             </th>
-                            <td className="px-1 py-4">
-                                {nationality}
-                            </td>
-                            <td className="px-1 py-4">
-                                {birthDate}
-                            </td>
-                            <td className="px-1 py-4">
-                                {email}
-                            </td>
                         </tr>
-                    )
-                ))}</tbody>
-            </table>
-        </div>
+                    </thead>
+                    <tbody>
+                        {authors?.length > 0 && authors.map(({ name, nationality, birthDate, email, id }: Author) => (
+                            shouldFilterInWith(name, nationality, birthDate, email, id)
+                            && (
+                                <tr key={id} className="border-b hover:bg-gray-50">
+                                    <th scope="row" className="px-1 py-4 font-medium text-gray-900 whitespace-nowrap">
+                                        {name}
+                                    </th>
+                                    <td className="px-1 py-4">
+                                        {nationality}
+                                    </td>
+                                    <td className="px-1 py-4">
+                                        {birthDate}
+                                    </td>
+                                    <td className="px-1 py-4">
+                                        {email}
+                                    </td>
+                                </tr>
+                            )
+                        ))}
+                    </tbody>
+                </table>
+            </div>
+        </div >
     );
 };
 
diff --git a/frontend/src/pages/Books/Books.tsx b/frontend/src/pages/Books/Books.tsx
index ca2f8e5..a90b069 100644
--- a/frontend/src/pages/Books/Books.tsx
+++ b/frontend/src/pages/Books/Books.tsx
@@ -41,41 +41,44 @@ const Books: React.FC = () => {
                 </button>
             </div>
             <hr />
-            <table className="w-full text-sm text-left rtl:text-right text-gray-500">
-                <thead className="text-xs uppercase ">
-                    <tr>
+            <div className='block max-h-[78dvh] overflow-y-auto'>
+                <table className="w-full text-sm text-left rtl:text-right text-gray-500">
+                    <thead className="text-xs uppercase ">
+                        <tr>
 
-                        <th scope="col" className="px-1 py-3">
-                            Book title
-                        </th>
-                        <th scope="col" className="px-1 py-3">
-                            Pages
-                        </th>
-                        <th scope="col" className="px-1 py-3">
-                            Authors
-                        </th>
-                    </tr>
-                </thead>
-                <tbody>{books?.length > 0 && books.map(({ id, title, pages, authorsNames }: Book) => (
-                    shouldFilterInWith(title, pages, id, authorsNames)
-                    && (
-                        <tr key={id} className="border-b hover:bg-gray-50">
-                            <th scope="row" className="px-1 py-4 font-medium text-gray-900 whitespace-nowrap">
-                                {title}
+                            <th scope="col" className="px-1 py-3">
+                                Book title
+                            </th>
+                            <th scope="col" className="px-1 py-3">
+                                Pages
+                            </th>
+                            <th scope="col" className="px-1 py-3">
+                                Authors
                             </th>
-                            <td className="px-1 py-4">
-                                {Number.isInteger(pages) ? `${pages} pages` : "Not informed"}
-                            </td>
-                            <td className="px-1 py-4">
-                                {
-                                    <span className='block max-w-52 truncate'>{authorsNames}</span>
-                                }
-                            </td>
                         </tr>
-                    )
-                ))}</tbody>
-            </table>
-        </div>
+                    </thead>
+                    <tbody>
+                        {books?.length > 0 && books.map(({ id, title, pages, authorsNames }: Book) => (
+                            shouldFilterInWith(title, pages, id, authorsNames) && (
+                                <tr key={id} className="border-b hover:bg-gray-50">
+                                    <th scope="row" className="px-1 py-4 font-medium text-gray-900 whitespace-nowrap">
+                                        {title}
+                                    </th>
+                                    <td className="px-1 py-4">
+                                        {Number.isInteger(pages) ? `${pages} pages` : "Not informed"}
+                                    </td>
+                                    <td className="px-1 py-4">
+                                        {
+                                            <span className='block max-w-52 truncate'>{authorsNames}</span>
+                                        }
+                                    </td>
+                                </tr>
+                            )
+                        ))}
+                    </tbody>
+                </table >
+            </div>
+        </div >
     );
 };