From 54deff18687cc4975f9dda1b9ea02b8be5ec2526 Mon Sep 17 00:00:00 2001
From: Jaime Aranaz <jaranaz@inno-it.es>
Date: Wed, 16 Aug 2023 16:07:11 +0200
Subject: [PATCH] fix(Checkbox): set color for filling and rounded border

---
 .../Components/Checkbox/Checkbox.swift          |  11 ++++++-----
 .../CheckboxTests/testUnselectedCheckbox.1.png  | Bin 779 -> 766 bytes
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/Sources/MisticaSwiftUI/Components/Checkbox/Checkbox.swift b/Sources/MisticaSwiftUI/Components/Checkbox/Checkbox.swift
index e357d9b4..d291586e 100644
--- a/Sources/MisticaSwiftUI/Components/Checkbox/Checkbox.swift
+++ b/Sources/MisticaSwiftUI/Components/Checkbox/Checkbox.swift
@@ -19,7 +19,8 @@ public struct Checkbox: View {
 
     public var body: some View {
         Rectangle()
-            .border(radiusStyle: .checkbox, borderColor: controlColor, lineWidth: lineWidth)
+            .fill(fillColor)
+            .border(radiusStyle: .checkbox, borderColor: borderColor, lineWidth: 1)
             .frame(width: 24, height: 24)
             .overlay(checkImage)
             .animation(.misticaTimingCurve, value: isSelected)
@@ -28,12 +29,12 @@ public struct Checkbox: View {
             }
     }
 
-    var controlColor: Color {
-        isSelected ? .controlActivated : .control
+    var fillColor: Color {
+        isSelected ? .controlActivated : .background
     }
 
-    var lineWidth: CGFloat {
-        isSelected ? 24 : 1
+    var borderColor: Color {
+        isSelected ? .controlActivated : .control
     }
 
     var imageScale: CGFloat {
diff --git a/Tests/MisticaSwiftUITests/UI/__Snapshots__/CheckboxTests/testUnselectedCheckbox.1.png b/Tests/MisticaSwiftUITests/UI/__Snapshots__/CheckboxTests/testUnselectedCheckbox.1.png
index 0cc45fb9ff2a2da95585a15db18627ad08115a1a..633f10cec7eb2e426692e6b8e53c8043c3e1623c 100644
GIT binary patch
delta 543
zcmeBX`^P%rRlUZ-ki-uR3=Cg9T^vIy7~fnqT-D+r;u83Mzq_EJkco-GD}P=K<L}pn
zp4q)vp!NE2_S|>B<zDaVtDI*W&t}+p_KSS_?QI+1|F-B$p7Z>tuzLB~xy9e#y8<Od
zPH*aYz3bR|f1ad|JN7gl`MJhThhfUqB8?9=+pn`d{c9=G;km8;1?!alDb9^E_B4Ll
zUZf!)+?dg%B4A;~aYZeIg~^@0@fJ(3!-E$J0kanfa>&TCWC@!vH7;&;$Zc{{C@2wF
z;d_CDrH_wkE2mWBfs2j{%p8@8)?Cr>kAb0{;s5{rUAZ@aA@Cg(0&i~_cD17Tzi3&L
z_K$^O7s4lO>h`;L(5z_I^PiUIKY#qibwtH%vO~N?7?WaP&;llBT>&9gRxN`oBysDB
zR&DQayl_hH<7<YM^>H`8*T!@jRr<s&$i01PTjA>&dmJivtjmt+oqsY}a5DdeO*%?{
zomthz^e^mM_g*PAZta}?HA_CJ%0E1}X6DRfR==EOzxU0LS263EBLR#;`4gX+GTApg
R-V6)`22WQ%mvv4FO#lbV-8ujO

delta 556
zcmV+{0@MBe1&aod>VGVzIi=|U008(&L_t(|0qmBs4a6W21nJCFgdDjB8nVlv0|zd{
z(Y@9$(BB7^koe?10dio3bfevYaGs|f$6@=vvpyJziP(g1INP>a&e@h_v3Z^?rS#WJ
zGip}NysoPYiP(gioKlL#7%hbG=rv7~iACko0O`lm;nevR6n_Uu(W*cBz8fUf2-1}x
z^$##!En?CDNm2mvfW)K$lF0$&0f|Wiq^lf29*~#-=;lcSpc^=96Mz8Zozm-A5=4SX
z&`+NxRE&yIF)G%3HILkngK!WI!a*aQ(t8#T!a+C)2laYm<bE84gK!WI8tIhYvv3d&
z!a+Ew*Bc}ESAPe+00030|Jx5%-2eaq07*naRPC8T4uc>NMY$QC$hh(hTnLZB4P1B(
zRy3cs)RI{$Nt?GY;*dyq{&||{_I)q&JQwF2$BBWMh>Zh~9Si`S(qeTFfB<B-j+GXx
zMuJEX3A**OgchU4XfayMdi{>7;UFA@gK*F&-&pJ5Ab%W$gK&`LhEX*fgoAJp4jSbf
zYaJYfgK!WIvfMDL_T`{uS#EtAqc(NrI%3lP&9CbUpibIi(g1}J0Mr2zlLlzpHUM>i
zVvHX^A~j7DkY13y_Z3b{kOnEGqKPX-_lx&ZP{cw^Dxl*y+Jy>=mikYUlF9YD`{zS@
u)Lyk`sU_wOMrW~cIp;b!PcVMM5t|3ox24%rmxW;f0000<MNUMnLSTYxxBU<R