Skip to content

Commit 62480de

Browse files
committed
♻️ Simplify component formatting in ReceiptFormActions and BaseSelect for improved readability
1 parent 34ca694 commit 62480de

File tree

2 files changed

+21
-30
lines changed

2 files changed

+21
-30
lines changed

frontend/spa/src/components/ui/shadecn/BaseSelect.tsx

Lines changed: 20 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -15,27 +15,26 @@ const SelectTrigger = React.forwardRef<
1515
React.ElementRef<typeof SelectPrimitive.Trigger>,
1616
React.ComponentPropsWithoutRef<typeof SelectPrimitive.Trigger> & { error?: boolean }
1717
>(({ className, children, error, ...props }, ref) => (
18-
<SelectPrimitive.Trigger
19-
ref={ref}
20-
className={cn(
21-
'flex w-full text-gray-800 text-sm items-center justify-between whitespace-nowrap',
22-
'border px-3 h-10 rounded-xl outline-none bg-white transition-colors',
23-
'placeholder:text-muted-foreground disabled:cursor-not-allowed disabled:opacity-50 [&>span]:line-clamp-1',
24-
'focus-visible:outline-none',
25-
error
26-
? 'border-red-500 focus-visible:border-red-500 data-[state=open]:border-red-500'
27-
: 'border-[#d1d5db] focus-visible:border-[var(--purple-500)] data-[state=open]:border-[var(--purple-500)]',
28-
className
29-
)}
30-
{...props}
31-
>
32-
{children}
33-
<SelectPrimitive.Icon asChild>
34-
<ChevronDownIcon className="h-4 w-4 text-[#666]" />
35-
</SelectPrimitive.Icon>
36-
</SelectPrimitive.Trigger>
37-
)
38-
);
18+
<SelectPrimitive.Trigger
19+
ref={ref}
20+
className={cn(
21+
'flex w-full text-gray-800 text-sm items-center justify-between whitespace-nowrap',
22+
'border px-3 h-10 rounded-xl outline-none bg-white transition-colors',
23+
'placeholder:text-muted-foreground disabled:cursor-not-allowed disabled:opacity-50 [&>span]:line-clamp-1',
24+
'focus-visible:outline-none',
25+
error
26+
? 'border-red-500 focus-visible:border-red-500 data-[state=open]:border-red-500'
27+
: 'border-[#d1d5db] focus-visible:border-[var(--purple-500)] data-[state=open]:border-[var(--purple-500)]',
28+
className
29+
)}
30+
{...props}
31+
>
32+
{children}
33+
<SelectPrimitive.Icon asChild>
34+
<ChevronDownIcon className="h-4 w-4 text-[#666]" />
35+
</SelectPrimitive.Icon>
36+
</SelectPrimitive.Trigger>
37+
));
3938
SelectTrigger.displayName = SelectPrimitive.Trigger.displayName;
4039

4140
const SelectScrollUpButton = React.forwardRef<

frontend/spa/src/components/views/ReceiptUpload/components/ReceiptFormActions.tsx

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -12,15 +12,7 @@ interface ReceiptFormActionsProps {
1212
onEdit?: () => void;
1313
}
1414

15-
export function ReceiptFormActions({
16-
canSaveDraft,
17-
onSubmit,
18-
onSaveDraft,
19-
onCancel,
20-
saveDisabled = false,
21-
readOnly,
22-
onEdit,
23-
}: ReceiptFormActionsProps) {
15+
export function ReceiptFormActions({ canSaveDraft, onSubmit, onSaveDraft, onCancel, saveDisabled = false, readOnly, onEdit }: ReceiptFormActionsProps) {
2416
const { t } = useTranslation();
2517

2618
if (readOnly) {

0 commit comments

Comments
 (0)