From ce43e0f7dd885d71153b8a2b574311bf1169d952 Mon Sep 17 00:00:00 2001 From: Orgad Shaneh Date: Wed, 3 Sep 2014 22:53:44 +0300 Subject: [PATCH] ChangeScreen: Highlight selected outdated patch set Gives a visual hint when the currently viewed patch set is not the latest one. An example use-case is when change A depends on some previous patch set of change B. Clicking change B in the Related Changes tab opens the dependent patch set, but usually the latest PS is what interests the user. The color used for this indication is the same as the marker color for not current patch sets in Related Changes. Change-Id: If7d1e673172a44ba1c5ef9aea332d8fc54b1125c --- ...er-review-ui-change-screen-not-current.png | Bin 0 -> 26936 bytes Documentation/user-review-ui.txt | 4 ++++ .../gerrit/client/change/ChangeScreen.java | 19 ++++++++++++++++-- .../gerrit/client/change/ChangeScreen.ui.xml | 4 ++++ 4 files changed, 25 insertions(+), 2 deletions(-) create mode 100644 Documentation/images/user-review-ui-change-screen-not-current.png diff --git a/Documentation/images/user-review-ui-change-screen-not-current.png b/Documentation/images/user-review-ui-change-screen-not-current.png new file mode 100644 index 0000000000000000000000000000000000000000..9a87c674efd30a744ef535382483cdb7e882db81 GIT binary patch literal 26936 zcmV)mK%T#eP)VGd000McNliru-v|l?H7w>+!$AN5AOJ~3 zK~#9!?45UWM$K~!U^!5Ej=v7I<};>0O$ocK+gI8I`} zj+5ANBL;&37Z61VB!nvJNJ4cbt;%Y5wY^Ng<(&7AnN=4E3|(zaWb&-Ww=S5aaY#{OkS> zGy{Z|6-pMrpRl6XBX1@-{R;>?mX*iv3UNrKQducvk;U%`q?BaWd=^=}hr~e(LeDAK zu`2Dla{!r@OBTOJ5P%6JL3#?Yru;F&va)jdT_Fy!SeE5wk;U(o&K4pogeVmw}?Y_u(HS^i!8Fp;yr*3iG}}{ z)WnY=61`b@Wbu1ob@~ypc5;6>1JU1^mB;Vm5N1X2L5D(lLZEvfo7evShY;Xe2rEiD zo|{D$S$v>1v}T}m4Pkh*!pP!-Ki>^6LieJCKx9plEOE%DwWC-hj*Abcsw5R^Yiifd|BE4)|E&WS-fW&AS;3_ameBW2ODHw zEs=YkLFrzE%!VG`H!4>%5N?W`M{h*yKb{rHyNLe&e&X>swr!s^c(Zut$b)U$L?RJ{ z5cvK6tbBi4ve&)Y^B}T#?`W%=1m$K4Y?e4=@j-J4D9yyps|CXkZnkIg`v$rPVJA?! znRVQ}TN|4YD5XfHQlwHT0)YU9g@sx2@D4%KG;G_ZqoadFB7xuUClCl^W&5sA>~y|( zaNjr4;%zA1la)ypzrE?FT$C_LjQT8YuoB^B&j`vAhb%roD;!V~VaGwqY$4wJ3DN;4 zkuDa?I&9uuQA>%G62maa%gf6ew(lTuNG6kXc6MUhHd1C!H2dx7bq!E>+FnNV9|9wA zf7{G5r9CQr7 z7t_$t&wm}IP}0G6930zypPZE_K-2O04Ww=3SPt@C-Mj_5H-LeIa#C3D@sTfYOdK-Z zM}!cWd8<$;lvL-InGk52KuP%lE0Pyz+F*x}w{eXSXyQD*pg?l|bpmmo?n{eTrS-)H zw-J|IqtrX3<}$^MVsK2=MdK>c|C8$79^1irmYH+s&$Ty{e->H1)hU0jqmt{oS>o^x zK`C{XEM#jN-UIoYA%q!z1L7cwbUO5<6tRqL3ZMiSrcO>krzjxM-!fy&*xXmWj1;zhoT2?Gw%)`P?dqK678@V9W%FAB%;WCE2`YmP9i1X*# z;O}M)yDYNEB8#`J+jR+Ug6ICNnQxy+vGsi7;MFOu4s+AY0=~Q`M7QF&cUG8;s^~;y^EtukGQ-hmxF#fIr{Q+?ho@aD4%H zm7Uv@--@As)$edM@0f~n=tPQtY4q}q>-Zrb?oG1ch{l8W zg{jceLwyy4_CAA#raK7jAZtGUdA_&p09(#?AciWanzD@B@3@zb-7}GDQ^#=Bdlz{S z1_s?kTUPV6>z>_f4q#zM6uO7zkY?TP_l=e-@KWp%$q2<kuO7jpjBJ>^K&HATbV9ow!+6P>WqEQT=aXoi_XeKp&K~Np%$oAc= zc=$A<{^TAe)D=*I{hme5-VtAcFbHT-VjZXX*B@Gp|4L4-SZ`a-I%c4s zw%uD;{q0}#sxy??pZE|9=ZvDKKjNH}c}DSJ`{AkEo+C3WqRs@?5T8yol-LpbJIH{T<5W5gLY} zp$V`pCrf`mh)@E}F!6W|F6-!YEG)-Kdqv-Y%~j|oey@&lQrNbGtwc7nEsLv0biiiZ z=mM@^QpB>b#8qJU6s?E**?IU3f0J9qKTPs6B1gcRr9yZVJ*Oh9UDZX;jA|CvdKngw zT$ME^70Hysa=CirFGEs}Z)oKwN8vwS#6+IP4Te2V5R3634Nxn_PYIUbSr#_KNXOq6dw*}@}dLM+xo z7m^emBK;OEClX8?ZcrJE@{27Iet5{kU7x7qD<1IlMA&#J&i5aJ;!hWFO^KgLg-KF9 z5q^9khtF0BioIO^`2~Q(?zLUKe9Gdm)bU$Vfm+^(0$54~+$F;9sp2p?a zs85~Yz)Kt1vHdWo{Z$N|T){AOq#eWV-pCV=Np87mGULk~l0B_7?mS9<%)vtY-QLJ_ zgRmZ>{X{EIz0k&Edv$_KDw(XGw?SvF={(iSidAhqx?AI;3(A?Q4~B*$(F3PhyRVP! zu8(Ub2eD)i9fw=klv0G2*D%S`#S720u%lBD9$mx@nnZMT^3s;mv?t-SvvTpdHf?)5 z*mf*NYhIXT)gH8t4mNK+O-BO0czrQpOY(LDgqrS>8tCBU_U-JE$LSp1K&Y~cJBFGh zY#`FYsl&V2)j7bhxQ3(Ls~lS$UD&vZZuW2QC%?Xw{NfOXb8#3?L*Qb@cC!AdSJ--_ zm;Ss4CRPBN!j81kdE`0%_vk@R+MI!HcS!vvJB!dUWIqF^kMhFH z`|wU)!pQncs{Phg68S8$NP8H|%6!Zkm&fcPCv9MQfPjOqFUfb;#ksHEWO%-Ta8){w z);urAez1!sgVb2$69D*U&&4G`ch2GcvjUQh_DUrM?J(p{G9{kCwhp0gg=H z|FVX!0xVwN5T)0j%jntpTt7}v-yfGDwoT9JKK}ktieVui1^FUb~6=WouU{kMRm-6z5l{z7SNP>L_qddDKN@G;e zfgq({lon<>^zS@43B}XpJ(?99nV;UG5C~0FQl31=hmxeA3JYFxR$H2(cp1M|&q~+iYI$1N< zcswSi?qVnV@R}YBT}OX|bUcZlTwemaXD`2cstr8pQi6-Ga-xE8{Ve8=zaLFA(3E;h zM|>Fz-IA|zr&)*2wu@mdqwCY(NUVn_{z7$bgpyy~P1VXyFg9~wRK4=Q*q~{!(hHmI+4Az4P zG$C;v37U?k351eJGV?=9TvwumhOX=BIf_ClnJK2t%o{OyUx|{L`)V0~z@Q8S=thdp zU9Yl!a}QmCVcauy2!7i^xQeWu_rW0!hLI%NzK@MR{1KarYY7$QQ0Nu7j*H#j!>$gU z_|o~5Oe{c?Ey(#x8J4SIb|pA^G{){I*oOuFSihY<lmidmPr#Z>V9*1sQ;0u}5R(h%QgXm6j^k|INqL4Z~z=2YYni_)~A#r*W zGLyM0g_Q*FDI6<8w%*qVy#NdSEzJlW*oD zTmO_7TU%&sE2Caddkds%V>qR|M&cr3hAq zm|7ggh`Mz2gAD$LHA!b@fgk^|Ewt{eVD0QFEGxgTX#0HaBV7l_N}e0XjIo_&-2z?j z9 zxMw=EKO#`BgR30k2^ZIKk(z;N=xBr91uH>9rpMSY1W1QeG=UVEd7~s!=@=djYzyfs zq^6i_DXcC{HXIq$kppco-LdTbz$B^6pjQf^NWL}kr)IcX~PyZ!<|K!VT4(2jy z(h^4EqMQ^dSK>;QX-r^boGB7%Tf`C)p=lR5M0CSI*E9OX@Dj+$!SB`4To)@IBQ6b0 zT|;Of9Sg^GaWT%O$z_qnW%s*5TSov%m;R)SR&3%mGoEnACDt;)k5+WCVsCx2gSSNytKNT4eR>Ztb3V0yNs_a$l;bcgA2QasB{&#GN~SG@M2ez)qA2`6Yx{( z2d^g4f_}z~53p^p5YPpr#OjT+>V-4>U`vc`5rgufx!iwSDW9HbAP@BM{7XIjFTl#v z^lOlJb1B#REDr4K;2+oa^UP5f0s-dEF5)k42s5k9z<aO> z;bHpBBBm}isch@yyB?R+`R}Iz9Tu%hvUPlbuL|(F^3rrMM-p@3jNzfgN5;Eu5cFwW zGrXTyy9%i&@=}q{6;jO4zQj%Q z7I1TIA1x==@uS;5LF;F}&+^ZHg8OPB#QJve(~mua_l}#m@t>zr)3}On-ntU=rc%85 zCpqx+E9_9UjKA$G{PF!aa@S=2LR(S@giPV~zQ+28zr)WSU(2h_9&$!4=B~f|EAE*; zl+gt`G8Mx;y@ls~@I!v_^hRE9O%N-sXX>4Q%@^-o$nuGWh*%WWwuKjd^q>6ClUq5M zs%7R~cQU#oPBhn`?2Yugn<7;-j+&|0^VzwN^3~1Id%4d0!)H)=APIIOuWgQD))z9Wt`x_OW;|C4*K+XW4q@`=zRxJLkUVcQCyoe|vWcEr&y!DY z=FpZybQLu)>CQjmqjM@5n(xK6;>f-OtoX@KS-q=?6a4~zaUJ76{CV!1JCt#SZ5-Ra zjfcLupR${Z=xN%|!EMLr4%aj8jz8x9xwSNuc(GFn^ypz$J^T}%+jxvK{t=9reGQ{J zenETveKbs4#_c0rTTyMI6vNTX}7q946*2@QkEKuKx-2}*^!R0A^=+=&LlV}4C;d>MF96TcF#%93@CC&%wgt|TexHKD9Qs#jy?5Lp8v(u ztn--YjSunv{`eKP-hMl`|G^DR(35EcT+`4K5!~iOZ0K|e)|FCUUVu1%_Nq)IKmkhN z2}U^l>bp@f7fHa^c-8)HgR~&R2I0aSiiHHs%t;Y1D~JCuO=8$~c{>_yKmvmHCHin@6pvxSzUr$`iy zX5ptl&kdu?sWs12+CGhrO^@)zYa4l?;FJ8_M;oXwF=^YricPzQ_Q^I{C)M!vTYb#- zrr392fWQ4&4{NGQxovt0A9DrCZ*^jQw4M)74>7OECDoVY<;PC&OMfx5K32hh)+pSL z2!HdVE`A*?AZNLcLL|C~u(Hk1t=E_Hi)CfRTl)FyA9V4&fZT#Y76jc3jdKMklWT9N zqiI^42dg>8_E%c?pS>m# z5#p8#xFoMbC=fP>UiV-O^)RY{(?{<~UG;#1%Vd>(@#)h6HSvZ?8Fm|dt}xy`MuGi-Q>`BUP2yE)FKP1#Lu zpBQ3ko%dWs=l$TMktVtxBvhu+80(`Un!qr<$V3b4zWxADrIt|k=WBUsl8Ze2YrgTV zC;54&#e<(7%7`h~@L!$%oayaHltiKZH0`rza@QmyZe(|fBQFh=t~Oc)EfTv^N+Dk8WdIC#7u2)95r;WBGI;=m%n-+&*uFJv;X~{ z`R34W`gT9d*S_%+0t0uF^O>m>bRJ;k-+qbxWB-)99{Mg1m!6@!aW!B2lT~aB=QDKt zQtDGpy!@}9W`p-*Ecy3u^1tIOoW>RW^Lxu8RKr?BNN8c)_k0`zxo-Q!}sHxTTi%sAG;p;K8JJWGv)RhxW(+Gx9ub^KfQ{* zrMFWyQ%9xR={WT&M-JY~v@V=0Xs;`JxK&(7#DYM0Ds zS-C(PXyp0lj}z5n)Xe}p*39cq{0D10Ysj1bAfG9eFmQ%lkN%2Hw=8DXf-#hd#HH^g z1fG&oh6nuQy8WE#JR4*E(Ny zlN?kUbt8+oaY6wLYkVXug;E+Gt)JGW4pyw_W8<*_`bJl>d|Dw@LXmV8wleT|48nc| zDUbm##il{2dWU)qDHkPl+@b}{Za6@^{Rlg^{F?RGk7A*vaam ztRJXk+HKcT?u*jXxQ}D2e#m!D-NS?TkD_e)HH<&m&Y@$wX)K+^wX>(NaNa1Yv~!wP zXldph>o~z_%cZC|Oi_Li>BcWm7Ux>AY6#86j`q@ZvYAumQ4)?sXaZ@+iFBNz`Q$Kq zqjp*;yK(wXH*=~{N4tIolb6ot6UltWm6{ytX{UAXA(~UA$b0YR=J93J4lTmhvV~`U z@CYx(hfsO#JQn3f=xf}~iXS{e>h9ZEx}X@`7=6vnG#}qZLFHmb&AWwrb2jthL)+M} zY8c+aQXZIG!muSbF`;n1g&iok3N2qNZDL&`_mHg`Wo5r?4ztH@M27aoJ1$5o11A5Ph;`SrOcYt z#PRJL*!0*76n=Cm1ryD)4g)Do3JOD*o+M3MwsY+E;Z!j|*U3h9?r$e>M-@K5ACWyv zA&c|i^=fS2+QqNhWBj+7X~k%27)qn8AfLaU<6~HkfCK~!gUnf8#q41@j0g+FfJFdF zBxci{RH&dqVV+4TU>Er)%Qb)`v92Uf@AOc9dys3#2B;28$kq7Vhi$wS9*U*JK_I2c zm`Xp>>im?JNyxF8XmxR_&!W=;LR?B50E&YwoUM~zkzjwPO?On%d!mQ$9O&hHg+cCJ zUdX&c9lx8Tqt)iQJptzad6?TK2PgXoiRWgbgMc$s>|sKtAwWktH1kr(0-hon>>qsY#EtXf%$dFxc> zFPg%*vN*Cl#X|-zx?(<2_O1W`AOJ~3K~(hD^wQUTh};j| zO8I6T&%t)u#~O4WT20}&&rvb_f`(ERG&GE7`qW7bADV*lZbn=G#nJZTbSI2+ z-HU{dVRsQf-N^GRYbpQh>CBlofw2XB41@<*_|TU*)?CZRBNMo@a0nAV`A>`*IghF# zRg}gP*e6mXx;yBJrbycTSZxPcxoQZ-pPR{b*G*!4;Q+C44-3bC54XU2}=^nKFp-f(Q3)31Z86q1ww(&XE zJhq3H&H;MeLQGx5V4;*T_X^}B)7^-K-h?`B*!8;6#@hX9z(-#Do6>W$0V$4PkkBhU2Yg%=`kphhO5@FShaGnkI%0wQ#+9y4|i)=zWLS^2!=sJ|OVi@M&(kV=^Q2 zleDjWh+jVP6#xC)Q#8!{BR({58lxv3XXvAQIb1x2IX5rj+J;=L-WWC_li(qcu7i~r zAc@2i@Zj?qZ<3QO5Qd5A_u=#C=h84V9mC_p=l5Wy^VASR$ME^^L<1C7O=ZgR>lkY$ zK%C%|uAv(~f}T?9uDP9s!$Rc8_VN1a4Lq}EFV4LmXW>29vbd~;wpE?1`ODR89yx}Z z35AT;4GfF59C&RK@%iJaoqs(I|N9C%yNdCTozJaz zPrw&TypImA$>N<0*lC~?v4iW_Ae!hYHc6!hII}BG-tcKmoHT+`x0l}iFSB`X7a_BP zI!!~_aR!<;vvcPhN(yRNeyU9Tw@S$ z6}W;x$e_B)WaEw=c3KK$IJ9@B^_QlDbQQKEu>i}Kq+B57lI*cqJrLsaMJB_-3gQY( zFS8c<(uSy&qECUVJd7(fsSgWCDe&l&dKCRyy5{lndz%2mpr|&7MYSOoSvDQbNp>BH zvHn1m*G}~FolODq76zCRa_R50SRDob>peWu+E36Gq*6AWF^5$hHbZ(GmbeNFNLdog zmf$t<7XR8|ITMk#PFHusebIXX4aj# zhI`6u7+P$RjDjqj%=B3SjwD^|H&g4%K^h_SvKidIY#iHKx6yLaJsVLfyml#5XN;rT zlOXms$N18&N``?)Bye2|%heG2W#+tev89eFT%_`%Yc))pUQ0zdy#N!zBC32Ay}}`p z(g~G~VaCUY6Y1+EdUO{%dYWiHdWNLwK^Qs+iIN@!HO!b&OHH9hDrph(6j121>ChE! zyqiQvJA1lkuw-}z)!`)Zln1S5Ci7|^V{J$vV_hU;r`g^;iG^VebD)W%oeIkdQ9q`S zZOJwc9Pj7mve0E0zmn+!S;dvt-rzNI(9>|fbTVHu+hZKpDo~^A`NrJ=G_8YYzjcTg z_qOwRT`_+##>5m7ONJ?}%jXZjB=IMY^Zoxi$?6wfBKgDkLNVwfoeyoh>E9727=ilfj1E}Lc_+|ojd^%w&KDZHU{(LRAZwCYh*1rw;0mso2X-Oa~1 z>hm$ZNMd)i(j04{$5%>Su#bH$Z8V;aFxj6uIIM0I#ggQlUBB(nEKUOsHF!HSVgNs<){xck9tS#(Vuf$se@ z9Xg2cNRID(md6hm=`J4SL-7Q?cXY<9X%;x<#U)&Dar9jTt^@V8v&@n4LIH4K%G6*@!a8yN@WYG zKgdX>B-3V;@<;bovb@keXV6M@?j&OXLlNzWv+kuc{M9QCD$mEH@gB;28o9v5^}%P) z6mu1>rD#E5>F3qSNd*aY-oaz(&U1qYJ`X-ax}lQDxR);D6BE5 zSUN11rPKP^@oERto{6wzoR=9PkgmgNqWok<3~#1$lK^zXBwnTwFw-5P&hGb0foTw` z$>nQz!6O^`cw$p8kM)9gXbv~bF6RF6ekK=Q&Kobz9`rnTtE(TFB>M&o&Ik|J=O{$- zJTci0y`2#rTQfjsbs_T?6fi5OE;QGKwcrm`bkbjI%(_I5d{}{eU*(!c5!MTOkvPL69y1Qn50{o zsdQH*nK`*kHw~PDlN?+9-~7uXN9mGf)Q>HvyjbE@=}u{bo^J@JwUQ2kVR131yqv10oGNpkuAqVgXgl!?MX4=gSJoNRKc+}9&3Q{Ep=$#MK}EEyXh5_i#zLh9>-1nmSnPW2;47oPQ0O4lLjf@2P0{+ND_?vL^7{k?o{o0-@ztRLlAn0^69I$Eow1Wh758cH)6h$TqbHrksU zS*VmkH$4Qx1(=#fZ+DdbzBpRZ+c}El$}CC+H4LA?b8+;@6o?!RFX^sSuHzydi2-c? z{$KIh4+l@vc5xC3`Xes={V7E79C4Os>n|kI&2;wCwJbUKeS@3smQ-@nSGeb0m@6>^zl?H4X;$VBHgJ@NQH7$+9(qmny#T7 znO?&p)6q@?>8SMS_SsSWfJVpCU6tZV2?_V|o^T<+O>w+2&LgV?cYkRpw~g{J%1qI6 zV1O?O``m$`7pD?=G(0&v(@Eh;Dq=R|532ORHhk*BuUwtoD8RDeWEYE87%ZBW!?zau zFya})laiq86oh<;ghF=|_%!@pFS97*b6+jt+FFAmD}B(LuBQpKC+Ko!??v0lVB>;{ zT<*U;%ssa%5`9Syyxz&DR`+8FLCul?MR9orlPx3Et2%%yV|TSA*;A$y4|}L_&Kn2a zq2qWz-}zYwjpHkLa9W6Em0Us^kc$3V5H^~S7^ZmYVw3qSovgzjh4~*2ph=L$KsQW=$w% zR5*c^c>6ILu${YNsc!b`a|H*$gEoJV+T&9^i z=Z-eM)R;BgcDj!4Va1dEjJWOd+&-#~(FG9_od@ZBc2{~+JLk8!!l`!(rTfs$917eB z(jncCS5u%%v^WSyp$rpYlu}^NWX7lGFky5tISCu7>3D-7v_L-jxgiqoMs=zJWe8M> zk86a(5KY75(;(rZgn`Ep_`GBYmO&t(VOcJgCDUuvkO(R9Dai5bsN4Y6Mn9br+)RIC zO{JNnu4#mW7uR5<6`4NlCO-7ZIb3sxg{~X8dL<3JR>O%mnfs25Suu?2IZIf&@;}(r zZPSxTQC5{67oR_f(hM9IG^L@FPSX(R2sho8DiHEwdi1nW?Ob^3wYn%`B?EbQ>Lzks z_&HXc>Ed`-6wjD2Nyo{wmNYRvUc9D2rjj7ihLn^d-BMKu1a7)6C(30i&aS{K)HxSf zCb0>Hiz}1Zz=XTL$igu-)cGu2BS1K)V|$7yF4ai1UuB|^kVtp1*;e+Dt`9QKc8T`* zFwJ~Q8fG&4wg?Bl^$eSy+Dyn_$ej~&C?8UTCXRBVH%?E=WSG5~AN=Dp3{3wR^T+xq z(9?_M;J)K@OdSIig5IX%B;rFTFV3Z`SmL&v`&HM4oPrW2OdpHqmoM;gOEXoI7czNb z9i}Gnc)QrUVkK)|Zsz2yd-&p_Ayfxj*t_l(R=#+QvU|VG!jT1(^=)SF<~2OBImp}x z?qd3gQc5-Jclr>|EZ%I=syDIR6?V#{&vsD(lZsLk+6FjuppO@n&L}1DN)R4_C{#qf zBHiK6XW%dLvb^RDJDWpPcLu1c)u1QNlRxOBYiK?*X8IW+-ePcBk!~SW9N^}7J4g3B z{9s54e>mKe34gdG4@X(IXMoCSNk9k9Rd_D1Kv~5fszgV-(gV{^3cF_o2RmPNs^O?Nk2v0vB;`R?+ zM?-l&hU4&?B(fS880Sg@mCML$=TP$084j*|mUXo^Fts+9T(5zQx6*v%6@K+(6ZTXO zd4+z|fXrCUmv?b!Dn_EOgQg~nNu%d5b3rjReY@GS`!J_nmudv(4tZ3UdNKw>ME+s| zB^6B0{W&L2EMdgRVGPL|Aa-mwubr~#DgY74$LFhIV(xa#ydhLip2{RY;Iy&#`4?yn zkHO3>P8*!>f-+W`4&egk6!&o|7GrOllC>OYJk8E+hgeiI zg1Uf1@9T#-@Y)HIzKM*f%cVH%#WfA`ifl-MX8JIECMs!R59Y2a7pVm?^GC5{*;w|h z-Or(oRlGWCF_TJzcnwIlAK>t-yI6l9Lf*6+nK#ix(615nCy@R9By}G;QJn5(_O%Es zmn%k+)MfvW@eN5AGq;f3idw2t$FaP%)J~hlyrKvLhd1!_hAu)4GbpMpxJrd8K=}w4 zNPKNB%F@w;e#}GyDOI*O`U6HAl(ey8F%roX$%F&t<&=(|!}9r?`1Xr0vGRo=MJ4l@ zH*Og>OnriF2cP57f7`}ZqlGOk0VbM0!hV~}~;x93C1ZatNI*)8&?bx4$e4T|w=U0R z;ADiAzl!skA524VBB?Vys8eQul#-Y$NjVvPc>eQQmw7&=(tXFnK|ZxGhZmb&4sYn- zAGb;RQ{XiS(h{OEWuzw|*;D*C!LMvu`bFm!wY zlll=lE7!InQzd+StVu=4pf(=m`4^+Cdqu;tA;(ZmnGj@torfG-UDim$^w3b3Vqd`G zc)wuk<%$udB$ANCEI?Bz<+A5>McGh|VI}FbV1$6S0bbnE!@q8~$d_Kioe^?$h*}bl zSEp`L5jWKsRD{8{T{=2aOeip@3+tD!!MLnlsS#|*vT+9-m^kH;esRC zx8_N<{EfwyaJq?&-)}<7BB7>m&mY~$jD`ZjB(aniEi|0PpShXX&cp2Z&KLRfkb^EG zG>=@y9)z2Txk`(8 zB2LnBGubogNUs%*kc_9WRa*BaB5_hF3rhwGl#b!X&o3wP`WjaK^LAeITi_&!&;1BX zu9?Dw{1AHaNER&4Wy6Ml-D^H>-a$F7-qR^2go8ls(T9 z*}ju?>vOT&3dr+$kkMnDZrsoQt+(L%b zN>1?cll&wvkV$qL%ry6wfzq=T(n` zTbU$vs)-XxmxPiiBac8~Eeozm@Z$artabZ|4$*M?nppez)5NC+m{Hom=sb}=DC`Y+ zfWmSVZl>dsxUA72v7B_YG~J`ZKk$qm0dj~k`^N~M1OeN{V z#4dRYH$^IuB9$@tl@TDvm6saTBAsgF+BS|OkxC*h9m7+~=w;V4>cHci`v2|ScbpZ~ z{r~aTnYm?qg{Ai{MG&w8w%9?UvBs~7`6im4Ut+!*Q?1_^(=jIBn4-p1V@d2i*t?O~ z5CjyFDvIo~<=#7U&hL-8?6SKTSQfUh`+hziK3I2W=FXfubLR6o=ggVaYe}UuSUhaM z$sBaROxAw5gqNPapT`D%z?za8);#+%FRw3T(joIXaAYZ!@g&t{l?u;dEuSsxbI*2G>j-BFAjC@Ji|-RJk7m}6YM+a zF!mici2nTtFnV$YMJrzB<>!aeHe|YZMJjM=Pg)mu~&(y;PaMJzj`P1`Txb;IrVo)K+&l$j(lM@UogfN}Qe*2fv|Bf2o zU0u)8NkvQ^Rn8CUH}Z?eH*@FmfDgSSBgYi;l@kg%bYKjV>vio8a4{xN@%Z-^%ggIR zPA%W|Z_)jj{3HiY2pK%UCohOzmKVlL7+;oTOi5GEd8*ptmn9f5*r#}XHTSNG;tU&D z@(Sb37+lVrfiVVpmb!+J|Gecf{fIcD%Y3kHXr9`76Dlp_*h6AC!^$Ykgv9cPGWo1a zslM@QZoc_G9#~UA#q<+6?SfM|YJ3&_tfQv?ROWrY7R*?#w%Iyw6tm7gi&4`a=kC8;&#!LWzN8AB}TwA*e1;#`m#H7D;qg0{F#lddf$=%<_+gwi zvoC&oy3|^9RT@7Q%~F(%Ub2S`aPd6c$fKA$?NK(bUd|h_I57``bS0DKpF{DGCmHr+C!C9CY-TxOo1)^z$8NOPqqLF^oBAUj{FEgn#_x8CEPmi&H=UUmQ|g zhqrZ73K!svN4or2PW$o@#x1;u+i$;$OIA0~5IEwM!x%mDXg+)CJdPMuNI_l_H)?+l zKK%n;_{l9iboC-$n0*4%Ckkg7aunQn6Zii1CjPdikg`FOn0?;4oO$pFCYCnv?rNWSejfSB=%rg* zx9P|8$jeVgEnm(9SH|G!2QmKbdwA)=Kk&mhp5|khUd(Y*2Qu7Gkyc>V1B7Ji3#k}+ zIH&*g$mjuN>u5-)!NhQbPUO?ynvdD~tX2SdGfv^O{ZHleZMx;H=fQ`TtxXRMKa>*= zo5+lc6saI;oUpEeI#)s2xVe1!C-cDBXoiv6E!1z^zz6A&!u=;OY`?MePY2ZFNYzy` z_PBExcYO44bZ8xQb}(ZPKZma$X`;0z}a2cKL;5@)z~sVIj)>fniheuW1EM1vk8fxEr`x) zVToO!d z5CTU9{5uz5CQq@j;_BD4bxf4wS@~556$g(V}``2hT_Ihr)t{zF}r**}?J()g)i&+rv#r~|)Ny`i! zqZBj{u%J&1?8eB9hLNk5)p!S=i#)kB2zF(fU6j!ET z9ubqC|FmEiE79vSy3p&AGNaO?96_k7h!b=P9g|e>;X>(3@3S0~cNZ{t2BW$#3@ z=rBuGId}4Tf_P8fUi+QIW89jim%imj`jYP2TZS9_V(E+g@6DBMl{3JGBaxUwEwfKa zDRH#YH`oe_EG`J^i!z=%(5iz}A8r;|v}RG4vi03w0V5(|yPcKao}DqV;F64G+|;ga z-ux1}&sYCT%8-jWj*H*ITG`DZ1D1*X%@RjWHs)q#s%oMAI{6{>NS!U;ps?}dWK=-17nr!Dx_24FPYd1U>{xax!XFsZ4D>bD=RnLj`|=AG>MY_-JTt#;UhF>CPn|r}MsIPDRgO$Y1#YPrS;n^_%IjwKapDm*zoM zlBi8@B$tVCHVAy8{(G%kE3dTpd0$~wSyb+d4Bvv%Py}_+;q`tIfxNPtRKpIiuv6;$ zHTl9f`$X~$ERs`Qa~qOZMC-X9EN+eGsZc9sX*#&EbO(viweY@A7~T@9=81W9T|gj8 zIW@;&U!!E}CG5=chv0s+#Z6+ho9mF-}e}j*I87LVV^kb-dpg2A8 zaf2`G2Xmrn4}Qs4M+UdYx6Ag<)_D|{mVT|R{m>pxvKr9vGQNHPVbu&7NB#Nvr1ll- zaW<5EjlZWKR-QLB`Akn_XZ99!1$J#j0uQ>6j_>JzF+r=JdXP79Y)7*7Lv zaTFuU-eR1ApkZ^qz>V+r6!@{!VuaWbeXRA?bM)af7Q-nUI@NRTQSViM!BRWjixx`R zVzvy4m^-O0A=nM;-Jb7)PRbO7lqljLpVS8YEjb|--}-{$gy?t|dn-=D)OAWvf~K=E z!%-3iA8fKUCL}$i5#6bZ1d7UsWm{yAdYxW3WoBPgKZMOvemD!@Lo^*}4dHLs@!Ljf zn>xJ|2@b7xcJiuksg=E3gc|gTj~^c0e>e2UDOdYu9QWL_Z-;?1ENd!rD-$eypL<$n z0(2z_A7R&Oslt)2TYFwv6Y=%m(Jhjg?)X1zUUac2!-t)d=;xI2umZG9>|AheqYZ}f z{*2!|I%oU$H1-?A)I(*M8dmdz~DaVC&M<~xAqHg*S<#_30 z%d+$GcoKiKjPC0XE7NFX*ZFg)t}uM!6Q~>%zDo3pmvrGdUpy7lX&3pABEhX#Z;7Yk zM!wAFGq)_nqH+bfM&1%_GQ4{GQv<2C*>+u!&EV|Wh5=vIOp}Pay7ngu`)AH3c!zh~ zeNRdK;>-PSxYFI$DY;X&!&X=m8$sY4$tX2E!_+xaY2UU}vcXAD{A8;t)b14QAZdNa z2~@oS+eBXJ(EWMbLdn!4P{yiN?7h=LLAhDATfSlM6Uv=`{-fwX??vysCX<#Bf%rb3?QG5ZNJI2R>q&-@gSHu!YxZs@ zb1KD5fO$rZQKwJor{+(cMC>`7Ei=vZtAjWCOf=uCYnj&@`Yde_tGr)X=a=u%)A=bq zQibfjV2N@Dua&cAJ;_@B{r!+4PFl^|Z)-3;jH0pd)ygp@0}q7eUAjeYnsP}Rykcuur8fXK9^AMu1dYiLZ2#6B zzR?@(J}JBmcZd{&9Ee6Rk-RA`+PXWK_v`|fm`k5`-!mx*DMRbQ<|{;>X}C!Aj7h;EUWSb(?=^?C`_ooT#7*;>9_%NB_f)fc z@TXdJXO7`s*j~v@=`{9U_5$O%?`OH&M>jdvAJ!M;ly?WzXuG9Wc+VENl^9BFhUr=! zjcqd`n|o{llNHd~>(CI`N5_YIeBb@%e(x2B&$_?@Eb_US4=MKMn9+%4#`}sB^?Yzs ze*aUIGUT#?>nLT4%v5kF*D*TK)6aeMHn zD!@RRm4(hW=HNIi`EjDG@m`c%-}LYhTO2uvFCmsuzzwbL2 zJHVkmr>DVYWlwTmZ-)5}4-DMmOe__BAmwBH-XhTPl?gV9a=bnJ{jfq9qK>=ScPmatb3@jhiQ4@JZ8h&aMF^e`abfhc=79vBDF@Vhtl7eKBgcw;R!7ff!ZQV z{JCy#;v?=0fZjb1Hs@*5a#EVVx{%(~@Ka+k2j)c?2Rt^L|~~E3i~V>rMEPa7qa` zvn0I=T)@Bb6mjW>N~y+1MZHMg+w*07{Q|<}xw#aE2qTlADKK=JvHc(;BhPy$^XBov zd{4GhYpc#R9^m?xGHS?6OG~?n{YAh#b|~5SSncxqAQScXq9d6h_nd=gY+b0UsRbZg zcz7PM|IMHl9e!{>I92cRAvzq$t5rDa!%NFcm3!R|NIIebZZGAd8_-qeybQo+c_}8x zKtfYw)Ey8FP@>Tl?%Xcrr$<28Kuzt12(8h|&C3ri3z#*q8RX%~g+xcukpMZ9^z)Y= zT}ogwSua5W75Pu~UNBRd(*d8szxbLvi}3J3l7p+%|98!4>*%7Rv~XT?;pMphNCHX) zvVY9-11hj!@Lw?x{)s8sxD@@L)_8b%JsI=iO-f)grqxQx3Ip^7rv%j1`#yd<7;5zT z);~Qo8MQ6gu;mB%w|;mARCbdwpw4%GR^>L(PVZXj?|pUg+dkfIm}<1A{DQQ4bofR= zHMukyplY9%HfzSh8@W`dI#5(u~ z2p9~=Fyq6b7Av6HC2GOeYdIEzYHAF+F)a`vg~QO|;P-VEiN{e!Uov0g=gIw|_d+2Ckl4voA#C(k;O5GO9%0-hsmZsd5Apa7!4kq5fT>~@Spoer66xE{hW65LvV<}6eiQc z+x=qJrxlL5uj@^wQR9r*W5=L4X=Y>(We#%5eH*SGAUSU}Zj3itT@ZM~;$~oE^bx5M z!PAYLSy$|Ei9 zsOmh2{?a$CC$ut=Y5olBUpmFYBb1f7pOB|ZFMkqgS9nQs4;sG1zZiiLYTJ^o0EnWU5H(#IKR|H=#7iMprMP6t0AWjrw5&b z8R}p(wV24mSMc)@Fw*8r2R*IP*v|rjpA`_Mz@wpQY1@b<%faJ*aBt>o>0p)gE1J#6 ziqlX$S%a_a?S0WzzU3h~dhfm{xXVSov~}THy-f%Qc3BQ2q7ZG>z?#v0u$h<^On?yM zcZ$*R#dMxZHA1}G;Vq=*=iDyScVOd+i6Df|5DfQ(0W3zaz5AJRMHbA{R zXUe{P*2<+8Z%vRU2|H$39vJDAhQ>*Ink3FBHgDUYMuZb0$W+JiWj1fEnq!vwBX?Va z#Sm4aLsO6t+GQzodqCorkd-cY?QNsP2)h1g^3&AYOC4$jrf8{@5H=i`4t_|Jv;MUu zP2-&&`(3D2XT(0+HVWcF_RAz(EOpFxu(3bYNxy?lvTWS)DPW)^UvN$tavho# zvfBS41oK0$7)#MKxbR{zQ_IhrL4|Xql~!Sa)4Qo(*PR{>fbM>>_ikj}#}lI#q?bdM ztsvd_I*Jcgp%=mvVKzP%A0%}@Cj<^^r*hxAnHFiaPlaul6CJsC8+PhD=-k@Ha6^4- z;Y_2sjbw$hIeTZ8YdJyjnuL>C z4U#EfD}weZ=>>+QYGY;aF?3%N``8$3y&wLPH(d-Nw@j7hj&K_`){7mFwf#^Fxo_oS;D5K$JbthhwB-4>8( zWPM*$)E&2tm%?*Czc~UQIT<3Dr%%OT-wPd}pcASFhjsSx)4;Dinu?yqn_@VmEm(Xu zC8*H*q9);UB5ulNG}M)z=tG*~nHS8$MGMBu!@v>hIUpoO^$z5uEVPg|io!<~ZX)ooXE6jv|B`aV($*y;Ityt4Jh> za}c=@+(tWdAb4)^hu_2h3m{jB#I0C^2_k4V0%S^MkD$^EQ&mNvkG74eOL-(tO65@Lmbx?m=S2Cfp zE{HTyw+9tfGN;XcRi0wlSuVP>?LrykY``w`!qTy|wKQm~3XP;1R+xLIkKVn(^2}YE z?$UIq*Vi{98Gqv;%FDwTqkb5o8tPx?Wmlh5v5A}&nsMqIfqNS5+6KSc1 zWqCev9k?;N+h5^Xm?faiyTd0?BPIMsh?ni6u$Au4 zz7A>2OJXeUBD~pw@cHrrpo@a%0~HuUh}Bczukf6dfN+B@PMBp0oqJ>(PM<`@Zk$bi&E03uHtsG^u|`$iEjBNw0|^h+lnQH`tt@1$u-^#w}B_`q+F2OoT-co&$OZ>Xtz&#K@MR9 z3SXBvLRD%-w#?{lK?+zPu}-Q{m$f;7aal;(-Wx_t0`AS8JYyC-?MSbr7j7iN4PUqB z3%99>Su!_Yu?;d3dxwI>nG$^)zrNoW1gu{@2!DE)4J}x|ME#=53Okky8aJ8#5Uy9w zZ{=FLLnU87AsILqAIiOhn$kZW0o?dTcHJ&3R4&h-o$k0eBA`%4zLB=IwN*z@QJMScT-9ioFphV1r> zvtaM6-?ih>fzz7X(FHaONrb993P+T%_Re(7xG7Wnn7I`?B?J%Zu(SK*4O^eYQsgy9Wt94nl)i_@D zcKGAhYTnBd9(d~7}! zGJkt;nSUvsxfIYtAmCP&Ua_^&5>%w!oIh3dkBF*(!_znuw$*)YBtIKlYySmo>5kfr z${1bFddfE+$t@-~_H5*pU+evMJ;R~;buE`6lrDFS-j zaf#FbPoBSBpDrqkQwCF%qD~ebb@1l2yl~1HUf%9uz~$|lk{zLS7xa!ZL+POFwxg@+ zHoU1gRGW{~s+Be7k?3l{Ql<|pqK!KL$!RkpvuS6+fW1Q02$vC@ZO>&|;C#ztu{-2Gz( zmq9gHkSpqddG?2FBxI8XC1lN5p zCi}xj^M5`PB_bj(-g>XZXD36Nsn6&G$1$HL7Y-C2*1X$UG-yw0iI)bo>dOw>$Wlo- z^Hsp8*UNdQO}&>XSw@2))CkUVTpJpko&9cqAJ9Yvj7}n`Ct-kSX$?tchm*EGz6xyG z2oP~26@PPaboI%B=l%!*0G}E_^4J;8xs1ad}*(rfMfH6>Mn$$ANFQuw-) zk6F=k$HQm*F^HuIj!eMsbAJ7I$m# zV{13V4gU>LJyHgyn%I86KhmhV{i>#Q`Xz*`B@u!LQK*$~VZ7T+%%tqrM^3t>NOqd91YYEXiltg?mOZ$@P9ECkb^r;L3Fm$g(=dn)3o zR1FMfoyp_A4Nm!`GeG0J2(#<5=k<3c$LHb@PHlaGG;A;6MgXYSxI0 zA+8m@tXKU5y}#F1_0pn&=k~UoRFkXNige%SQwLmcpzTMy8u7hd^uUN;hHlDj zCr+oVlG~G+^J7P#zJhu>74}Iha$35v+wE_-b+cZ9sBk|yhQhZaPM^s;?YM9)G?gLQ z^-!fL8gRCt6YwhkYRHPtXwv{<8ol2xAD~EpcNOW9v)7k=D8SkK!E{AWkq84L&s{^< z)}P16u}4Dj&frc>P3MO`ckoQDl&gjIf9fT12oybaM}$Bxp zY6Cmr>h+X(Sq#dAy@VxZF$w{`!T0dkAXoe3{`IXO+f{|{>LdQgm%~Qag4bpCRVvWL zCnJ+y&SStr+I^yM=|vFWrmqeqlj8&+#VEs^WGYf%RLPA(uj)>eU_A0y9GPN z*L8(Lrvf4q)7ocT*~W@FZYLKyovnu(#fgD?L+=SjUci2u@QFN3JFhTU#Gm;8I&)c6 z*UX+SZXqBb&AI%KfN`39{i0SAqRWOA!c{-bH%&0NdAgo8vPgv@qgB-x(cXcmJalEs z`P`%{i6#H-_kI_lU8x%qla<>$tDYDTIFcjs{Kyy>_-|Dchxo1~YQQ<^3zrYM-f+lL zncZtwz!TsnmyVNgxra4dwWnp5FeWX9>wIWNK3M;IlTc$FW@WPM^~V99QnRJWvX~!E zn#j(8teR(Mh{+1~)3}MdVoKol!hjYe~>Ge;1^x(`mSjkX@v$x$A&4t(Y4{Vvz*fK9t@dUYvt zZJJSMS0kZYoE`Dh@I94Mq8z>K59U>?7_T`*Ow(*V<_Y#BZTJ!EuQ>pb1A)f=&id=IUtIhc z-XO34s*#{UC%fEv?0H|-34UP%ZYr>9w|obw^7}g zUfC;;C%E6&*6h7X0JZ!e19&mCXXi($fZM2J*DZfoxp7ourO2>gff^Xmq~?($KY1$( zmMOTn$$Xzx0=RJ}SUz#ZfN#m+5cPT;$w5C2BU_h5QgeYRX{sImH|)wwE$47b9r1an zaZN`QSO0u1;BE@$l!}+GQH}1s{w0o(IDeNB`(4^D7Mo+xIuC_HEPnYe?i~$$jSL(5 zlH)~nFzcVu9K?>`)})=sYR>1n?ncDNePU5yHMV6rw;pAZaKCcTF3t*Y;~XUsi`gN% zDVaLhdN#3PM}+_?d^NulY(24hmeM0VVQ$t8O5#(B_#rzuC}fp!j`)lqbO2CSL6*&# zxXb!IUP|~I&V(C)v$FErL9_j5z|}wJR54dNAl^N19k!JA@N)q{c5!0yt9UdT+P?N| zC=jrInb6ozBlT?!rzjHqXfRx~3jxuoMa0#Ud`8O-FUV6FTfa^Nkro8gZ%;s7Q)G)y z#Aobh~}u1Axi&;H|VBzqJn;Hj}YY zm$=!5vAwdQ3%=)wyAmGcs^)|@ue=T0Q$gJoSz-P2byLGGC~4;h`B$c*kEZJx0Ssa0 zwzrR4x$LLPVz6!#R&U6K-#W)Sw9&xA6Q1Glwfi!GM>4R^gv2XI6osMCdvfodd+bW} zOx@{zo+(+Xd~%u3!K+f!QkW(2&5iOI5qWx!EdWs5s3`Bx1b*7^aM?TbjZOMF!c6$q z+2+wwId9{h!qAU?xicbbi)&?8df@JpX(U(g#H5;pA+QXC2Pmi6D%%MWyM+R78v_0x zW0U~O26CHPC{hT0=Z<*L$+6ur-|~n^I}A$~kX~%Wp*9|YM>jNQ$;4$5AZ?Vu3nQYX zX7+xyH=2{h&w!u1H9O4>^$xcq0&W4p!Os@}_`vKlIjro~lXRgUUiOzaN(}`pEK(Sy zMLc+pVw_NP?PIDKt}VTC_0fPBZhJ8wnRqmM!E<(Kn#z>*%yfI083x2h8+Nib%7?v$ zybkvs5CBmvzB6=U^@#lB%kx~iGm#LI_lpNRwf_gsn-}@GKYa2dV9^BLOctI@{MYtB z;%4U^JB+_{WWy3KlNJ-x&p%mz3wuoFx`qL27D7;XS>0{88DY@wGH)H{n2G74m$KM z<9)qK9UjF)vt`+i*0*FA85TC9I|pYB&7$ErOrxo{@=)X`NIx|u5Y-PXst&P_s({ck zHVLxpOTn~{l_M2Cxwf4nS7jjb<|!xQ*ZD3o^OZoUC;SWuJ`99LhaBqz5l)S9l#93_ z;El@BUKE{xy}-0N$r12KkPt~tG(h;I%&}d~&pb?yy5lh2yvYJEHA&C!nET|VSVsd2 zp|r6SDJBKXTvOm))!h*VAlJKuT+Yi}@Y}DRu~-+As03Sfe96m6pHdgRa+%B@)OWa3 z`}*$89DPJR-{4r?fln>KEWH+@&Flm?5I|OYO4P6gz&HSN1_kcnz4kPB-}oQlb47CW zwcB-T6R>H@81i;tPN|rzde=P_K(`QL^m3QIj>bRnI4GzV}4M+ zRYhS&i}3%DRHDTgv&Hhi4@tvh_ZfZuGAw{>9`toB#?(JJn!f^hni*k&QKlyVopDK6 zEl`0W#!IEX|4QTGV7?bPMbO00mHtoaCS$aEU_qDCzp5W}sHsMW{d+^3#5rV4`15{>phG#k%9{Qw#M=hy#AjQYRH{(rpwpJo4lEINr3 ceQ~uYOgUy{9n=T>#}R?Hy1rV4iv8>V1>wdPKmY&$ literal 0 HcmV?d00001 diff --git a/Documentation/user-review-ui.txt b/Documentation/user-review-ui.txt index 80b14eb589..192876efbf 100644 --- a/Documentation/user-review-ui.txt +++ b/Documentation/user-review-ui.txt @@ -391,6 +391,10 @@ operations are only available on the current patch set. image::images/user-review-ui-change-screen-patch-sets.png[width=800, link="images/user-review-ui-change-screen-patch-sets.png"] +Another indication is a highlighted drop-down label. + +image::images/user-review-ui-change-screen-not-current.png[width=800, link="images/user-review-ui-change-screen-not-current.png"] + [[patch-set-drop-down]] The patch set drop-down list shows the list of patch sets and allows to switch between them. The patch sets are sorted in descending order so diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ChangeScreen.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ChangeScreen.java index eabf0e5e87..d21acb9b14 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ChangeScreen.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ChangeScreen.java @@ -126,6 +126,7 @@ public class ChangeScreen extends Screen { String pushCertStatus(); String replyBox(); String selected(); + String notCurrentPatchSet(); } static ChangeScreen get(NativeEvent in) { @@ -416,6 +417,14 @@ public class ChangeScreen extends Screen { } } + private void updatePatchSetsTextStyle(boolean isPatchSetCurrent) { + if (isPatchSetCurrent) { + patchSetsText.removeClassName(style.notCurrentPatchSet()); + } else { + patchSetsText.addClassName(style.notCurrentPatchSet()); + } + } + private void initRevisionsAction(ChangeInfo info, String revision, NativeMap actions) { int currentPatchSet; @@ -429,16 +438,22 @@ public class ChangeScreen extends Screen { } String currentlyViewedPatchSet; - if (info.revision(revision).id().equals("edit")) { + boolean isPatchSetCurrent = true; + String revisionId = info.revision(revision).id(); + if (revisionId.equals("edit")) { currentlyViewedPatchSet = Resources.M.editPatchSet(RevisionInfo.findEditParent(info.revisions() .values())); currentPatchSet = info.revisions().values().length() - 1; } else { - currentlyViewedPatchSet = info.revision(revision).id(); + currentlyViewedPatchSet = revisionId; + if (!currentlyViewedPatchSet.equals(Integer.toString(currentPatchSet))) { + isPatchSetCurrent = false; + } } patchSetsText.setInnerText(Resources.M.patchSets( currentlyViewedPatchSet, currentPatchSet)); + updatePatchSetsTextStyle(isPatchSetCurrent); patchSetsAction = new PatchSetsAction( info.legacyId(), revision, edit, style, headerLine, patchSets); diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ChangeScreen.ui.xml b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ChangeScreen.ui.xml index c643072b06..4e145d27c8 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ChangeScreen.ui.xml +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ChangeScreen.ui.xml @@ -338,6 +338,10 @@ limitations under the License. .pushCertStatus { padding-left: 5px; } + + .notCurrentPatchSet { + background-color: #FFA62F; + }