ui fixes (#456)

code100x · Oct 5, 2024 · 414e55f · 414e55f
1 parent 141eb02
commit 414e55f
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 12 deletions.
diff --git a/src/components/Jobcard.tsx b/src/components/Jobcard.tsx
@@ -18,7 +18,7 @@ export default function JobCard({
       key={job.id}
       href={`/jobs/${job.id}`}
       className={cn(
-        'min-h-[200px] overflow-y-scroll sm:text-sm text-xs text-slate-500 dark:text-slate-400 font-medium flex flex-col border p-6 bg-slate-100 gap-4 dark:bg-slate-900 hover:bg-slate-200 dark:hover:bg-slate-800 rounded-lg ',
+        'min-h-[200px] sm:text-sm text-xs text-slate-500 dark:text-slate-400 font-medium flex flex-col border p-6 bg-slate-100 gap-4 dark:bg-slate-900 hover:bg-slate-200 dark:hover:bg-slate-800 rounded-lg ',
         className
       )}
     >
@@ -36,7 +36,7 @@ export default function JobCard({
               </div>
             ) : (
               <Image
-                className="size-full object-cover "
+                className="size-full object-contain "
                 src={job.companyLogo || ''}
                 width={'500'}
                 height={'500'}
@@ -60,13 +60,13 @@ export default function JobCard({
           {_.startCase(job.type)}
         </div>
         <span className="flex items-center gap-0.5">
-          {job.minExperience && <Briefcase size={12} />}
+          <Briefcase size={12} />
           {job.minExperience && job.maxExperience
             ? `${job.minExperience}-${job.maxExperience} Yrs`
             : 'Not disclosed'}
         </span>
         <span className="flex items-center gap-0.5">
-          {job.minSalary && <Icon icon="currency" size={12} />}
+          <Icon icon="currency" size={12} />
           {job.minSalary && job.maxSalary
             ? `${formatSalary(job.minSalary)}-${formatSalary(job.maxSalary)}`
             : 'Not disclosed'}

diff --git a/src/components/job-card.tsx b/src/components/job-card.tsx
@@ -3,6 +3,7 @@ import React from 'react';
 import Image from 'next/image';
 import { MapPin } from 'lucide-react';
 import Link from 'next/link';
+import _ from 'lodash';
 
 export default function JobCard({ job }: { job: JobType }) {
   return (
@@ -36,7 +37,7 @@ export default function JobCard({ job }: { job: JobType }) {
           <div className="flex items-center gap-2">
             <MapPin size={16} />
             <p className="text-xs font-semibold">
-              {job.address}({job.workMode})
+              {job.address} ({_.startCase(job.workMode)})
             </p>
           </div>
         </div>

diff --git a/src/components/ui/accordion.tsx b/src/components/ui/accordion.tsx
@@ -14,7 +14,7 @@ const AccordionItem = React.forwardRef<
 >(({ className, ...props }, ref) => (
   <AccordionPrimitive.Item
     ref={ref}
-    className={cn('border-b', className)}
+    className={cn('border-b last:border-b-0', className)}
     {...props}
   />
 ));

diff --git a/src/layouts/job-filters.tsx b/src/layouts/job-filters.tsx
@@ -112,7 +112,7 @@ const JobFilters = ({ searchParams }: { searchParams: JobQuerySchemaType }) => {
                                 >
                                   <FormControl>
                                     <Checkbox
-                                      className="rounded border-slate-300"
+                                      className="rounded border-slate-300 hover:border-2"
                                       checked={field.value?.includes(item)}
                                       onCheckedChange={(checked) => {
                                         checked
@@ -128,7 +128,7 @@ const JobFilters = ({ searchParams }: { searchParams: JobQuerySchemaType }) => {
                                       }}
                                     />
                                   </FormControl>
-                                  <FormLabel className="text-sm font-normal">
+                                  <FormLabel className="text-sm font-normal hover:cursor-pointer">
                                     {_.startCase(item)}
                                   </FormLabel>
                                 </FormItem>
@@ -165,7 +165,7 @@ const JobFilters = ({ searchParams }: { searchParams: JobQuerySchemaType }) => {
                                 >
                                   <FormControl>
                                     <Checkbox
-                                      className="rounded border-slate-300"
+                                      className="rounded border-slate-300 hover:border-2"
                                       checked={field.value?.includes(item)}
                                       onCheckedChange={(checked) => {
                                         checked
@@ -181,7 +181,7 @@ const JobFilters = ({ searchParams }: { searchParams: JobQuerySchemaType }) => {
                                       }}
                                     />
                                   </FormControl>
-                                  <FormLabel className="text-sm font-normal">
+                                  <FormLabel className="text-sm font-normal hover:cursor-pointer">
                                     {_.startCase(item)}
                                   </FormLabel>
                                 </FormItem>
@@ -218,7 +218,7 @@ const JobFilters = ({ searchParams }: { searchParams: JobQuerySchemaType }) => {
                                 >
                                   <FormControl>
                                     <Checkbox
-                                      className="rounded border-slate-300"
+                                      className="rounded border-slate-300 hover:border-2"
                                       checked={field.value?.includes(
                                         item.value
                                       )}
@@ -236,7 +236,7 @@ const JobFilters = ({ searchParams }: { searchParams: JobQuerySchemaType }) => {
                                       }}
                                     />
                                   </FormControl>
-                                  <FormLabel className="text-sm font-normal">
+                                  <FormLabel className="text-sm font-normal hover:cursor-pointer">
                                     {item.label}
                                   </FormLabel>
                                 </FormItem>