[resumes][feat] remove resume title

apps/portal/src/components/resumes/ResumeReviewsTitle.tsx

@@ -1,13 +0,0 @@
-import { Badge } from '@tih/ui';
-
-export default function ResumeReviewsTitle() {
-  return (
-    <div>
-      <h1 className="mb-1 text-2xl font-bold">Resume Reviews</h1>
-      <Badge
-        label="Check out reviewed resumes or look for resumes to review"
-        variant="info"
-      />
-    </div>
-  );
-}

apps/portal/src/pages/resumes/browse.tsx

@@ -4,10 +4,10 @@ import { useSession } from 'next-auth/react';
 import { Fragment, useEffect, useState } from 'react';
 import { Dialog, Disclosure, Transition } from '@headlessui/react';
 import { FunnelIcon, MinusIcon, PlusIcon } from '@heroicons/react/20/solid';
-import { XMarkIcon } from '@heroicons/react/24/outline';
 import {
   MagnifyingGlassIcon,
   NewspaperIcon,
+  XMarkIcon,
 } from '@heroicons/react/24/outline';
 import {
   CheckboxInput,
@@ -36,7 +36,6 @@ import {
   SORT_OPTIONS,
 } from '~/components/resumes/browse/resumeFilters';
 import ResumeListItems from '~/components/resumes/browse/ResumeListItems';
-import ResumeReviewsTitle from '~/components/resumes/ResumeReviewsTitle';
 import ResumeSignInButton from '~/components/resumes/shared/ResumeSignInButton';
 
 import useDebounceValue from '~/utils/resumes/useDebounceValue';
@@ -369,7 +368,7 @@ export default function ResumeHomePage() {
         </Transition.Root>
       </div>
 
-      <main className="h-[calc(100vh-4rem)] flex-1 overflow-y-scroll px-8 pt-8">
+      <main className="h-[calc(100vh-4rem)] flex-auto overflow-y-scroll px-8 pt-6 pb-4">
         <div className="flex justify-start">
           <div className="hidden w-1/6 pt-2 lg:block">
             <h3 className="text-md font-medium tracking-tight text-gray-900">
@@ -531,44 +530,41 @@ export default function ResumeHomePage() {
                 </div>
               </div>
             </div>
-            <div className="mb-4">
+            {isFetchingResumes ? (
-              {isFetchingResumes ? (
+              <div className="w-full pt-4">
-                <div className="w-full pt-4">
+                {' '}
-                  {' '}
+                <Spinner display="block" size="lg" />{' '}
-                  <Spinner display="block" size="lg" />{' '}
+              </div>
-                </div>
+            ) : sessionData === null && tabsValue !== BROWSE_TABS_VALUES.ALL ? (
-              ) : sessionData === null &&
+              <ResumeSignInButton
-                tabsValue !== BROWSE_TABS_VALUES.ALL ? (
+                className="mt-8"
-                <ResumeSignInButton
+                text={getLoggedOutText(tabsValue)}
-                  className="mt-8"
+              />
-                  text={getLoggedOutText(tabsValue)}
+            ) : getTabResumes().length === 0 ? (
+              <div className="mt-24 flex flex-wrap justify-center">
+                <NewspaperIcon
+                  className="mb-12 basis-full"
+                  height={196}
+                  width={196}
                 />
-              ) : getTabResumes().length === 0 ? (
+                {getEmptyDataText(tabsValue, searchValue, userFilters)}
-                <div className="mt-24 flex flex-wrap justify-center">
+              </div>
-                  <NewspaperIcon
+            ) : (
-                    className="mb-12 basis-full"
+              <>
-                    height={196}
+                <ResumeListItems resumes={getTabResumes()} />
-                    width={196}
+                {getTabTotalPages() > 1 && (
-                  />
+                  <div className="mt-4 flex justify-center">
-                  {getEmptyDataText(tabsValue, searchValue, userFilters)}
+                    <Pagination
-                </div>
+                      current={currentPage}
-              ) : (
+                      end={getTabTotalPages()}
-                <>
+                      label="pagination"
-                  <ResumeListItems resumes={getTabResumes()} />
+                      start={1}
-                  {getTabTotalPages() > 1 && (
+                      onSelect={(page) => setCurrentPage(page)}
-                    <div className="mt-4 flex justify-center">
+                    />
-                      <Pagination
+                  </div>
-                        current={currentPage}
+                )}
-                        end={getTabTotalPages()}
+              </>
-                        label="pagination"
+            )}
-                        start={1}
-                        onSelect={(page) => setCurrentPage(page)}
-                      />
-                    </div>
-                  )}
-                </>
-              )}
-            </div>
           </div>
         </div>
       </main>