Merge pull request #26 from R0m1k3/1.1

Add a thin grey border around validation icons

client/src/components/modals/OrderDetailModal.tsx

@@ -394,7 +394,7 @@ export default function OrderDetailModal({
                 disabled={validateOrderMutation.isPending}
                 className="bg-green-600 hover:bg-green-700"
               >
-                <CheckCircle className="w-4 h-4 mr-2" />
+                <CheckCircle className="w-4 h-4 mr-2 border border-gray-300 rounded p-0.5" />
                 {validateOrderMutation.isPending ? 'Validation...' : 'Valider Commande'}
               </Button>
             )}
@@ -404,7 +404,7 @@ export default function OrderDetailModal({
                 onClick={handleValidateDelivery}
                 className="bg-secondary hover:bg-green-700"
               >
-                <Check className="w-4 h-4 mr-2" />
+                <Check className="w-4 h-4 mr-2 border border-gray-300 rounded p-0.5" />
                 Valider livraison
               </Button>
             )}

client/src/components/modals/ReconciliationModal.tsx

@@ -255,7 +255,7 @@ export default function ReconciliationModal({
                 disabled={!canValidate || updateDeliveryMutation.isPending}
                 className="bg-green-600 hover:bg-green-700"
               >
-                <Check className="w-4 h-4 mr-2" />
+                <Check className="w-4 h-4 mr-2 border border-gray-300 rounded p-0.5" />
                 {updateDeliveryMutation.isPending ? "Validation..." : "Valider le rapprochement"}
               </Button>
             </div>

client/src/components/modals/ValidateDeliveryModal.tsx

@@ -108,7 +108,7 @@ export default function ValidateDeliveryModal({
       <DialogContent className="sm:max-w-md" aria-describedby="validate-delivery-modal-description">
         <DialogHeader>
           <DialogTitle className="flex items-center space-x-2">
-            <Check className="w-5 h-5 text-green-600" />
+            <Check className="w-5 h-5 text-green-600 border border-gray-300 rounded p-0.5" />
             <span>Valider la livraison</span>
           </DialogTitle>
           <p id="validate-delivery-modal-description" className="text-sm text-gray-600 mt-1">
@@ -182,7 +182,7 @@ export default function ValidateDeliveryModal({
                 disabled={validateDeliveryMutation.isPending}
                 className="bg-green-600 hover:bg-green-700"
               >
-                <Check className="w-4 h-4 mr-2" />
+                <Check className="w-4 h-4 mr-2 border border-gray-300 rounded p-0.5" />
                 {validateDeliveryMutation.isPending ? "Validation..." : "Valider livraison"}
               </Button>
             </div>

client/src/components/ui/checkbox.tsx

@@ -19,7 +19,7 @@ const Checkbox = forwardRef<
     <CheckboxPrimitive.Indicator
       className={cn("flex items-center justify-center text-current")}
     >
-      <Check className="h-4 w-4" />
+      <Check className="h-4 w-4 border border-gray-300 rounded p-0.5" />
     </CheckboxPrimitive.Indicator>
   </CheckboxPrimitive.Root>
 ))

client/src/components/ui/context-menu.tsx

@@ -102,7 +102,7 @@ const ContextMenuCheckboxItem = forwardRef<
   >
     <span className="absolute left-2 flex h-3.5 w-3.5 items-center justify-center">
       <ContextMenuPrimitive.ItemIndicator>
-        <Check className="h-4 w-4" />
+        <Check className="h-4 w-4 border border-gray-300 rounded p-0.5" />
       </ContextMenuPrimitive.ItemIndicator>
     </span>
     {children}

client/src/components/ui/dropdown-menu.tsx

@@ -105,7 +105,7 @@ const DropdownMenuCheckboxItem = forwardRef<
   >
     <span className="absolute left-2 flex h-3.5 w-3.5 items-center justify-center">
       <DropdownMenuPrimitive.ItemIndicator>
-        <Check className="h-4 w-4" />
+        <Check className="h-4 w-4 border border-gray-300 rounded p-0.5" />
       </DropdownMenuPrimitive.ItemIndicator>
     </span>
     {children}

client/src/pages/BLReconciliation.tsx

@@ -465,7 +465,7 @@ export default function BLReconciliation() {
                                         className="text-gray-600 hover:text-green-600 transition-colors duration-200 p-1 hover:bg-green-50 rounded"
                                         title="Valider le rapprochement"
                                       >
-                                        <Check className="w-4 h-4" />
+                                        <Check className="w-4 h-4 border border-gray-300 rounded p-0.5" />
                                       </button>
                                     )}
                                     {permissions.canDelete('reconciliation') && (

client/src/pages/Deliveries.tsx

@@ -449,7 +449,7 @@ export default function Deliveries() {
                                 className="text-green-600 hover:text-green-700"
                                 disabled={validateMutation.isPending}
                               >
-                                <Check className="w-4 h-4" />
+                                <Check className="w-4 h-4 border border-gray-300 rounded p-0.5" />
                               </Button>
                             )}
                             {canDelete && (

client/src/pages/Tasks.tsx

@@ -445,7 +445,7 @@ export default function Tasks() {
                                         {task.title}
                                       </h5>
                                       <Badge variant="secondary" className="flex items-center gap-1">
-                                        <CheckCircle className="w-3 h-3" />
+                                        <CheckCircle className="w-3 h-3 border border-gray-300 rounded p-0.5" />
                                         Terminée
                                       </Badge>
                                     </div>