From 4875e593706b4d5f678c7bbd5eb3276feecf596b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=89tienne=20T=C3=A9treault-Pinard?= Date: Wed, 8 Feb 2017 17:43:40 -0500 Subject: [PATCH 1/2] plot bar/histogram trace in bar layer update selection - as opposed to in the bar layer enter selection as previously. --- src/traces/bar/plot.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/traces/bar/plot.js b/src/traces/bar/plot.js index d9ee36c2265..ff69c21f01f 100644 --- a/src/traces/bar/plot.js +++ b/src/traces/bar/plot.js @@ -37,8 +37,9 @@ module.exports = function plot(gd, plotinfo, cdbar) { var bartraces = plotinfo.plot.select('.barlayer') .selectAll('g.trace.bars') - .data(cdbar) - .enter().append('g') + .data(cdbar); + + bartraces.enter().append('g') .attr('class', 'trace bars'); bartraces.append('g') From 9d9aaa70523b51f8f07461bf309ef423956618e0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=89tienne=20T=C3=A9treault-Pinard?= Date: Wed, 8 Feb 2017 17:43:53 -0500 Subject: [PATCH 2/2] add mock of bar and histogram on same subplot --- test/image/baselines/bar_and_histogram.png | Bin 0 -> 13334 bytes test/image/mocks/bar_and_histogram.json | 13 +++++++++++++ 2 files changed, 13 insertions(+) create mode 100644 test/image/baselines/bar_and_histogram.png create mode 100644 test/image/mocks/bar_and_histogram.json diff --git a/test/image/baselines/bar_and_histogram.png b/test/image/baselines/bar_and_histogram.png new file mode 100644 index 0000000000000000000000000000000000000000..d5e426b968fc634a9499fcffc96df6eb91cdbf74 GIT binary patch literal 13334 zcmeHuc{tTw+qN+>M4{P~G7lAzS%WBaVIum%>I}@B0vf(Fv~&#qET2wcZ|SwX{8TD)a5l2G}DZt-XKt=C<1Qds%-klsh{@!(e?@wBdu|NqUim50zUKUcK0}9wDd(8z6So&WeP-R7Tc{glmuKr8_B2q6LMKR0vHd0`omsSq#hLzf_HLA2(?>OWM{5%u)YHmel3U`> zEr}pUk1=o)$9}5f*3q$Kl`G$R^X4&ia0rw+g^mPp>LxH)F= z0-%5I%=0xAuV4`^s`&Ue>ojU~JaCoo7&>~AU|56x`<`2&F*wFa1RYj>X>f?71!N>> zYV@AQ<}lRmLI=iY9om3`$O~EHpTPhN>&v52$bb_Y4X`>-ZhNtD8(?{?_%~84ny=gIFNl7tu#hLr$rFiw7Oojh!DGwrW?~V(O*qCZEBfH)+0!gY*8m!{gz#*lQ-%V zHD{!+8g6xSqSCc$n|(?E>lX1M!rnTsJK)YA4UcD#=M&ZGSjM??yr=3F2)yo{566n< zh|@JfAt^dC#nVRu<}wP1!}{h*Dnk_WX%eqm@v@dRIUzyHC+vnc^4byBSa%}sNcTw9 zV{W3@saMzft6XQAP4TZ^zjmdp%q5s$1D^5Hc9Rza3*u66EGVi+!9h$XXKF4o1ob-kNz(ve9u%dyk7ni(q`PixDu$4g`t z?awgJdokN)n&$&wWr)9$*Fj%t!m?$z203HxeZv<{!Efq!iq#=bq7ee&-Nt6o8%JyM zLx;yIK*1$Nl1e#LHyL0(<$gwoydaAg9U!$5Q5q*&K!`1T^b^v2!B={+;?ir}1rOsN zHVilTUL%ZNq^1)PppM*EGda}Uj}S1sjnc^jQg`i5tJ z?AAZhq^3Y5j4D%Y)tv->DJc2sJ^esra(P01DMwrD61s}!BKmK;tPR~UAf z;MG2m`2C-?YD;MUdC}6ztE5_+m z`~WW6@Y*(7x+f6)lc5n96OBU#oYA^x3SZXcs1(E;)ECkKLg@#f!N8bwH&M?eY*Mn3 zaQ|){GiS~2M6#WK?#Ucv_2(<)?Ut7D18=)N;I`9(o7s#7(IQVXxJAc2w#UWW8J|7o zt04@~$CY9m&4aAA<7ycjhT%qq8wkvtz<3i`&?2AhdPe9>QK_4fOYVbxzaOIXL_ms= zj;xl8d3D1y(jMoea4r6rFCd2iy@+0{wzB)!Nzb`4-TwZ5%NAyqu*W&@v30=I(EJ>d~CiM z*%u+yI@tf+EPe-jRONcGPpkI5A3al3gp@_o>}ZF+l&s<16*(EQ`s$Fn!>S;Udhhs3 zBuSj;Mah?Lyk#%l=;kF%Txn)?b9^(<<=vNX-xk-WNQw=!jVqDv+d0XwO zjB@3Aznx01q8XNj@{Yu!w(&K9TV=++`cb|jGw?m%q=F87%dVB)JOD{%!QL6lVT8rg zx__^ESespVb74|WK7te?oHlVLa@o&!%Zn|SQ#fv`K-9H1g_5?XJtyn!OTAG2Aa!P> z6>nP?k*axbbMsz8sHZV!k%17SMLY~Dw7$lLonQ=DH8elbkEsY`#|$;4VkY`34}6mS z$;ZPW^HsYy!XA=On{M)Kg&Yr&)~G$lUcWux(PFXRn!Ed*n}hi)n?vkV>^WMDWvmBI zMj4#|qbGMB*NAtBN-MXhc>L4wP}DB^)h2UG{!Iae77_= zuFs)5Z2-HXrQ1 zbbp_0X^<3Xyme?m(WPg*m0o;D?bJ~US+)pZ@%`8kJ|TYj*xYP~v$|XJ$8ZV4T^@<1 zJ&|YkFLd*H&-eDuP#B$Qn)ZP6BbW-B}Q;nq{rdBxYfalk5ga2 z$VfGE#832mjTxKZkqDuj4$)Vi`QkFLV9~wU@|DA-QVX~uefh)(F8!2ND^JHs)9q^W zZ%6Yxn(|t$VyddDva5!3C}TNht@rZ3m5j`4DxVshs+r>Mp09B^y#P?23Cv_k1Q-Hs z8TQ8$`zDu0)yS3%3-{p{G0r85T^=JDm@2DW8Qe7GpMMmBi85A4jtb5U435c%z zz^0`dpw6ECaitQlQl?k>%X{*6IK2_v3!=`eYRoGw3iF;@ z&Q>>vc|D&=etQS=GA@i{mB!~^X?sM(_6NK625iw=^{8OnayHbp5+2(`yGY)U!2`Jp{ddiUM@ef}^qomGQH zF_`S~<-hs)5*VIR!qbUN8?yHlB4mk|ZLp9-l);muf?*pB9 z;2Sw*B4aG`H5NDQAq5zmDH}Dl0JbPjUCeNTmXou?9T8{*>T7_Rd*Wrr!Sq z>UQH4>m~u}QKQBPraqkPY|RlZ@`@{R;ksmih|~7_rXxCtIPocQe3AxwB$6ipWXRv2 zbI3>ljfL`aayrbs;t6x{nj()YRgCG$tgpq??8PsE=uFEjp;+hglDlwLI7V zRIMgBIQ(d{JQL&q{nZD2s?j&biE$Bq%*jntsVOUFZ#Y`Z7q3^-$je#Q<20u ziG0ejZ*(FZI+XB~++g#n@W$u~u2lD#;b#1p$I@tl{VRW+_0vasvV5jmR}Kz_;+MS#mqeSl3_wl z-%lu<22)WHGrNDjY(oZl!`4Am3)XFbW9<^pj z<;;2lvP%3a5`p8tBrp1Yh0$BokBA_70&PyH!pg8_@Ag0qy5~{|wdhx(wl;ikpWz3y zCZQu3uzaH8^R{p9T_s85ZbE+ffO`2Us%Xg!FSyC!p!jwU2E<)7q&_{zEz=+)_TZ?KleB+0te6X1+IG}M1{?*J7G>R^D(pMVk z&tYxTHS55ViYKN=3Rkn!b=%N>K`se~$kaR&X30tj0e`06`sIBS@V<%klo!jmjR9yL zJ!_f=g9lW%oA+OVl!U`C{}44+G}R7>>7`adH!Z&@@8Do!$$}NO# z_OTB)OP`%jDqjLhs(&Hixy(W}<%hbimtjcOd=_PRN>!~mnX2G)Gd8Hsb(V15cl8c% zXDLzWzA4*vt(td;=sB(%qB5zCUL%#!@HPPXVgPH#Fa&6a zng2zkO|L0$58mAe3F&@x(xWlpwbEiCvq(~eoy~^CH9^Bq+3Y|%BoAdX!{lpPm6qi( z;DX2M4UW$N7mR$Y141u+3s1&5^!IgkIBV= zE*gLp^sqz&ZSI^aN1*D^36+G&*HpwB!$?E0*-+NFX$5TFGw{UgQt5hdW#tQbQbOoLG#xc~hl6L0vt=EJwIL zMnT5&I#?~}`Aq4o0eHD$9w!G2)#HNCZubp-Nyf~Lbzy2B?ZJ$7zQt5O;2r8IyM^hm z3T{hIfffB#|1>7UaRyb%BaqNGq)1uhr;K(KB$)q*b0==Itu>mHUS5?oYq|5TMkl6Z z>&zv=#yyRK`KDp%v?sz%zc))lG(xLbIiSLb(ckrTOckcNiq3wjG+Wp_B;vQ$HeBBrU~yx=1M6mY$o%AI$Lgz4@r(3wY*4~E8+h4Lqi4m_V$N(+>a-F zt|Ygs!hOKbuB0PDP3?IO1fZlV8~y{bT|V<3w&J=7QXqX44hIhW^f3D@6YO2N&mlV% z_nX~_l(CM7f;a72^(UDLsG8z%mp>Lm+}?J&Hr%v_Aj#?nE1v+*;iGC^U#M+^d9D15|zg1uDih zyC4Swj((9phQ#Feb-0%(<_#)t#SsvM)@`2_ZbIy1aoHvcfJq8tc-3ZzR^DHL!IS+5 zbQCnq4c-Viw~>6Eh@urOt8baL2gC`tbuMavgo^g@>CHfPI)V$~mS=EGCZnD+y}ynJ zu;-sR;Hi1r{M~N81*~=5S(YAxG=%M<_uO`YhEVRkc;p|{5E5pM|G{9uxt&y4nk#?g z)dLw*0Nw{eQ8N?hu%Nentq0N2`=;;n48`-WBSHM)Mu;b(g+h=KMm@#LlXXK-9UezK zYm23;Au8FFD9ck?0u%yOMzP~K9mERvIPP0Ze};f?-oPi$nh!d{1_2+qDkRLbS|=cG zAzivWIX&MWf+r>TsCD5;acxZhN0rL=CS`SHC@F~VL`S}3OwsW7+OVN$hs@(4YKxbY zmVV}lP&!y;ib8EK>8!3eXQy+yJ*1Fj)Or3ZR6Ym}4u2JVc@8Yv+C|V;S6v7m8!hPe z9)1xnVVcGxVUnUF{r2Ky=}Irf=`r`t=$TqWCk~(SWS4P0rfy1MZ=>QTT1s#As{Oka z%5wAG0{I&uDjaEDu0I}B{S=a1?oqau8r}*{O=j3{8b4mLG-jCpWyuq+4K`x??uaxw zWEIOVNU-xSci!l59BOyS(p{Cwh)`GVp7P;j#}Z^jCz}|9JqxHz{ablI?x68b!V-_ zgSsEW9Ta&8GpYx8I`cBCBpNPPOQw5nap+kc0P!M1+A^BY^XF^V21Qb?{BDjDVmN^0 zV~YYHZ`HZ%rp0C2M8i#!pzePoEiR}=Z+Mg;fK9r=Sy;sf6Ly|?qjcsBK<%w>_0tc5 zu%6E+&V3fC9lY zb#|MPKkIis*NY5;gF?>t2|rw1#wwoDmBf?flP=dU(;>}n!Kk_GE|}JpDnkha{G7sz zFWsdoZSuR2iS632l+2C6FUf-kCeOfHj`@eX8wMz3;hUp3p&1;-EiBXFLa0-7EZQRt z9BUB6p8`z?0sC__zzhK)gTDmfH=PGzA1-v3rlT&U{7JB6&jh_uwP84ebpYeP*>&>vy z&e!(U*o#QtAKVyl5X)pb*<|l-i`^OmQRw#b=v^dXGU80I5?dchZ4%%-iImD+njmAp z^;eN7;$f@Zyq=R;n5t{;LRV=2)aKF$y8h21qG{PBNAS^ks(SnF>W0_*z#)R?3A0ix z<1`b=S|Mp&h3@!0IuR*vT}WxX$~RJOF>#~??@-8B6=@XG5O{gXP}C|FghJJJYO zK9VETZW{yK*M)WGIhaO^7QZyjiZ8R2-a{=<{TP?;1UO?Q{%{{>9WU`(YNF>*x!V5B z>D^G8YnyFk+~Ro92Hq16t(m$p3eL02MLY~05G8na0g@Ly$q~Dptdcet;I4$#da(0} zWMN&TjQ)iS7YJ`F53JRc{OF~F#i1r1qFA8o&u_1Dy;l}2AXc?H~;=O=Q{RRp4olU(dX-?`wfwI`>2P>4aU4w+z?*f5Rpsu(y%Fb%Pa z{O>N0v{0{Sc<#4S&GXf$pgys<`r*1{7Ws!k?wa~B_!7x@CZS%xa~q}?1>xm1_8vQl z(jE0HxuLLN=9Zx9h=E3s+H#QENTm@6z63ky{P2!n+X&eX&qqK^k&i9;XX$uw`Z{CG z&#U*Qq=(48d@Fvw&RAPNWIl5B{~vF7Ma_Gl9y<;xr-$w#yD6)4_lPg+H*Vcc9)6jX z7R2v;5WSIzgNkWr#G2x<*xp*QE?6U+u5@`ohM@1WNSKc4UKaN1<4VxBUKuteD3nYD zi#MJptUv^p_L0Q(aiD!qca3{1cr6kV^h%ey#<_{&_)7$SW|f8SVM9gW@8cZG(jJKPo@OBztp*O&EH%-=Vt%9*Be;Y zN|Z}DUd4NP{(Hf>@!~lvkL8JKp0yeFyuuLDTtU@ItGu`6$S#B8_psI{YLtaWyv)sB z*5-st+Vha;`$*e|jT_Y|Hy0NgeKN^`(U(g(TmW)hyWCn+`6=_&=w-=D)z!rpjKbbs zSNeEY!V;D*PWq@V)asjf@x_Ed0$0U3MumwLojW!pK+*4csKs}jD|HSo0L~vyH1}H? zFI6AyDpaU{B5Mz~r6tp;GN(N^?+5{jyDH(@jEFJzC4UnAvfuj;2_`CYoi0O6>hp)9 zRXyc*&^5Cz&lZ&Gg4X1|_|ddv|AK>(H}hlAgCY*{crJ#5Y|d%OAY8)kIs6>o#Njhv zQCr$J7No)LClZg{HQHUO?nP4WzH@yy3E-Z*ZpWn${h~!1@$IFuD}7r^wYgF?+D5y7 z3dLi)OT5A;eKpl;v#)D?QqT3tWOz;7vyNSkXyx$R1XA-R{CWKncouY(j*D_^=;*$v z0}d_EbG*h}5@IA-yqUiWUv0H?PemCJRvIBYf#422(b1rtgb?91*zj%2O=)d!`D%>A z&qOVnKQ~)GJRiey|v-p(SRTwT~H?KY;#wp|N%7_i)8q;DH-IEe&<2g23_( z%CDabO1!`JZsHvLJT62-#Ul8I(?X*+Sk)T;`?3DC6;!2O3^c!tV>l0&D6dH)uD^#P z!q3Ga@W%{m4H|zh{_n_BbK&o_{||ef>_dCywc+khCP_o^_dYboHT8~W9=7rSH_ZGo AX#fBK literal 0 HcmV?d00001 diff --git a/test/image/mocks/bar_and_histogram.json b/test/image/mocks/bar_and_histogram.json new file mode 100644 index 00000000000..c14416c6785 --- /dev/null +++ b/test/image/mocks/bar_and_histogram.json @@ -0,0 +1,13 @@ +{ + "data": [ + { "x": [1,1,1,1,1,2,2,2,3,3], "type": "histogram" }, + { "x": [1,2,3], "y":[-1,-2,-3], "type": "bar" }, + { "x": [1,2,3], "y":[-1,-2,-3], "type": "bar", "yaxis":"y2" }, + { "x": [1,1,1,1,1,2,2,2,3,3], "type": "histogram", "yaxis":"y2" } + ], + "layout": { + "yaxis": { "domain": [0,0.49] }, + "yaxis2": { "domain": [0.51, 1] }, + "showlegend": false + } +}