From 561a4c2b9b7499e84071ea37fb41225feec507ff Mon Sep 17 00:00:00 2001 From: Mojtaba Samimi Date: Tue, 25 Jan 2022 09:31:31 -0500 Subject: [PATCH 1/3] fix issue 6089 - max group height in row --- src/components/legend/draw.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/components/legend/draw.js b/src/components/legend/draw.js index 0f5e3538fe4..7d0ff625fb5 100644 --- a/src/components/legend/draw.js +++ b/src/components/legend/draw.js @@ -714,7 +714,6 @@ function computeLegendDimensions(gd, groups, traces, legendObj) { offsetY += h; maxWidthInGroup = Math.max(maxWidthInGroup, textGap + w); }); - maxGroupHeightInRow = Math.max(maxGroupHeightInRow, offsetY); var next = maxWidthInGroup + itemGap; @@ -730,6 +729,8 @@ function computeLegendDimensions(gd, groups, traces, legendObj) { groupOffsetX = 0; groupOffsetY += maxGroupHeightInRow + traceGroupGap; maxGroupHeightInRow = offsetY; + } else { + maxGroupHeightInRow = Math.max(maxGroupHeightInRow, offsetY); } Drawing.setTranslate(this, groupOffsetX, groupOffsetY); From 08657d7429ceefb68717602722c5073489b171a3 Mon Sep 17 00:00:00 2001 From: Mojtaba Samimi Date: Tue, 25 Jan 2022 09:48:07 -0500 Subject: [PATCH 2/3] image test for issue 6089 --- .../baselines/z-legend-row-group-height.png | Bin 0 -> 22304 bytes .../mocks/z-legend-row-group-height.json | 32 ++++++++++++++++++ 2 files changed, 32 insertions(+) create mode 100644 test/image/baselines/z-legend-row-group-height.png create mode 100644 test/image/mocks/z-legend-row-group-height.json diff --git a/test/image/baselines/z-legend-row-group-height.png b/test/image/baselines/z-legend-row-group-height.png new file mode 100644 index 0000000000000000000000000000000000000000..56db7a5adee36704e2220cfd483c3bdfc787c208 GIT binary patch literal 22304 zcmeFZmDuoB zlqi1j-us(ZF4R}Te-EP6Y)h#O(w=I!ZeEuPB@}NS64P-=5Cd%`vOquYaY%$KZaRD) zGH^@X$A<9v9Occcx{n<>&@nI&9>op;G1sm5f8*-VbO^b5U=zcof~JSQ@CHUi0X&-E z|9eOW#@T;6>l-I9G-6=LjBGa#|2yRWUFH8ZjQ>}TA9$JZjTvv*7rKCefW_V2I07=d z_oc5%!k<2Uy0o!DY-(z%d9QL&q^GBcgM$MP1OmO?Hqv0aQn(-5pF|lD_4m}uZ)rT9DM^oXv5y>qBwX@`0qHPzJm%mR0A5cCQoYF0L zAq-6pgNK2;s6?Ccxlnx6R8&yP+5If}=!U!d9A>I_si5Sux0|C>!q`-z^-Dwb$$Owa zTqwLK!i$Tg!=f=><6=sAIo~&l*r2(lyR7w&p}D!N(W75N1+AMccDMLlZ>^c`bT-^I z^7ZDP4-6!7aB%Ptnx^%Hjf*Vc#xP7}3#m6Y6YR{yYe&c_b{Z=(N{I-i*-Uvqp)`VR z@y!RE^;nNL+20rE{iuR9;u~YTm)i3d=>D`ib8n<(>J;huIxOHU>?DEpJ>C4si(+gL z)`h=Wh6WUix;P=Z7wSuI=UzpgL-^Q$VRhLgM#wyXm6fpGTQ5AfQ2OCwzAuq-+XkbM zOoCca9zQqOTR^}l;F%AB;<#Q|7#OEjwm5-}ho2O~)X)(#Gx0L4!FWCv&jjZ(i1lq% z?d#;!!D@#GX*i}^M_d~uk63n798V=`C%QVe$b>vi0%q%*zO`>{{&?m7e%GCjM9*$8d7&X6y@;3_35sPNzq4V?DTnW=csqL?JdK*1QG^EXk&a4FT9gSow zSiw_mfeL=3&klY;^wmA2xow)VN>mfho1A3U-J$7zMl}=GC-q;a^n@Zh@IjrPza?`-^i>QwXlE)SS4_a1Wu1|;U0E4;cd zueHppW&W5)o={+~ye;B_i6WOGVYU(}(sb@fgVq%+aj#D}nV<0A2>zzP?8M^sZO*Q7 zk_;W18ASNVPuC4Ou9%vqSWz7k_ZX4#HNo4TY_U`O)0XR}y;T~Pn#7?oGw1o(A)}EN zE{%091eMs+d0w>+0WQ30bRyuSURPnCJ51bh@jI>|o5%1fyhYt?h6a`bapC|%ql;KxT~|ItDT!DylmFt12zar;3L&QJW+vUL=9OFfz<9R-1Vn;R?o<8pyhDI8eU^)cNGvRCA39G0(hGz*}ziRxWkd557+4C_n7Skwv2uFJfLQL?u zy71FU1z^aXNJYVPgUM|XXsUNp|Oo`F+(NtMB%e! zn@*53o|dbFM}DmP8GI|i?N)#q+7ttB*xmVu;(0DFm>BF?!7B%6*B}o z2(C{~<>JTr^1)Inv0f>*W^ch`KNRaQk+=k(?~}7n>7tB6+-IPV;%@k8f$kK_H><47 zOe67ybDHUoAGNgN?rdLk0&B`OdJlh+cm~nGR(!DgO3No52h4ysdht}wBT53VBVXiC zuG+H;m@x?a1t zKi^Ll^Boh7jC#>^B&s(%Lqr@m5ANqRPGRnOU#+>(U7Qe*4p$l&hgv{ug2az zA#z6sY96YzQvKy>g5*iLXBWat3?_h(A{jn_1miJt9Sk%|zKA z%pq2P)je(pmbzXzaZuBdX@Vy_jF(Etf2WJa_xmQSyGv_EoOg^(w9UTz8DJt}ex*-J zw{1Z1X6q&QuGf2x7x0~7Z=tk)OmuV9MfOG{ItYIDtjoGTMjdcm{xB+6r#L|n_$yY! zwHej~sTTR=5d%1V>N`Nhywu`A%^qF-B(j8q+ME5Qd}5i3y;BA-LoJd_7rFcJv70u@ zVa<++ovYAjVA|abXpA9~r9NE1#K$;%up)&1b%;hQ9=Sn+?@yYx6d2OVTD7C>090D0gCA*Z{b@JbX z-2LDU{m*15ejmF8MUS)&4AlHV6Iw#uC)I3!#9z_mZiTz%Gq{8Lz~}>|cyNa6zu+a> zi#Bt~l_zfUA=#Vz)?s~l`ihf0>dq4AWF+P(-x`b@8AWX-e32+Ub<8FD$X4q}`eeR_6KSl-G>d(&g7^mr;v3rLfl`(Cd{GEm!pO(C zyLARXXlRJ?E0Mm>TM!KcCT@ZihV;pktorux1hx>9ak2(GEj5 z4&^b+ko17Ghf4V{)Iw5^qdj>Oym*Pllr`Df|1A>Y)#1{jWfwoy&KRRDKW@9Df?vN=`OC}~$X#M$KnE85$Vaua1P=7=sn28h|d_5ft>he%ttP_%r@I*e$}EcVa-aDnww z{5^A#vwE3#M6W(aD}{Ma44$NUZQv?#SO>4d$RgVP>eg89s4`}(+8*|3Y2F2}6cbXv41VyxLEy9QgOPtgNijdKvEQ zAz=WG+HH1I<4=EsB)s4@4XJDiMA~lo z;Wsx^9&m^B4%9)-KJ#&02KJ<^qLeL$)8g# zZ41tf%c_(ih+CU$3qAE(6P$hYRc*ZF1;LnzU0z&Vs1$tkn>vo;FPyp_e1&VCH|CW! z+mENud26I!i)>aMNd%qQ;5@28=IaaM;#};$+XzI?Ji^n7$HVyHYtwITPp<|C|EiGf zK$wF!AHW-=viJ=tMx{s)Y+5E{N!aK`dU_`OT=xgqy&sRoueA>hD+~$3qwqB!FbB9j zGwDtcgwpicfD{avuBWevDqf(hx!N5G!lyYB$_(S#P z{sygiU%6I0h1RFN^!@PVTMtMM2EThe>z7+TW$v)n*>@h^wTu4ibJwt}j6Jl_IN}s% zvxdpP@$F2^85UP~Yh2S%-VFV*?~&%@50#4?{`jD1FDYc;zh_r}}f5@`C`ub(nxWTMN|AC9LMr#c!==4wAhwHfOwnYn=* zfUvl@68*WimsalQSRZ{l?p(ZKn;RGyF~9ED$v8e~9Q;6zxvtej&tru{z++(*pLJp6 zRo%GP6$!?@hr2jCsRc?I00|jw#IW-u^SbP{^wETdIFlTq^L=B2Vn!p7H@aZ|0-Sn|8r`IV?3$^> zD(Q)DZHl2B=d-T8uYMIDp_{Rwj()5jy+NW;kyLMj%CPCd0 z;hE}(&WO8K%SNy#N59h{td;RzN=5l5Yk#OMd7H7ExWc!gWtRve-!l&V9+(&&-jC9qgBEIUUFTaCE zEW&c+$s4?O6FsL{2^*|+yC0~9;J*vS;)Jl|q*GiJq zypo$=Yb-}XF)UwBq zmk-F2AUnv*3yCAApofo=YJxG-C=#!!8WQ{T8xo=-9Pl{Um{YpwD#Vk%H0^^$fsuXu zJAY;92ZNd39U99KkO`Ney@Z*OLANf3_j*%8U+7LYSAE`4Y4!<)YiHn%kM|A3nqfPx z7N8TX>tg6Kn|6zJX!A3{b0!XR^GLIzfU56?r0Od?cFs|i4ceiSX}@dbX4b@a(Yw0x z5B7qIJIZv}d6OTAMf*lyd*$PC0Ly#Z9chiC2s#sG)ySaYPIQ`wbVWw4Z12jt_GdE< zDbT*FE5jlIY-%#kwlvncZ|chS+#jKo*vh&fzqYaSR+K!0)1?H4DjE$${p=t@UGI8_ z<;amVNIE5Y_IQ*GZdP8qwkmm~qitmAwzTxKgpUKwEA#x7Bs}?xx5pZU!F@UrDxnKM z43cvAezCl1(Ryi69scfQCtSfDxzN(y-;SFeNscK7*WPvTVqg@bf>tN#3h4k?!#tV{ zAy_H36SKVD;wYO^gEp9{w%E`v7rF&Z-7|DN?Y!2mevstsDE&t`m~f(O5Hr25uYG;* zrVYD}C;w?dnCa?bA(HK|i=?s6F0Ne6Ry`}p8b6deX#Z70$7=<{-+QrBIiwX|)9hjnL1+K`sZ`?=-X6SYt z0}0}5;tjHYP1oT-b(7D#zr6JiI$rp5W?mz`OY!izYl`{nEE;;-GC{Bn@ctKPmcvk`Hp5Mf6r)uMV5pwjnl) z&n;o2*{m%|563^T$82wP9|S(}m57+{`le)|b<>al#Cj?FIRE!9sHDN2Sd*BW5$a$QNqb22ujF*XaPbY_5wqW zUIEJ28vFLT_pdJUo>KblNdK&}C(sF1_^VaAUrR0h=sXf#AxrRO#2jHQ+M``Ad`lmK z$OBER%FK7&@jAwtbWYMY3;Ea)B=ONm!e{;Y>2Uf*Cno+L?n$NDt0)E;XkeM)$-+M{ zl=%`rGoogD(0hk>$DPz?bErWl8Z1?_(l^x&LvlbWnLE(u_s;g!nS%qFg?=}*9rB+y z403bfG`akhrsWWU3yfm=c~FFn^Y;`KPU9KgVI893v&7#q_NHqQL7MDK|DElnVrxnQ z?mfEA-Wwm%Q5afI*Q0``=FP0~qI)-;A~K za;nN&%%2+}Q_RBK?~(Jgw6tbdUQDiFRLHY4(IT{jDu4kBNr-MZ0^0`w_PfIrpID%l zRH;JdGn8KOs8iN7)EcU|IRw({~?`zWO+&O>R>wGClQ~mvwn6Mznzn&UE?f;S_9J>s@=o%)54G(4UozKVDcyYhxB2#+!SixlVUeBv z-%0KdV7#_oNs0g<&324OSo7cD0lZYmHIbPZ5}N-5k_P7m~c>XksG>9 zsMFqe!H>tjbv{|X#Arx-S%+oU?uXQ&oy5P-gt5}t_oGKUnEo!|U-ranPs6uUWJq&T z-&(GFz5ud%>!H-Ji8S%>2A~w5mD=;cHg%=>jNa7>kAjwTvY24&e)s~ze`^7*L+{Go z=Ug4JY?O3*>-c9~Gs~%88mcWz7a=tiB40f78@FWN_2DrWP%ho2+7ywU@cp`NrvMX5<5y^LXa)_cIM=jRr6zqnC7= zJq*|cee$0iH;ZK6U(D$2i1S=)Rc)UavM2rt;*x577S9-mh`iHY_l)|Lsl}rS0OObQ z-BLQn@BY;|F-*bkFd`8;bt0d2@5x2!PrY zV`~aElm&n&8ZadEsf!inV9Oo3<<8`7$$Y0-E=<*z(bD+{NmWpS{8k+XE=C*doA69H z{QJ3(#0?76SM@<42%bMD%MsE=lb87@Z?*sHv_xpctJ%04utt-A zY(z5H=iLI%1<_R>X04*cC$v5!5ZQm>_vgy}-kr1G&$4`fZ$*#-tb?bvbxhP?be>cC z&f6eb*}Y|HWpm1 z*AY%T(SX5XndwlN5ogFcFdA^#n{e-|8&-fZ_ zCUp?x2QBY7jahNSc0Umxn4|b(e{bR_nifEs-b0R)Hba6z&a+1!ajBL=pG<^kg?BaN z6-1<^@ku=es=T-VQ3TI@(nZZo$+N6LQp-)~Y?72skX=`cb??%4OQ5UWB+eAQ@W;`Q zel~eh)gq8ezlDJ#$+-!d1Cj&jk>wCQGlazw2IMU!F9bORqTkV5Q`7Ds>vzJ(`F_WS zNUMqvuV9AR*rI49>mn#n5jMH^k4}oiGXB7n>LfHDZ24w?Qw)%c&E6jKrtW=!6D-gy zEF}VW_<}aboe9&Vc3nKf74MtF`QvcUWhRSaDK6*_VGGnccn58RggiB?Jnx#k^Rc^I zd)e=(TWe+)NT~jcv8u$@AdaA^30Tn>K8`#oO3@yhnG<2Y7WNH|AsSwGQUam@V118jQGJqgAn zdHH~Q&wD(OqDY`VALt!pcLfxTLbhv};BMQe?8;%`9U>J1rFGkc!+7Do+TT0SkiAse zwoN-lN3O*-$e!RQ4=^sL80!7P=>hS4Ej{RHoAW|{r4f3RX>MF~!-HA}PshxJ`n9D)aCGvLc-AL2Dx2ssP%? zSAE0_uM^y?I&(Kf+!@Nt%ezQiUOj|D9)&nO@GAIK#SqP^(66os_Ft(lcr(vx@aUO*l9n;^R z$-^D2CN2O*aqoOiW$zHF$7M~~INd~Qt0Dt!{~}HYP&6S6=X&@fHvIvg-JT0Te5UYg z4aR6*XLCah$D@gO?0)h=oJ&B2u{~J+pp^$~KEwALN6R92&>tCRM@FCCtvdL*{rOzd z(D1#sdPr&%40%pyFF3*UX9*u#?-+hMGQ!GUTdk@Dkt;WYB-+eMe|9L8#(!SFz35`G zZ=#yCrWiR$=q-CO?da(HmAj-RrkTu3VkX?tu9n0;`M(jD%5SpsjhKQm`n*4{x}kdQ zC4^qFkNjPWf-n`hUrq2EUs1|XZFXDBpDz1mKSJf9h>kn+5>FrJvr~%jiX17|v`JsgiGsIJZay+E_$Td($E`zTk2IyCb`Os3c{W+>EYP0K2hyZLg$yDdMLX7}=( zbx}{tBa{6H`vBrItMP|qMw&i4v{FyDZ`ZijKT`42W{tz*Zo zG@~BVX1kUbOBV@jDfR&lO)S+@0%Z1`G~q2nS>f1sL`g-e1@8Z0l>ENR344=)HB|Y_I%WrM1*F(Z0FlbBj`q+)* zScCS>6#AWgm#11+5yTFqejL?_^3K7?xCPl#dpjJPWc{xwyXc#XDTFZd+s7Y*na&>Tz6g9U!$;HBcER- zV#{yeei#&5C`qnD^Xv#9Ez+z=g93>J>Jjvc9U(Cxz1~EERdIz4aI3lIRYZnk4nn2~ zoDTYu8mOZ{%C81sY;q_*2?@!m_YC2r%;zVmlBK>z?IjTt^dYSorSa@w+D{Xd?|N;@ zql2g)u=`AaK3AAA$ruXKkCy+ACphMm75F$}@U8x+EUe#GjMyIiGW>S`QBN2 z^cxL{-jBs7`$BX{fbZ(GV-^d+hqb^r#BI-+`DSlzG#co>K(w?hD^#?cU~+nXGch_U zo(WQqVypQL@I1E5dHLbqA8ocWm7_uxJ@fRx8?cUcT}q}-ctjyY#J@4I@aJ@Ycv zTY7b>f!;2f=7)yLObf(t3DavaIX#^p7!K)6f*O&k;siZYDT3xi&*^v1&?U zBfo$Vi?T2-4pzCuzYNEC zf8=}w%GihAGj$WRTCqNOgq27zE_+*XOToz6{f1+HMg@I*`S$PMDyQjtM#iRPDdm&p zLK+he{Fh79C)4us^4R0!p0@-&Q?tTCmkRb5+Bva{EV4=UUP7xz)k@a2=f@bvZx2R| z+($;^Yqo84cFCFqj?K#;vl!>Lb#Bch3Tk{NDXCZK4?}Zb^n%zG7$h;wU+V+k@VdHl z>V6R!;qzsT8M6jWY@_J!6Y|Sz+1nzbN(_oR9>>XEU;oRKO}cK7o~A@c8TyFy?@>As zExRZ-Dz17l3OD7z(SAKLO@}h)%wMap+M%>NrrzNX)9bLJ6HQ-fDCqHTH{!fHOM3Ad znb41u7Dd;jTk2;x%E9Cr%d<^p6Ad*?@|D35f25YR~wOy>FRO{}oMrH?4azLh@G%CR zAf8I;I&0}na(7ue7W~+lXkurir}vr;mmved$vpE2!R7+*F{Y+quU0FojrH?p(||GX zGw;6@+(y1GkFCnf6X%BrUX%?ko;k9^^8j#6s|5CE_?H;RVfle_$NW@`E#ms!(=q$L zCg(T{C4dfjEBo-XMSxsOxpB-e8vci-AN<d1{Qp_H=Pubaun}k+a4Ua!KyG4?r)^kB|6Q{-fQi7%Rwm z{0;#J|7@A{7|gKLv{rNUYoZ^k1^51b)Da|T<=nxPNf8rwB5Bn#`tND4_${Zo=&xgCoV|>e`}hxU zHsn6>qR&F-TB+l!>B`0(2di!u&8xn$_0@fkB6&&YQYp0 zOBJ(01v)+FQRXl>a`xzC+My7}c}^K?QU|E{aFCa^j<%wy18`8vJ71CrV%m*N)W^yvVHnTmG0BH4xCdiCd`e{5+za0~%f z``ocvj4f;+jKSdbr<&sUiTBYS&Ho(KbSa=t=^5q-1IMBbdY2Te3Gt+^$=6E_65Y*y zgGu<|QinW}(zgaMmBs*bSX~X2J2}U)gRhKTlC2;v8pHseax(Zt+mphI|5|pJmFHQq z{3FK3eu#_Nr?NekJOBFFrvk``8~3qxiI#W@sie7qqmJqUACR+NyNgQ_syCSN!NF-J z|5T}kff;%6y8r}RF_3{ajlUKepEbijUbz4Kewv|8hBJWvL~JHz*2NQ>vkbd#TAxTX zxG1=D;~oaa4>a76-y!=aoUU1AFP{t@O`9PAhx>)%A-u^?LLLDrbVVB1w7KswOM}0# zl`0(I~yUA zGJw&~Q*?DL{wW=s-PG_zZUqvk5f%(pK6)RY5Ft6?ps^H4b$V+B6t6iApJ~zcj}?^q z`4Ctmq_uUZfIvtReIVy&x8zDXi*wO+LIX+Pl5E6SZkj ztO;-d)YOZ9{~Ut&60k}=!rkE$DIlrz?Q97qK+|c_o4G#Q$j%@GSZy4;1;`u2-{KI8 zgfuzkV6&v|KDUVt{)UhTNI9Tt3;%n*(+%!s&&wwD?!dx2lX`gKy2ifbogcWhVK9pG zCWDADzGCUKgvNS8`&?(semH3MBBd3LW!e;=0TY9qZ z#PWmqHZ^s~^+#OaQvAnvp2aY*>{kYhZ1Mppn=-HDl#h@*dub3DkE4&n`uBmfnqt*ws-#6=zq~8rf)t)PBO$d=tt~TeX1SkT_pp4d#CZjb!u}bq|7_~dR$%6rJ906{ey*x{D zx{~|M2bts*5wYRyuk?1BtK(fW(ds?`YE=J+D0V9{p0D?f>r8PG4m~~YR6obp6(Fv@ zd%_4!Vn*P2vF6Bkg=c3j?xo`!|6T>;zSm~0ah@|L)}3!ls+8enKaR#r-i2XZ8{>|Y zB5VgO_V?mnff@kWGvld%g(?yfm}ljS}dbQBCYrUIsid_Rc(Y92P+z6laSyP>ap zCwq9h&5ec{R_NH`GFx2#v?c=-z~`I%mpBM~cQ?Y?g#KNa_up=Cn%8Nby$cCR-%du) zj+fwj#f}(xKGQn~meG-+&%jD}3l)iOi;C5Q>Dp(5ccrZNj*z9ll<=Y(>6c#T(rTHH zIQH=SvDLzG^w>+s7>DrqNn!wM{LL~yt-6m2_7@j$`@k;fXT`kZu(r4B%I?Ig=D$-* z4@LzOpLFpKl)V{^%RCjgP^@_}MbAf?FsKkTW(zH#lgtRczSFfU+>lHQg$By>zY@JN z{TUXg;m4QheckS+*7Sc`LoZny0Jy|)ngLv~fLbD}%^T+1 zOVn~d9C=w-*w75}YoSzb8`S6PC9tlTH64FHcds-cGlqkboS2I?-khE-0iq&EO;Lly zbcE8T5N~mrB)-74f0Y?nUf9L^Uu{i-~g{DA)*`f6RYakRoA2cJ&5^p}f{^~O{lf1$CTrH;F0 zrLKh(-wQ_@+P3Q!`vt*X)5eld2fC$C{!_76!6tK{%9(gt(D4hsy$P&(1&{+8|L<|{ z`N;SgS*hM8h`nb2y=!VzYhG})V|kFDDecD7lF90LLzcikqeDe#d)mh44JhbqK{O1v znU9npZ36ej%5sTF=*%RAf&B9az&p3=06OW)|KBB}jVaM+#`8Zf+;g+jyd}~uAfvsn z{}KKwK%Ik8q2Mn}d->r&{x=|hF}@A$WmES_?gZ+Q|H+s+mkgExJn1G>vECBu%x|Y&-sGT}BySjq3mJLGEOu^ms!6;g`*%YC ztsMOe(w9==wkh_Bbb>eOGVhpT31O4~uMKdQspxhv)qKj*^JXes4s0-R+w$aE`c&K# zxC*HsfkWcb?^+Cv!l67>0Pe$F1TzO@JOVCU@;wFApQ?Kah^*eaPJc>zrgw7O{rP^q zIhWqQOO{B-Vd5XShlS);1uB^*LmTb}S~yw905XqHCQW-)da^Huey;V+%N}5t8BVhv zho*f|mzI_-fT83249Y(jYIj@p0*L9w2EbXkx=t~FWt7ByzB%ja_sjm(P%3@lmiPy| zTSeQd6F=08w#N>If&Vm~qX?w0Oa?~0ulj3dMr)xg;BnSuRhBGtE#w8d`zKU>{rTct zf&ZJUOt5|W$Eo3WVc;T)>~jZh7@YUrr42vNQ6tRN`?k>%uRvPeqpQg-bFwvm&^5V2RPb2+UX?v-*duj}ABz=3EjDKt3Yi>?pE zd^Enw1_gTC3BP?l{=5>N*{C>d9|qC*hnV3_5JW`+%J6e$$6_E zq3!Jq^85Hw3FSR@X(m6O0Xttr09!xQ3kxRi8Lp5$xIZ-*;Ji)vl~va4Kh5!?%;izt zC2<0}t;74S!^?Jl@A*dY{|SAHO2suu^JqqeZuDP2$--|1^n`MY{eboPbXOAY8@Qef z#KYoDG+bu2Ow^q^m`;Dt7fTANHP63Gl&6hy?rP8t3nW+Sag;ghCpo+bc^W*V@@Jvv zbE$8}cc(nJd%%@o)%B@3bb$FAt(P?3^mX4Z@t90y8PqJMEA=z6IkSy;!!FR!)SplX zQ(4(Vy}BNq ztDufNvtt5oxBZma?L@EVdHgNvGyL?PS1w|ldW?jY2aktM5zvx+SsPzHX0X;ntSi3i z`}lM`uP_BWe{sTfZS0kPu5FWzD0$@XxlcFba-!N`7o$I@r&nZU!w1MPGoF18n7`F2 z{P1l8PgxVE#wtu~x=-+l4|>$)rVOGoXq>r1PJa6K^i=vYdX>HOvAa+}z?{BvMEJqo z56mjjncENTARNF&=jafd()?cMi={3Wh19iecz<+6VcZ|*mrfn%d6EFPlLf5*RhVoE zhk<>*#~e=p^FW2FS{VCjvO<1G=V{*i9oL}&nLljo?6~w^&K>X*7fQxc0!sFF*YAlW zEIq?`{!v=Xg*!f+t>MduYZAl&_aAmDO;ctl^*u=x_oy$KRoK}K^Y@j25zW~2Nl5m) zcUFSEcbC4`#h}c+=8dB=j?P*gsG*o3&%vjUa}q$;F+}u3*k4cHUdp$9A0sC}za(5( z*RjauA)i-hNw}-Sns@Z&yDK`2wn2xvXQhmuf|ZGE_^pa=os$$ao+bYKkhp`Kg^xb{ z{7NJi9P+5hvwb01?UO|~|LNg50Y^(XoB3w6$(+neeel8Um)j%nSappN6KpSpp4x*| z0pTD|Nb{ecrg|0gehgt7lhUb=?s$#!X7`%Cl-(`kh5jiHvZIE2F();1%39Xr)1!cm zFHo*Hzf8H!W27aQW&~7a$c{ZH)(70a-V>!0-k)7mn*6e$3(}t{PEgsVO#YBPQ79M^ zP4BFrHXcCG{(OH<(&_di?F_O9^kC4Zo&C7;yBHUT3-4`0>83)p@FFMd<+uSTLYpvN z5vX1qgHG}59$j~Yn|#6AXGV%XkSuEcQ~dkxU7pk438ANm#Q=1!oI{4tq2#C&0E7G zDP$FxL9e4J=vA=CUu(;saRaxGHtS;EOuF#WXSzJWg{V(1D|7r)XCs4jy{!nT zL20}kH;p^c4J+013X+$M7V7jz_kQ_8%!5YX9kz@+jNRAiaV#VxyuUs%T<Rs_;UPkA8m?L|^lX?l9GBF+`cKRZwxEVy!d2f>3*NwY`+37>dC;cb>c_xeH4l z*f}d%<~tPa4Jy&^I*1LbYPlIuKi%Bw^MjN?0o)VLZ!#&qztFsy? zQvKSz*QvF&L0VFh&{y;0(^N^bTV#lOU)}@>4)I$}Yg$UO2)oO78yj0Hyn3~`tjUka z3|%I&qQ1=MJ&=Q~YuZQ1OhMfZL!uh}N-ck#vcEZbJl58KWL(9wxg&X;vj&!L&~0t7 zXAI2_s`i(ZT^am%(ylL*4qH{-PD)m;5axFb0zSYn$i}nNK7AbHLcvg>F7>&(;Yn{P zGWIw`I~iM(=`;5}=<=CwzD21l6jNc*-+|t`stGs#O!UXaKUQ#_^{e520^Z;5+6KL{zPEzXTQk<*?&rw#0O>K!Kwangr`XW&(<5NnZ10}U2CjYgm*XYGj%V(H;-}>msjmNiuhX18suiitdPx1f zanfjXW0E?mSI#p-nyB9E*dqR8w%xe<@o%mrA;SvEAm_RC;n%~LC10kH4I1(Ax^8{? zolZ5GVl*qAYD9tlpJO}@9hHGpAe~35_Pk%Q>bN*c4*n_fd&g3DA1UCQv3PmMs4oe#+{B;lKFC$-_Y6`4VW_&Wj%NDOEA8%qV<*-|P{YfPjPiTIue$4*59Fgjt{}k6k|Y1d>D! z>*Fk+>r-5viidjU3idd|2?|KRBn60oVcqi@Gjiem`L17O^gbwWIi z9$1T@_u&0{27iTzni7YmC}_WHNo6?j`I0&;PEi+D%8z_`rgGBop;vQDaO8fS(_w@!zM&mKIRwL`a8Jf$_=zy z)e3&96s1x#kt&=a_gon#c&F{Ye zAAz~C%kst-MPh*+OqZLS3aZTjlX6My^$i%{Lm^;(zkbus`i-9yFVY>R!S6mdzZW#M zOhbA#yQcVLquVUHAbDo$Ws;?+RzHs1+x#NRYzp!-zMyZh`=T{`oYQ zUd5V^`@Q<=C4NsUr)(?;{6cScwh|lJ^LyanWBD8fZDzyMm&($21aqQtO}+JH`VO}8 z=3KS50dZTjG>{oa+(ggV%psGC23pA{W#^f*GtZJrbz9T0Ug@H&L(ZFLxdB7>)%xH% zU#2Tf_4TNre*;QxQ9h&tCT^Yd(*D74Om@DBP=E^oGpBs%Xq4-~+um-XX!~`8mSdCf z6Zlz~rL_b4`f8`g2K5^me%?$3+}+^N3XckL1MGw`vf#7fqD*Kn(fe@BV2Kr2|pkKMo6GDtOIh=c@?mZ3{CRAHnjBGNZVvemT6l_&RU zchulE(;)@0ax$m2fuTMOsQ5vneC5wFO@Zev*Nt@Q!$8VpOWBXmGYCUXbRv%e@6=`1 zbiDWV9)+b{MV5Ax;7M65?MeY7o)<-Q767zc&(bb3L@j`&-Ipxwa^U05SlT@UXm^G} z?g7A;WWmxdR0DdErCn1%y8$j&BoM#^`Cq;ydlHYDe+ub>?JX>BTQr1QyUJM@dQ`>Z zdq={$vWS$LSQXH;A^|A^ltisi>rbs2K9>m14mTDa8jE2hVleHLt+n#zDsJ~~1B)V-pzj7-I}RL(#` z!aA7affULl(f*^~rhG2uIitqY#iHUN7By3y{>80qA_haq;4nkX5ats}OeI z_G0-xT1z#64dTxhGyez$iHiP_jM}$&+kNaP=i40wpM;h;PuuH55LhH0h%Ii?DFW$% zOABTfjBWWhF5h(37q_}8LRqVwVQhHkg!$1Ed_p9FpzamCf3i$S zRX9vI%|Z7JS|Ik@*Z$6Y`y=rCvM^w8 zar%{I!rGSxdAqIM=nQa@Z1qZHZgR6moe!UuSyl(r_jk`xkL8CG<~AA{S_^v(1pqMX zaJ?)ZU^th*Dgmr%C0PEwC5B3+R=9NCzUem!s!x1nOllG3lh022A|c)0sVmV!F7@>x zljct!<=ywB-`WuCNBVVk=FHxs@o0o7&aAfF%Ku#_)VuUnKGNAS=ClC#WbJ@mU7hmr zRgG}La^m+r2?w57F2JR};d} z@Rw2sKD9$y+&3&V>o-~k<4vdHsom6TegfWzmL3C+kFg%x+o0JtckmlbC~b| zPvZ|sh(9P$7(WdWPE2y)$RCbYH)$_k-;==c94Mg6s~jD+N5JY!PVlVH`8TtTXBKhX zT>3kFx){P{N;yS0RHeb(u>|t~XK^fa^XbG;I@R(cR?l@H$CS7!SYiP_*)MlVZ1SlD zE#}Dsx9&^IKT1anxtY_CJ(BOQZ;B{-`H0YBs_zjpppQ%C{u{Vioys-v9O&0>#vX41%tl58-Frka0iR}`6%!qM;US5k5N zUMqxVcL6;6R;g`n-`cXs2&-5!0520J+jh>9B8o?AeJ#X|>qn>gxx~%z7YTwmO0tS@)9I1YIxXGsz zQ{+xU1Sd(Yr=V%N&iac~Bkf(-(zCln;=_t$rPtE9Y&VzDphL@2Mchx7)8YVIGo&Ku6r*MYptr-!$QOxn*CrSb_GDbT z(SE~Lunbsv%@5REEj+m#tq(;kOIo_fBwQpGN+v$m@&@h)j&coMS)~@1P3`?w3gFCY zGidu4J!7JCOCFH8$$YzYfuSPV=zdy!*U6Wexc?-9n@W5@dSKlDNFr%BGO4;(r1fRd zJP*@v`ib9X2%`Sx(r%g<;u2^=GPjQz;yEmsWBw>?T|C_Hvy{vqMMZOOkxCDMkS`l= z`%o?oAj|Sb03G!k2JBM0CQlMFYCoq@O@kVRbOIx1vr6O5^~3nhb3V?g9U;LB;~|I7 zZ*%yRt!c8@da2z|NlW(cWwKqy<3*3`(E`&m=u{CWPr=}YCfu4RDf`bU7`^b60eQ$f zdV8?REbJhVFX2`Z;bq9F zqi>I=92i?IRsKrC#>@o2&w|gX>g-L|Yz#OP4mev#kViM{vbWDS8EnBaU!JNePXjF3 z;)co(Ihc3yB66|=o=SCN_M_6OGPQlyX?0YaP^{xzOIw^nCP)YxedEQF7tHFeZYp*5 z(qN*f=F3`Fy8;+pvaZhKYR$Tdofvwq{`is;q5FMNmEmz$O{(s8Vea>_xI=^30$V?r zZe%kznE!qdJVG)xDz6>uz71^_h}N`CeXmKvNGIGXEwIJdUFVZek6UWN{_saU8?27l z&HYUgI%E$5$SuqI%cBH^n6*`iTUlPzaY_qD3(9}4O|(Jz)!-67Zx8P&3Uwm{PH{lu}5n`{>=*@v^q95S8ga}II&vT4jz%R*g=?7cK9{Wf}nx9J$ zH0Um(>ADSVZZ-R27@LZrg0wC_mHTHQO1}K5A%i(jP@Sd-oWiZwDF(r9oeH{U`3ku^B>2h*ncrBS?Qj|Z~6mE3u<^Zi(R zOrpn&nVK?cGLN&<6Dsc}?ycm6XsGtW3_^cG)%=>%hz~dg>QeczLNt_w3Iei?eOu4e%DRwyz~I|dJk)-%W6JA|5c9a61{@+|j-7*? znHxK2BQWrEae*@7of`da9(!Tl+^a7U$BSDGOkB>MOyglXyO@~m zb)Z$H5P)9#fKer%N};N2S$UPo}N(^TH2h& z&Dx_43cI703_3w!XEd0W6gDl~#F5ugmwrFM3{idOy*Y4&fe5m`c4)xTDV4B(8le^QH5Kd|Bikf|YbpAm>J7E2v0Hi91f0a9G zWIrZ&bBGcr0@ongV5;HR4q1}wBpYsidxE!eyGCW%3P!pL0Sta2>)P1LR?T68<6iOW znNE=0u%G7F?U963;qCn1fF8jNQt?~v>k4}f%UzJ;i1&u4g#8q zIzfS&3m0VH+w3|G2KK@zv2ysMr-1?S^b}=dY-?An?Q{iljU8Cyuexn=R-Re)PSe5} z_|;+oVg?WOy^&hKgo_1I;Kt;4jr$;s^B-{U+w?r|Ihlb;SzqOy2 QIR-pTjBXf~U3QB8ANqUYHvj+t literal 0 HcmV?d00001 diff --git a/test/image/mocks/z-legend-row-group-height.json b/test/image/mocks/z-legend-row-group-height.json new file mode 100644 index 00000000000..6d4657e09e8 --- /dev/null +++ b/test/image/mocks/z-legend-row-group-height.json @@ -0,0 +1,32 @@ +{ + "data": [ + { + "legendgroup": "A", + "name": "Long label in group A", + "y": [1, 2, 3, 4] + }, + { + "legendgroup": "B", + "name": "b1", + "y": [2, 3, 4, 1] + }, + { + "legendgroup": "B", + "name": "Very long label in another group", + "y": [3, 4, 1, 2] + }, + { + "legendgroup": "B", + "name": "b3", + "y": [4, 1, 2, 3] + } + ], + "layout": { + "paper_bgcolor": "lightblue", + "height": 350, + "width": 350, + "legend": { + "orientation": "h" + } + } +} From 9bc66b9799f8a110e9fbc3cd8ea8953d9ed44897 Mon Sep 17 00:00:00 2001 From: Mojtaba Samimi Date: Tue, 25 Jan 2022 10:00:40 -0500 Subject: [PATCH 3/3] log for PR 6094 --- draftlogs/6094_fix.md | 1 + 1 file changed, 1 insertion(+) create mode 100644 draftlogs/6094_fix.md diff --git a/draftlogs/6094_fix.md b/draftlogs/6094_fix.md new file mode 100644 index 00000000000..92fcf361fc6 --- /dev/null +++ b/draftlogs/6094_fix.md @@ -0,0 +1 @@ + - Fix vertical spacing of legend items in horizontal mode [[#6094](https://github.com/plotly/plotly.js/pull/6094)]