From ba34e37bce5b7c29bc11c9aa4fe994977fb21e91 Mon Sep 17 00:00:00 2001 From: Alexandre Paillier Date: Thu, 15 Sep 2022 16:32:26 +0200 Subject: [PATCH] Downgrade to make the app fit on LNS --- src_bagl/ui_flow_signMessage.c | 14 ++++++++++++++ src_features/signMessage/cmd_signMessage.c | 14 +++++++++++--- .../snapshots/nanos_eip191_opensea/00005.png | Bin 478 -> 457 bytes .../snapshots/nanos_eip191_opensea/00006.png | Bin 464 -> 407 bytes .../snapshots/nanos_eip191_opensea/00007.png | Bin 463 -> 449 bytes .../snapshots/nanos_eip191_opensea/00008.png | Bin 426 -> 407 bytes .../snapshots/nanos_eip191_opensea/00009.png | Bin 404 -> 349 bytes .../snapshots/nanos_eip191_opensea/00010.png | Bin 448 -> 0 bytes .../snapshots/nanos_eip191_opensea/00011.png | Bin 470 -> 0 bytes .../snapshots/nanos_eip191_opensea/00012.png | Bin 406 -> 0 bytes .../snapshots/nanos_eip191_opensea/00013.png | Bin 464 -> 0 bytes .../snapshots/nanos_eip191_opensea/00014.png | Bin 407 -> 0 bytes .../snapshots/nanos_eip191_opensea/00015.png | Bin 449 -> 0 bytes .../snapshots/nanos_eip191_opensea/00016.png | Bin 407 -> 0 bytes .../snapshots/nanos_eip191_opensea/00017.png | Bin 349 -> 0 bytes tests/zemu/src/eip191.test.js | 2 +- 16 files changed, 26 insertions(+), 4 deletions(-) delete mode 100644 tests/zemu/snapshots/nanos_eip191_opensea/00010.png delete mode 100644 tests/zemu/snapshots/nanos_eip191_opensea/00011.png delete mode 100644 tests/zemu/snapshots/nanos_eip191_opensea/00012.png delete mode 100644 tests/zemu/snapshots/nanos_eip191_opensea/00013.png delete mode 100644 tests/zemu/snapshots/nanos_eip191_opensea/00014.png delete mode 100644 tests/zemu/snapshots/nanos_eip191_opensea/00015.png delete mode 100644 tests/zemu/snapshots/nanos_eip191_opensea/00016.png delete mode 100644 tests/zemu/snapshots/nanos_eip191_opensea/00017.png diff --git a/src_bagl/ui_flow_signMessage.c b/src_bagl/ui_flow_signMessage.c index 97546a3..a6809e7 100644 --- a/src_bagl/ui_flow_signMessage.c +++ b/src_bagl/ui_flow_signMessage.c @@ -7,13 +7,18 @@ static uint8_t ui_pos; static void dummy_pre_cb(void) { if (ui_pos == UI_191_POS_REVIEW) { +#ifdef TARGET_NANOS + skip_rest_of_message(); +#else question_switcher(); +#endif } else { ux_flow_prev(); ui_pos = UI_191_POS_REVIEW; } } +#ifndef TARGET_NANOS static void dummy_post_cb(void) { if (ui_pos == UI_191_POS_QUESTION) { continue_displaying_message(); @@ -22,6 +27,7 @@ static void dummy_post_cb(void) { ui_191_switch_to_message_end(); } } +#endif // clang-format off UX_STEP_NOCB( @@ -46,6 +52,7 @@ UX_STEP_INIT( { dummy_pre_cb(); }); +#ifndef TARGET_NANOS UX_STEP_CB( ux_191_step_theres_more, bn, @@ -61,6 +68,7 @@ UX_STEP_INIT( { dummy_post_cb(); }); +#endif UX_STEP_CB( ux_191_step_sign, pbb, @@ -85,8 +93,10 @@ UX_FLOW(ux_191_flow, &ux_191_step_review, &ux_191_step_message, &ux_191_step_dummy_pre, +#ifndef TARGET_NANOS &ux_191_step_theres_more, &ux_191_step_dummy_post, +#endif &ux_191_step_sign, &ux_191_step_cancel); @@ -100,18 +110,22 @@ void ui_191_switch_to_message(void) { ui_pos = UI_191_POS_REVIEW; } +#ifndef TARGET_NANOS void ui_191_switch_to_message_end(void) { // Force it to a value that will make it automatically do a prev() ui_pos = UI_191_POS_QUESTION; ux_flow_init(0, ux_191_flow, &ux_191_step_dummy_pre); } +#endif void ui_191_switch_to_sign(void) { ux_flow_init(0, ux_191_flow, &ux_191_step_sign); ui_pos = UI_191_POS_END; } +#ifndef TARGET_NANOS void ui_191_switch_to_question(void) { ux_flow_init(0, ux_191_flow, &ux_191_step_theres_more); ui_pos = UI_191_POS_QUESTION; } +#endif diff --git a/src_features/signMessage/cmd_signMessage.c b/src_features/signMessage/cmd_signMessage.c index 77f5ac0..4c257b7 100644 --- a/src_features/signMessage/cmd_signMessage.c +++ b/src_features/signMessage/cmd_signMessage.c @@ -177,13 +177,17 @@ static void feed_display(void) { processed_size += 1; } else { // fill the rest of the UI buffer spaces, to consider the buffer full - while (remaining_ui_buffer_length()) { - sprintf(remaining_ui_buffer(), " "); - } + memset(remaining_ui_buffer(), ' ', remaining_ui_buffer_length()); } } } +#ifdef TARGET_NANOS + if ((remaining_ui_buffer_length() == 0) && (unprocessed_length > 0)) { + sprintf(remaining_ui_buffer() - 3, "..."); + } +#endif + if ((remaining_ui_buffer_length() == 0) || (tmpCtx.messageSigningContext.remainingLength == 0)) { if (!states.ui_started) { @@ -248,6 +252,7 @@ bool handleSignPersonalMessage(uint8_t p1, return true; } +#ifndef TARGET_NANOS /** * Decide whether to show the question to show more of the message or not */ @@ -260,6 +265,7 @@ void question_switcher(void) { ui_191_switch_to_sign(); } } +#endif /** * The user has decided to skip the rest of the message @@ -273,6 +279,7 @@ void skip_rest_of_message(void) { } } +#ifndef TARGET_NANOS /** * The user has decided to see the next chunk of the message */ @@ -282,3 +289,4 @@ void continue_displaying_message(void) { feed_display(); } } +#endif diff --git a/tests/zemu/snapshots/nanos_eip191_opensea/00005.png b/tests/zemu/snapshots/nanos_eip191_opensea/00005.png index b1bddfeb436638f9617d76879acea9e049076a23..1a86e377a60d743f4264c7a54fb603066da0906b 100644 GIT binary patch delta 431 zcmV;g0Z{(l1IYuBB!7@eL_t(|ob8#xa>O7AhRsg%{tui(_K=Qs;D@C~MK^rYLt+pq zA}X6w5ClQ+RJOkA&m*Ojwq0u(#(DgXd)aV7Y}2aPe2Yg z5}W!fY>|3Bn#&j2Bk&bIz*pX0r|LZZ{-93fYip)i0c8u%%YTxvQ73^bg!!kN(@Ta- zK;GGSP9QC0)8l&}&pcf7Qkm!FZ2*EFMU-IN_O(1ta7{s_XDyp>5>i$v!*!NpNu%(C zG8uH-91lr+Z7}ZGc7f&{4XR%B^UBM40km|wQ0!;mk#T;2wQKzYUqH-(CtsC?mg9qy zTirHqGNcSpkAMGx3My*lwaLBTlp7Z@cxi$~0$6|7Z$p+5$#BYL%3^@ASC(`>uS<1A zT|g49R<1YN`|5^ST554EMZh>B<#uzlol>b4%5=lK8md=1 zA7DW16;M{^`E&ZT2`l+WDs%EiNfMfxWneF!l%(C5KPXH@zYtshkm{X+9#2d_5Cp+< Zc>~+`M5mIN?+O3_002ovPDHLkV1hsT&lCUv delta 452 zcmV;#0XzQ51KtCWB!8tzL_t(|ob8#*c7!kphU1+2{txWJS=gp2{2G3^~XCAJ3s?781Z2*G5iYURjtaEvs&@}~>p0#YkNl00x4A)tXC5^&g zl*yp&=D11XtAlaJwhL76Xi)W{w`X3?6QHHjiDEwkUm3>-SUcAPd;l>AJ^83CG#w{S z)^B^yP?OXDeSa(_`8p8xujRGDy>CIPM9Ew~h{3ZGtPQ}jyM73=j7Wx4&QKNujJ>j? zqKBB_^5sJ+AL;^<@NMOKqrI*kV@Q?)jlPk`%3 zK8sGN!U|>l=~NBX3!G0d;D<^ppxkiy|8%wqEBQyNZawlwNfMfxWneF!l=ydx)3*ti uP52UiAvQ+vR$<5wGEX8VAP9osxx4{|Ra$+bZhj5`0000L_t(|ob8!G5`!QNMT1lCe_$7OK_`t6FajEcyp>f)>HkTP z0ALt~Va$rt9st0;{g2)H`{`D#;Y`N99SoR{kM;l}Xa>aI_!PlJf!F8CqCEfr;Dn~2 z#CYI67y@krsHRh*Z2$;77X{;(33tDmYc&D9U$Wf)T#sV`+WTVz^Qo=ISYs`0?ozXTD= zg%{*$&h`n6Q%*!K%L9vjA?<}S_3O~y51_40w#ON-)XL64x@JligSXK^F9>U5TpZCD zR;l1-dd45KJ$3D%x0-pyBlNc)fY48@)K%(1ej}ma6_3!d0Z1I_R5%~Jv6Lw%*j1$7 z_ge=gC`Cq;eEm6nY`nvxN8cIXLpu}HnKSv_1-z4eh0&f~d#fFnpgGW92RzBBy;Z|7 aj8eX+dC!iXSYf>Y0000JhIN_m|G-|j3o_R9L{}+d?3Y~_tUu67 z!ERZSBuSE|vIc4&`?r%#Y`F_%!xGV`6h!?_r(|hSFbbkYX6!L+w~s%DvMWG2n9>8E zz9k}spiMaJ7)rSLscv=}?oICxKy-3?e;ACb)BA(T7+gtap)2w4!{sbof4%xXDJ4A~ z@iMU=+b0$L4}UwfA_CwraC;#jsGUj?Qc;t=MwYrx^KMfdRfJH*P485?RW;#7CF*-k zS^>>sma$RKD$U+mgQk>pZPe7KW?-^EWcrp|*lMUlq`M|yvH+A~8B8f>;^8j7qVb^g z(sd`8EC2`?47b90e?`XN&rMkRLrU8H6a}_c?7nL;pfAHfFSvaZRy2*qNoTxkV(t|5 g05Skck|fXN11JYR+5rjyOD`O!Ud7)^nCSV zm48R~fHVz_gJJ3!JB$;A73W%!0pdWH0?92?e|oM4;Hw|TmLUMNuLZb};ok^+|xiwFd1}a)rFBE)`xpff&pC(>OcSi33 zPQ<&-toCh`ir8`YG?Q(ksdQ&h4}&!Q)A~Sx3z{1R{~!y>Kz^Aq48t%Q@d5O+z+!)U RBVhml002ovPDHLkV1lt6!6pCz delta 437 zcmV;m0ZRVC1J47HB!8AkL_t(|ob8#*5`!=ZMN_B!{||Oy7tUlT5JH|Lma|ecm=Ym) z8-n9Fj^hmBI>U8lEHft~Pa|6HtMGIf?{Tiz$9AR35U}QC$a1cv*Od56*zU}N%(fWh3 zYHD9()udOT+_CjHbRVd2b>Pc8r}G9#<#eNWKf<<*>%+`;uOILLd<`slR2ABeOpv>; z0W8z9zPRu5XMcqddgp9PTSi;j8&R_EfCcDz)uRKa=EK*_N!j#IoX?kcuBkGC9XG{A}#~M||W-VyMzeq}-El-qH z*okyfLuJ8o9iJ^vgtfk3-7m=iR%oo=gatG{>10yjqb|m*3L7o1j8(FDn+sU-51D_$ f7r=2G=YRPEfjU?+9`Ex500000NkvXXu0mjfuC>j3 diff --git a/tests/zemu/snapshots/nanos_eip191_opensea/00008.png b/tests/zemu/snapshots/nanos_eip191_opensea/00008.png index d5c4fde635b43d9bdf246bf35def68777080c917..f5c2d6774a86d59f5cad0974eb03748765433d08 100644 GIT binary patch delta 380 zcmV-?0fYXk1D6AkB!6B>L_t(|ob8!G5`!QNMT1lCe_$7OK_`t6FajEcyp>f)>HkTP z0ALt~Va$rt9st0;{g2)H`{`D#;Y`N99SoR{kM;l}Xa>aI_!PlJf!F8CqCEfr;Dn~2 z#CYI67y@krsHRh*Z2$;77X{;(33tDmYc&D9U$Wf)T#sV`+WTVz^Qo=ISYs`0?ozXTD= zg%{*$&h`n6Q%*!K%L9vjA?<}S_3O~y51_40w#ON-)XL64x@JligSXK^F9>U5TpZCD zR;l1-dd45KJ$3D%x0-pyBlNc)fY48@)K%(1ej}ma6_3!d0Z1I_R5%~Jv6Lw%*j1$7 z_ge=gC`Cq;eEm6nY`nvxN8cIXLpu}HnKSv_1-z4eh0&f~d#fFnpgGW92RzBBy;Z|7 aj8eX+dC!iXSYf>Y0000@j5qtw6W#j>_L0#)jJ0!Jzyd@JMzT~DnvRN-r;EW{ z2G3z_wQsW!n}6M@XfG4VrIsfZvaeCJGpgy8vTtpYsdFQ>9?Qb=baqzTM%Cs%jP8co zqw%R>xmzm$v@_b)D+R6|_F=66RNLMD7##qqdC7i1kUlD)0l=-^CfH^_}!yz*BZkv^2=?Hdxc{A*JQ7n8RtPp zK=N5{Easxhaf*%Yv5kPk2P1#Has1IRPeHBs$cs9T66OLVbARr_7zz;OoG=T?BQe() z5gxob-#-pvr56zPj+vZs*7zvc)y3?k1aOLJy=-1h0GG6GZi4}#20ekxLQ8>o4!52c z$FXo$VhXU?0M$^Bekqgf%Ka;M*^X|4Es9YCQ(xfJKmNC#+8dY%2&wT+>D^<3C%}l9*&i%XWSoJMrBXj>p;ob#c|nYktWRuGe4l zu?5)6cx!W;o!k2w39=|MHnvru!LTUgW(Tw-?aBpFqE99UM1LEgl-yopnBstCJ12u< z0w@9Iz}CU0!IDEop4EB)6xCtmFK!&l;sDbS)LK?^2p1+~wQaD>Ru`uQw=B$MX*gDw z^bV9Kw!Xpb9TlQhSRHvg2S5s^f!2P7yfXTS7;UTxJb?6pC69_i;kYpJ93PNczEXYH zooDizySN3FU~3FQOp})}aZ1i`_aG8u#A}8sDUY;UMWCxU9A9%v$Axvg8xiqKTOVFDnNs=T=H2tc({?|)zy?ZTN1^7lg`4sTUck`NG*;4~FQQ541f{MPBW8ZAAQ@iexUO;z|9 z!#8N$1hCehm)(zehB}~`pe@_;$hJtml&R-sSU7fKTONvH8q`vlOV;s>pbuq=wfqAG zV!ZW?2!Ji{&4rK7^`ABqnwkx=0OB3cBAw<0)PnLaO~=$IbYwGk(?IN%?pdC<32zRA zMptZf!gf-G4Pu-F;2!ANgeM_oK^ZzGoJ+Aa%|9r%`nf90!N9p=`vpR~%j10T-=jnF z zc>B4Yn(c!v$9@{~MHfa-KwezA^OjrIBE9u!eq(`RnTEC$>{9GoJSeW%6r?98w=6VG z1V9U1x$xFF3dLg7A*z%OFCV&0!1g&iLvsSMpe#(2G94P~d8y3v8r=YnJYnoDExgo- zBZj6RII@;aI0-2WmZ3gjT~aNyB&xLE9FO`)zkzYbwijsrqQTV*KVCVU7eGs=3$6W( zbf%pgrnPHz;17s77|CB{q2(wH@+x!Sl=rWGSylR971wTbQK2(;4`RA7$sTa%Ergza z6FD)#7wK6El`Nj=F2VYz*8_xDcD)p}$=i=u`wekaN0CB}{Wa4f0$RcPG@cEWS-k2h zmDH!ERA@oROG`7{xN;ZglI14xNO)b{^wMT!)KDk%W~003P$k8~avl1OKW8{9W>w5P-G zex~Q;JkqvTh2m&8MlV3#T>0ZIkD{dZ)}#5xg5)w!?OL+ea^GS>F_kL}5n4vQnI;2D zNx5^&M`IK$Y9v?4q+)vYA!Grz{r54vCJ+g!!n`Q+&~VF3Ri4*q2PE=>iH}@)sfZJX zmY_1SmY;B{l&n&R+briII^wa1=WFP>Xw)IS0plOruHgQn!PJWmubj>cpq0~w_I^e@ zGR}|S?OGpj1F!}oxm6Wfj*B4Awgz7QwApcw#%$r0@oK_M5_FX2x=>VTuw(DAD$0Juwjz37DF@qWCA=5p)#-Jt41vvO&XnEI zClQ_jN;{JaYumim^0e4dTCm310RR910KngT0Ju6Fb@P;L+yDRo07*qoM6N<$f~J(a A&;S4c diff --git a/tests/zemu/snapshots/nanos_eip191_opensea/00013.png b/tests/zemu/snapshots/nanos_eip191_opensea/00013.png deleted file mode 100644 index 6b52a0997659c7bd5e30e09a5d5917a51dd9668f..0000000000000000000000000000000000000000 GIT binary patch literal 0 HcmV?d00001 literal 464 zcmV;>0WbcEP)DUfW#j?F zesxzI%CY}~dQaXXIyv@b-FQ#2tE3v2M))9n2WvZPOy5>k$KDvjr7J-4GQD0mEg0x7 zwS-h-NnGhMfFUb@g-)!z`vo+4ix+f3WCm9x8~ah2Ej|Ens!}4di1ZyvGp(T>!xGV` z6h!?_r(|hSFbbkYX6!L+w~s%DvMWG2n9>8Ez9k}spiMaJ7)rSLscv=}?oICxKy-3? ze;ACb)BA(T7+gtap)2w4!{sbof4%xXDJ4A~@iMU=+b0$L4?DCX0^l%kdm$jGok|f> zQIow!mby;!Zc`jpgiysz?^L=~HQ_}i>U&LE0nK8Tu~E+|&E8pqrj>)YPYDV6s1C z`j%YSYN$h`yCz_=0F+`GOetsL;V!%eG^tRjmAl5ylZ0a6!ZWx07;T0&*cLs2R_;X3IXK+0000H|@ zz8wshkB{~MB4`H0-uM*3M1j}m%A!300N{kCpu~9KJs1LQ1E{7`qHO>OJQ#XfRy9Bv z^==(ieYk597RD@{OEdS6GT+Lmuf_h$vek1^KVM4p7nbmHWt7bot=9>X{;(33tDmYc z&D9U$Wf)T#sV`+WTVz^Qo=ISYs`0?ozXTD=g%{*$&h`n6Q%*!K%L9vjA?<}S_3O~y z51_40w#ON-)XL64x@JligSXK^F9>U5TpZCDR;l1-dd45KJ?)^knt8<|^tT^?&`+$? zRq8>0Bcb3GkI=CJNF3=@I3K*Rlqo0JRixhcTL&d5MMji-{W*PXyu+hM-x=UTI}_BI zGx^;Gypw%}(Vkv=s~wl1InZ7QJjtlNRl_ihQog8p&yJp0VZ8tV002ovPDHLkV1iJ- Bw$=au diff --git a/tests/zemu/snapshots/nanos_eip191_opensea/00015.png b/tests/zemu/snapshots/nanos_eip191_opensea/00015.png deleted file mode 100644 index d55782f429e629a5ae929916914befa14613edba..0000000000000000000000000000000000000000 GIT binary patch literal 0 HcmV?d00001 literal 449 zcmV;y0Y3hTP)9Z)nZ zoKdfPKxBZtOD`O!Ud7)^nCSVm48R~fHVz_gJJ3!JB$;A z73W%!0pdWH0?92?e|oM4;Hw|TH|@ zz8wshkB{~MB4`H0-uM*3M1j}m%A!300N{kCpu~9KJs1LQ1E{7`qHO>OJQ#XfRy9Bv z^==(ieYk597RD@{OEdS6GT+Lmuf_h$vek1^KVM4p7nbmHWt7bot=9>X{;(33tDmYc z&D9U$Wf)T#sV`+WTVz^Qo=ISYs`0?ozXTD=g%{*$&h`n6Q%*!K%L9vjA?<}S_3O~y z51_40w#ON-)XL64x@JligSXK^F9>U5TpZCDR;l1-dd45KJ?)^knt8<|^tT^?&`+$? zRq8>0Bcb3GkI=CJNF3=@I3K*Rlqo0JRixhcTL&d5MMji-{W*PXyu+hM-x=UTI}_BI zGx^;Gypw%}(Vkv=s~wl1InZ7QJjtlNRl_ihQog8p&yJp0VZ8tV002ovPDHLkV1iJ- Bw$=au diff --git a/tests/zemu/snapshots/nanos_eip191_opensea/00017.png b/tests/zemu/snapshots/nanos_eip191_opensea/00017.png deleted file mode 100644 index ce795f34e8569e986af689fded3b59c9a8af2961..0000000000000000000000000000000000000000 GIT binary patch literal 0 HcmV?d00001 literal 349 zcmV-j0iyniP)O41}p;-~WL<=z+0=1Om1tEU@3H#qtw22}5*_5JCvCo4!52c$FXo$VhXU?0M$^Bekqgf%Ka;M*^X|4 zEs9YCQ(xfJKmNC#+8dY%2&wT+>D^<3C%}_*k1^mvtO~`RE00000NkvXXu0mjf-kqOA diff --git a/tests/zemu/src/eip191.test.js b/tests/zemu/src/eip191.test.js index c54b013..c7f25bd 100644 --- a/tests/zemu/src/eip191.test.js +++ b/tests/zemu/src/eip191.test.js @@ -56,7 +56,7 @@ nano_models.forEach(function(model) { if (model.letter == 'S') { - await sim.navigateAndCompareSnapshots('.', model.name + '_eip191_opensea', [1, 5, 1, 6, 0, 1, -1, 0]); + await sim.navigateAndCompareSnapshots('.', model.name + '_eip191_opensea', [1, 5, 1, -1, 0]); } else {