From 51c3dca960776bf5c66454b9317b09ac71e6f752 Mon Sep 17 00:00:00 2001 From: Yevhen Martynenko Date: Wed, 12 Feb 2025 17:15:53 +0200 Subject: [PATCH] *refactor migrate test from enzyme to testng-library --- ...ome_minimal_Components_Tabs_With_Icons.png | Bin 6668 -> 6709 bytes packages/ui/__stories__/Modal.stories.tsx | 12 +++---- packages/ui/__stories__/Overlay.stories.tsx | 4 +-- packages/ui/__stories__/Tabs.stories.tsx | 3 ++ packages/ui/src/Dialog.tsx | 30 ++++++++++-------- packages/ui/src/Tabs/Tab.module.scss | 4 --- 6 files changed, 26 insertions(+), 27 deletions(-) diff --git a/packages/ui/.loki/reference/chrome_minimal_Components_Tabs_With_Icons.png b/packages/ui/.loki/reference/chrome_minimal_Components_Tabs_With_Icons.png index 3420428f88154028401ebaaab6ef571e718c09bd..58ef7f11f8e5091a7b3499cd8717cc84c77a6b0b 100644 GIT binary patch literal 6709 zcmcIpXH-*Lo5q3#MXm)@q+At{qM*{GtAKz3LJLS&K&k;D^iWi;qSCIk&^rW@KxhF% z6%>#nNJ&B!lo}xP-ZDG*t@$}?)|xf@N3wIy+3h`Vd7dX9bhOo2SvXmkn3z~K)K&DD zn2!2^KJ>&f@EI@EtpvUfA@tN9G8J@iQNfR+h=&>mCqVN%Ve_7e>GFt%$^!%MB-*f# zx3(0%px>f)h;Ug7Pdihu5My!(9op!d1qugyP~o?JT2`=@tG zdGrVq2j9(WdSYj|^TPf22)28V5U3;I?q^Nhck|Cd(b19Fa3ymCy~nZPNyKX!j%F0Y z0_~XIrMww84~)$ui!Rv%DRlosiyixI@3eu^Nau>tC}SAE>{%Hl`QWeA*etui^T^D6HA!5|?Bd zy2^x;3sUpw$+6Tk3LdwmEM!_g#o)%Dx}eJfY8}TYZ-=%VB?vY>NLh$=1N@VwgdtC_)lJ0q>jn-#`0-WLdKD={&+-qsBvRTUa# zY;DLx-ZxnLhJeWAqV8KY6(O);@kvoBi=L)VBlw;lsRprw3}^Cn+TKZ^r@~Y*nK4kr zUZD@ozYgl!n0hapN;gSc@S=4Q+BJPDh`HddR^mMq(GJ4nniMtCto03{t9Oj`Z-g3r zi`DU%*}*Ig9q+kug_>%rn5bWd78 z)&~;nM6Z8hUY#U7Wg^+TM6{FiXb>djnV0m=GW?&%_PNxej~DGEGu_+5+PBilOf2G> z`PhtsMk>@-Z$&zp-0<&FqCxo`BAL>o+h49}R{{Tc58sRJUv^o@aPtZa#+YUWA&?(Rap^B_U*1(q2F~3-*gxaS=A|(It3dJrG32*ss*)*>v*=7eA&5^%@|O9t zv$_I@euNT3*a>j!*Xf-lxBR1%{d>my2E?RhyC)3x;L~lwmbKMO-U?n6B`WJF4&uVA zfejv<+XSw+j$%1sh%WO~qi4aKr0ppwH6MpKa2OaS+SfcHLs{)NIH)?qmOTRDs9vE7K113w5Ppf?)9OrdxsGzTnvuw1`mw?Gu?Cg!C4+Yug zPwA4#okntAYO_yzJvHv0^mj(M7aDaIaBCh;GdX4n|H*0cbhM`iA?L9$)G^+-zIejC zG6^HIu&W#++`oGnuW#cLo+_-debpvE0g1dEK_3|uU+Ze^Tup^i92?#!)vII0;>^9rkb>X$QL!mk>4*GR=f_#2CoB0j@mlU@U(R8CoMVdcaPGaa+v?EX6` z(Ddu?2fP&wahyw2_B%Tn6?cLoTEBc(72#G(DK~a!L&7nxZeW{m+hPh- zfFL!V_J-lhMgP&O|t^yjQ?(f@LED`Sw9DU#Ux>M8Y_mC}J^*{Rrer>#JtkP)liWZ^mPnltJCcno; zE)cUy>eM6UXJYpy#1DMVaPn~K!a7=AFWnSdN6uRe+%1>5=lD$hP3>T+Ue>x-2(%J2 zv(7F#O_cRrVU`|Sno5M}Qyh@&mlBfDP46@FSd#iQJEe%{hYa!!EM@~XRV~%_Fp8ap z({SULNbc0d_kv^5`Ru1Ar>FE!4sa7J36A$#nsPE^*Zg54{zU_uzh*(MwG60I28Bt0 zca-0C)4tmA9?a+!{k|5Lj~yrJ6!!pTYeivosg*vtHM$=_;qgCTsz)2#yd2TrZ%A5} zd@xYx6JkLf!Kx7aO0B|hI9zWz5*1l<`r}O-NUrC6csFo100*!=-`LX* zi{CBaRJ4F9Tg;f^4pKhEXfns0TLz)3_vU9gB-7H_DVBEHE`S#Tv5SNS3L9G-rB!``KX_ z5?dgZQdeW{sX4a?opqe(F_Wu1b2s3{NVSi?u`vZUQd7{15=&i>h(ESAq@`HK@9=bC zi2CGLRVKusdFPapE~s)^T$j^qwOCTDKI~k)bOE=I59^-k3?ku*&>-Zlvq@hZ+fr>a z6uG6K+&0;(B;RyvdNrPpfu}yFjarecI`%ID zaPcCE)y;WG&m}a_H1qH=5v$o_;HBIm%K_(NiY;glThw^m5?P5-2Fk zp=2VZGGhZtLL$8gv7+Hd|SM1Q{zT0#mw7eDnx zVPoj?vZmgz+1w!-Ip36l8cZtO3A@KvyF!J=LOc{cx~#$dJF&QJjdB=koGh^ ze=qa?P&=RVUC&!3tEHjCOH9xr^H&eR2fgE!8Mlq-2#keR=HcC#gp`7BHOD3F>Tf*O zR77u;j#ne(=|<|GFyEZ`7L@m@g9K1RHzU zdc=n>ZKF}pk^Ue%$1ZMe4#qVy- zInzP;W{g1xD_2PY-EiE_&Wwnu|I+N#wticSQ_igCMwm$bm9z@I9&wgo&uT%XE4)V$ z>q-}=Oax}P+IQ@UIqh*+;TREz`Z=$UE1ckvuT(`4uOqPf`~d0fzaAD7UBbL`cNePu z#p}1MmEq==1-&7mpIP;Y>-nCi$Vit$WONKPH_B%(CrhrLt@YjDY}7xAM-_?J`J`M< z#_$1uYqQ}#uZ#_^{FHplMJy;m(jj`mb6!=_rNwPxe0;i%(59KA_uSTYLay>*8_d5v=UFz=v3lJxI{@nxH35Fuv+YBait27KvIH*qdF?Es>m+ z=A?|DJk4V0+r;jJj2HdERm^VyGLQMMv%0{xR>d4Tx^YtSylA$=%NJdyS2)ZC-Pmi{SoO4S%9%d`e*1 z^CHKly(QqUCqJ~XguG|OHv6Sg-ND)u67OU(gKIEvAx0uk=WMj%q#)sE)*q{D#?~a~ zKVqvA@1AqFDt;H1c=I??j#x#spZVugV@d~XU~d7t;A!6)$|mu9t)5fSMki&(m=dT@ zUZ`qhK-_PzNe=jY?~q1!a^YmaP;D~}$q^jouln4^R)jW_OixVql{7`v-Q!PWnOIKF zU`Khwg0&M1F0Oj?GHY~;hxNPF;2dVBut7zxkF_9mc2!khQYwZAdA^l6$-<0GeBX(p zb1p|Pu$b3myHWOIsmhej3-JJe5|kzjTZ&i{%he+X2+pu+addkU_M89l3!7c?4LsuU zQm&;BgDBG=du3+y(YQle>!<4R` zRtCzq{$eCANqk= zh?JP#k}Vc>$v$$UhhW=mtUxJESmFc|v0A!6IWlmstuZh1%x+QvJs{AK)f~ei2Pq1Q!4@>R8xIlJ5y*eIq!p7&Ttls{i@6ODiqai4QcfTX^CYu4X zlanefqAO6*9NR$~xP*5YZdDVjm3pjvyhhNgT=wHYpbo?cuW#LXBebWR#ZC5(aVpLN zJwxJRCoH_mSf4OQw{~?s2;`N-^)0;leJbP|xAEk>tt>aGtG@Qf{x~g0PWZ-c`^=Vy z9gVuc$v$lJa7z2-PSWjSs#9ng7Gmy^`h?~SSG>$D{mU5`yzu2cVDH7_hBC}yb%F0` zA0Q+^kb4_Ob3I&qP}4j|rnqzr{1V8u6aJlf&yq*N$x9^^cXtP;euE0}zlC>f!#SL_ ze>CxS3i0o~n;{k8u5z1`>OA0kFBc>Ci>_(M*9~{?N?&OqCWKY)AyR+()oZ~M4SSAX z(WZWj7s1%Wv!%4K4g(NYgPtO*bjdo|g)~o|TC>qDlW=yz&X)!?$I6|O$JnEFg)b=p^$e}-L(6D6na$dQG=tEVGf(^b`+XNiAglfKhwfJ= zGE#CE(sd8*b1v&y!iRYZo{b62n%Kr3R!R{uni^Q9^_iYmJC>_oMNW{KRc`#s18Zbq z@1<^$6L#ipWGEP)PT7WQ*GPE_HD6wX%H=Na+pI7+TJPO}PUf1Um3kr7e}%1d0h!|M zLOA*DCekH%zi_o~P?2q4kMZt6PRuPVqLyaKwM5aGG!hM=P;9Yy~ z8Jo^uKuE~ZzU095{X762;u>~qkLf>`H^1bj#(hy+n)vWAecNg=hAK?{Q@3a642Vt^=AhktKmV}{w5*AFQ?&Xxoq z9KhZ>6_(KrUTSUSKH_6C4LtIK0It|~72h{DHr`WgeNn~d00a&bslYPVj|NXvvMjh{ z9|L?7=@yg@=quwvP%joMCzb(1i1)nl~0i+Vi;P7Jh$e++y@BE#xKDz z0F@Q$)E_$cAg_XZ@ifm5QyCeVe-yl*0^I^T^6yz$cRq`C&$0xNkO0((g$7$JJ6rvD z2UUvXV`Smi59(!kYLxF3sj=VoEbib-h5_T7)IQZt>ADY=XS%5t&*qmkbl+}2r|09x zx)N*6%*wLJtW3N4xsMaG)HJ|~=aF(=X}Qk9ijgOfi|EAL@K`c##NI4%>SU^?n-@aY zevwn$h@m5Up9nd){rUIphW|k%w1&?gUbG+NtfdU` zGrcl)cUUPs82F00q_w5xS(oj>Buu9oqW_~v9>|J}i%*}OV5q5}-8}t&(kK~I{C`c> zuFShxt>_tf>{#d>ugi>;n3$eFlF`_e}y^!NftUmva6dk+iq<6NrmpN(;T3f{SAHv zHj-KJ-vhCSDT_tL#obXBVXJL#Pc>N!T@-WL?cEq()4V9tyJ8rGPNlT_gshnDfVz{g zk7qXF*d%h6&+s85Y<-L2()lB+9w}v}Wx*|IWwCyuG>kFe4OH>gIz)bO*@1y+903aO z^z2c zVdr7smXu2l2{MWKHMV9@WW~J3BQHO)ZfIz@;a3PJ2@4w)B*g}7rc`=4r%vtisvZC1 zm1*`YL)o(tK-ZvxJxQ$4PA)t4>adF#>UesVS&c=Yrt77o*e3>uOo7np=3dyjf;A&v zLow1Epd(d72vk88Y2#%Sfj}_C?>zhxT#E}qiOvQ!wz5zZx7+I37J9L zo(mbRF~U@!;3R9R_kR6!h`(0sV4}~NxNl$V9VxhUSjbOGj!Rj~kK)YjGndnI!W4i6 zntM*EI^XgX+_+9}hph>-5~AFTfhg68?vX6>Hq)$ literal 6668 zcmb_hXH-*bvqnWxu^drU`cXhcN<;_5vE<2t~SzU5fNhD25W60-*#5 zC<4-J2py45=q1#^ebMiJKkoYOx_|CoE6Gmw-t*4PJ2THSlXsfxh|?$8PckqtoK}8t zPn&^((FdGs96tu$V+A|m;NuWR8==6E*UC8qzA$1Gly#4T!{@lwTLuQUV&!}Cx=$14 zhdgy7Cx+{mnU$2RRh*B>i}&47@eB*oZZZ^iejupY_^p6bqf^?*$jRDB;Lbb6fZG;` znuKKerYlN{@@me93$z&%6o#IiywiJq zPYwYc7#ON8bZ>e5Ib|rVJbs`uID}6maFI~0S=f?i-#}ku$5*pGOMx4*vc`U-Bh9Al2jayG z@o(!37T=2BxGpTO<$PZ!{wpq>Pf&FEMb`Y4fV8^p(pmEW%UH_sruMw-5E@tDqv8T< zej(}JfvoV7Xc;f~JqE@?@7Y(B3z(bgZ#*2s_=YHXg?EhEiYz)ZZ1PTlC7zY(m9oJW zMd)fDdIXyCysU%DE>7;swfCK~?GC=3c5yG}(-odtyC-m|pmk@cPHpCo*JrqspLwJ= zs_3uKP}%8@L9FN4a+lmV_?6Gl4h(Y^VO-I!rgwc{k_qaBTY9A9iI#3GJo8 ztd%EZg`G3C*=ZX=Vw8>nRpXMy2r;PIm6U9#_4X57IxbsHD1c;k8{sG#)Vz7&zzdDq zui{3}--M`V`@#4j9!y8KIo_KVL`H;VJ@a&aAFM&HMo_6^(SuEy+6M1qj(k~}?Lp30 za<|i_<=AHjkgGNiyE?vG)pEYoyJuBYeDA*fe#+A>Ij+je?uG$X4chd#Q-2R$JnO!! zrC?D|SFv01t;g|)_;3^?Mg(hS%CBsWmipm8$B4sC2|N21AZ$IoirSmYKDhZ)ql~I< zeHUIo0(89Y&3bLA3|43x7?NGLjw#Rbw#|{o$C{tO5iezS?aHrL(iG?uE70_I0VzvV z9MEEl`B^F@|JvGDk?fx8WaTjg%B9*)xf4i^jEhPq$(2uXAQ9)o34E@RCZ_DMv=vYI zKT+uqv1zITzZxst4;(Y8btTGm{;fEwW82EJOg7@iXEsWkn(VK@IJPLwpDZ_%uP5e- zgl|D*Y}qJb?Fr!@UEaWg&(Kz8$YhDGZ!vM{UdJG>@(1qz3^2BFvd#8oa(W51H1i|1 zlYsLderZ?XH8-d!#$?g-S=#{L za43zH_dG;vO3X-euwkch*vS37=LZIBh&D zM(n3%4vF=9|E)j71j7ofEw9<7y!<}y1Nk|5(PQ(h9{O>+P=1y5g0^PhX9CBCC7Q0M z8x`U$Yusg~t*zeMHJJV3@Ng2o1&)}R{=#>(LQb!xk~q$W@%5e@93$se5l^pmg-v8P z?Mk~??jC4soi?~QikUWOwHGh397&{NwK~~gI#t`It9}n|XpQ(w6jCToa+CJN4QD@FC2@;f_#!8x5ls~;`6 z(Ebv1HrQZtrhMhC{Q9t$FY`K=oShnnTlEtwDkZu-?j~81ZI43qQAWfUwhy5Wtv>Xm zkL~1iU%xwYMs`~-`c5KCpM`NTCIyh>z_MCr(R7kdNbfX70|H=IzBL%pa{E)o|9MMZd; zMqTxpKL)~TwhigQ`qWp|s)$`ys`>oj=GK|^%?2G%awBamrv^FRQ|$MWzfl~oOD+j6 zjyX+8QP<{#_I2ltg4w%EN;tXZSiypm{$(ccC-2+yg3O8a9VyM!!@xt0A3%IR2#bu= z`*M)Sg37hWU{Yz+BLwpL^M22MKiE_CI%dPT=>c6bKE^|7usC2!;?DH5Z9;y=p^)xy z(k7Yw%TTdSgn}e4#ZFN3>bZl4`lP?k05DB2Czq8Vd%vSB$_j7gDTrS z2v;z($Wo>>|U_wivvm$PhUlSMFz0cDt=pL~+NFm=MXYaaQr2I*q8;AfRsF z{N00GfHdw34`TxNA6{S-DT$bm3BrPF`TwqnB9$Nf7Sf7XvKk_y#ll66acAT8V$Vs96H zpI9P)8fVtrSu9wis&3T}a!!mO&>^*rg$ihmJd<`gMB>S_IZi z*RbqM$K!{i1YXULt$LKzB$dunn?4mE`Lc{3(>31n(-3+*9%QXsS?;bqP`@)*=oGPcb@2(r-Jx@Ca`NORT!5udmO(Y1P&#ck1CarobMn ziG&||Z9j2<54|AWs*`*&8yO|U>GH?%aM|`P=`*&>pZ1nvJa*`J?IuR3WP6l@_a(XV|mqG%- zd+>uW#N&VBst0APY*T&aNY5l4uu1}qG9)^`1JIHoTA!SDtDv%R z6*H9XGttOuN=m?`=LLb1J#jd?K?40NZpzv?ENNi`u^RRAhQ&^+_E8v5%dP@oM$Z`R41R=?-x7x)&Vj~-z5$8=RzLqCl z*cHsrS{0(=_$c3uTs1;}AU-7;1#+Wuf08QY27iAM5Eg>tj2RwhkXXUf87?P-Zd~j0 z5MByxmfA}|Q97^t!$GaSKtSiiPHii^YY>21&hn|h15CNvG*^KUc9w$^T34?pUc++X zLR9h>Se}&IejpUV3d*e*EU%6>amLH=l^py@{|)Sv znSP~=`pL^slU!qBQ?X5JlVkLrB6CH3tK|OV1Qb8T7#thDcVpQj16!4x!fQt94>Fc8 zpF3VDYi=T##GoV{HabrP9&a8cCy0-iTU|>&x6jZI#u^=HKH+gA{=xw$xsA= zw>2{a=>l=L*Vk4_ACBS4&n!)t(DHC`Z@V-C8bf0v2~v0nG*_!fnH*mmQAw#}gw z^*WqTm?ILFQrKboXiyd`Of`cVdU*9~0PYo-s0m##~jr~7KI9)cj*rTm^ zA3p^)?-u?LwHYrm^_r+fk`r7cxLm20ZzykdRRAc$}dm zX2AmB+chjY!|_iyoE8xmB`PRz8FUnSg)bCOCTL;GW6-;-R=eoEI4;iA$dMf0gDx6u z`XEnJl5ujCyrqu`E30ky`NcVXO@kMyzgBrXNx<;r&x~@*t&V`uAH>D;Jr8$y=y$ld z6|zP~qnCF=c(qaV_F{)qJ!Bj$OS5fdZd=EhW?lxhsTH&4W%yby%yrx#6?kBEKE6-{ z6e;b8{_$H<-71eWrpO;t17wc=dTt!cDiaK_lHYmUP0;X#@_1X*mPqE}a zCoW_pF;mC6!{u{HS6I7Vy5A^T>Yy9X_?5ao)>wNhJ$;5=-G}OG)SS*jl_(m9R5|$% zev;(!ngIlbxgyDP24Y$oqZQV9K*vAmncHldNabPUxg~F->f+K)8ew$hDgDM0t%lrx zdCyg3^wfwpNzKbS0}VCr;-#-{%koa#<3fhSzyh({51 zt65^{Z+@+-HXkCauiYf%9!Cl(&km;eei_-CjJT^bKb$?mY(rf}@Q~CrUc4wRljsB# zajSoaHizeLdKB+R@_SENkF|Z{FtmpcbNI*-4y7KrCFrp#Vu(>)Jb)g@;E+65=n>BS72 zxv^sSO!He|nP0g$rU$p0j5aO9Uf&tk?1{`R+6a*l>2oy+A>>&T>zjt;d2P5pI@KRT zja#KvR%Oii!?_-<-iNBXy6#epHD*J?FYzm91W7}5W&PF22;}}1<$`R-T*|ft>4?UM zZ_Q`7HG^r=2;|i8qj7a_yc1-;zZ^+|1a*AAq9HgjC*5+!OxV9FL$9{qsH5upG}u#e z8U_Kz1pOsl$(IoJy&vUW-0U-_ku>{-U)HD@Kdk*bP4nPu*Qg(An{>;g(o#JZ_0|@4 zcXXv?WDcOgX{Yel>)>ItI625Yx1oUsz=c84Xlg6*kELk!j4hJV9oeNUVr+a!n%p5Y z-KycC30*2TSD^)`XXFn2?>*+&%Sp#*5DqOZfShyH#XPex!>9;!}Aq&rz3y6$OPVm33Ms-BCU z8DmAbPWO815sJ0zdk7L+u{La=txMMmPz?#UvMeq8BYg0`+m9Vr0*UCeE#x`{#O*kO*Tp!2&iUU*kpDif0L zNN2s-i5!ykXdlnQUY~rNgV)bWT?p~e{ONt3oNP99mDu3;q3^y1L9}FtM!K{-a>2NTA1YB$R|C;UnzU^WizJWPj8pmw2TK`RbQuH8;K_1tf z`frJfKM&?SAi&N4EiaHz##}w*VESDY6V1S2(@{7U@aMsc&7g9`tHu8jB79tx`1{fS zb-MWb>c!X92ZR7r%Wr>e{yt$axcqN}{=@SB%c%1AqyI0$CctRYevVOY9$q(S{YN9+ z%aMFDpew^09SN90$^}Gwo#syGk#vB1_)1w1)g71ieu^Hi8W!^q{BysJW!$I^$PSCE zB+7!?sL>q(P4{9bd82QX_3B!g>kK3adq*$-*u$`FRbEW=zdWp;UY+5BYrHi`(d8D2 z%eIXUKA8wtJzHdtEBg=~TS#K*E3?J=(dlnfd-C;lO6n#>|K+%xlS4K2jg2ZB-bL;2 zx-!%p1Z`Vq)|GjFjD;q-*!{|~ISb1Z?6RFI-@tu2zqa82;sWPNq*3vrsCS7Zu}X(W z1i5NTBB7-mwaWA8R#W{(E!;jP@@xa79{ zc+ebbKWDLB=CzIr`_y)LBpR3lATn&f_uY!!Xr5nX`9zt=aM%QZkK~_hUApBxaz@3s zOxi+x6mw5)WJRWw@!7AoX|ZqaqP?i=q3q*bs(@8MEq1Mj#_S;QkwZAIWH7qc9{OYG zW5Jc9Fo5JTaWOd>mwd8@#F*Bw`#U8wMlI~nwGlz5SU+x=(yB=j!#u8X+Bmb%ty|&* z;oA>r?!{rBJOl!}_Vr)RWH`t^eBQS6yl~Abq!p}6;Ztr|bVvCBk9OQ0KI}fbe|mN%K@5uYj@Lh?(~C)S@=cUU zysAv}*(xYr9841+!jm%E;0sQG9y;lIba4An!;5;Y)^5lTKBH_&J_ImTkLg?v-(JjECGg);5l~OmcQ20c+k27Zi+3WC+)$K8ijB23F8fiwN?2znWV$g@ zh#j_|cA96yhD6HPyTdyW2X&em{S$u+_y5Eas^>eQJ@}HSS$A3`Q&YABhe*=-%*hQ& zEizuO+kg7);T>Z`L)NX@VkkOO8>bSt)4=3afQa^??89V_{}IKnGc9Q9e+l!qI?$;n zqNJ`VO$noyYougoXee{pzf>1#l53>rNm%cW`e7$6(>t~EC`
YtdtE2@;#8ELE| zb;;1!7`4wNdGJV@J)@Vf ( { children, onClick, - kind: 'danger', + color: 'secondary', }, ]} > @@ -124,7 +124,7 @@ export const WithDangerActionDisabled = () => ( onClick, disabled: true, disabledTooltip, - kind: 'danger', + color: 'secondary', }, ]} > @@ -145,7 +145,7 @@ export const WithMultipleActions = () => ( { children, onClick, - kind: 'danger', + color: 'secondary', }, ]} > @@ -172,10 +172,8 @@ export const WithHiddenModalActions = () => { id pellentesque eros pretium.

- - +
diff --git a/packages/ui/__stories__/Overlay.stories.tsx b/packages/ui/__stories__/Overlay.stories.tsx index 0e195f394..d733f336e 100644 --- a/packages/ui/__stories__/Overlay.stories.tsx +++ b/packages/ui/__stories__/Overlay.stories.tsx @@ -9,8 +9,8 @@ import { OverlayProps } from '../src/Overlay' const content = ( - - + My Account + Billing diff --git a/packages/ui/__stories__/Tabs.stories.tsx b/packages/ui/__stories__/Tabs.stories.tsx index 2b715d40a..1518877d7 100644 --- a/packages/ui/__stories__/Tabs.stories.tsx +++ b/packages/ui/__stories__/Tabs.stories.tsx @@ -115,12 +115,15 @@ export const WithIcons = () => ( + Tab 1 + Tab 2 + Tab 3 diff --git a/packages/ui/src/Dialog.tsx b/packages/ui/src/Dialog.tsx index ee882ee2d..18bb53b2f 100644 --- a/packages/ui/src/Dialog.tsx +++ b/packages/ui/src/Dialog.tsx @@ -50,28 +50,30 @@ export const Dialog: FC = ({ actionOnConfirm, ...rest }) => { - const actions: ModalProps['actions'] = useMemo( - () => + const actions = useMemo(() => { + const props: ModalProps['actions'] = actionChildren && actionOnConfirm ? [ { autoFocus: !actionDangerous, - kind: actionDangerous ? 'danger' : 'primary', - ...(actionDisabled && - actionDisabledTooltip ? { - disabled: actionDisabled, - disabledTooltip: actionDisabledTooltip, - }:{ - disabled: undefined, - disabledTooltip: undefined - }), + color: actionDangerous ? 'secondary' : 'primary', + ...(actionDisabled && actionDisabledTooltip + ? { + disabled: actionDisabled, + disabledTooltip: actionDisabledTooltip, + } + : { + disabled: undefined, + disabledTooltip: undefined, + }), onClick: actionOnConfirm, children: actionChildren, }, ] - : [], - [actionDangerous, actionDisabled, actionDisabledTooltip, actionOnConfirm, actionChildren], - ) + : [] + + return props + }, [actionDangerous, actionDisabled, actionDisabledTooltip, actionOnConfirm, actionChildren]) return (