From 8936d8b51777aa8dca03378904bea7c733c427e4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nguy=E1=BB=85n=20Vi=E1=BB=87t=20H=C6=B0ng?= Date: Tue, 2 Oct 2018 21:43:45 +0700 Subject: [PATCH] updated to code to pass test --- browser/components/MarkdownPreview.js | 5 ++++- browser/lib/markdown.js | 4 ++-- tests/lib/snapshots/markdown-test.js.md | 2 +- tests/lib/snapshots/markdown-test.js.snap | Bin 1724 -> 1729 bytes 4 files changed, 7 insertions(+), 4 deletions(-) diff --git a/browser/components/MarkdownPreview.js b/browser/components/MarkdownPreview.js index 0114f515..6618411e 100755 --- a/browser/components/MarkdownPreview.js +++ b/browser/components/MarkdownPreview.js @@ -483,7 +483,10 @@ export default class MarkdownPreview extends React.Component { eventEmitter.on('export:save-md', this.saveAsMdHandler) eventEmitter.on('export:save-html', this.saveAsHtmlHandler) eventEmitter.on('print', this.printHandler) - eventEmitter.on('config-renew', this.rewriteIframe.bind(this)) + eventEmitter.on('config-renew', () => { + this.markdown.updateConfig() + this.rewriteIframe() + }) } componentWillUnmount () { diff --git a/browser/lib/markdown.js b/browser/lib/markdown.js index 20782a81..248dbb4b 100644 --- a/browser/lib/markdown.js +++ b/browser/lib/markdown.js @@ -265,9 +265,9 @@ class Markdown { } // FIXME We should not depend on global variable. window.md = this.md - ee.on('config-renew', () => { + this.updateConfig = () => { config = ConfigManager.get() - }) + } } render (content) { diff --git a/tests/lib/snapshots/markdown-test.js.md b/tests/lib/snapshots/markdown-test.js.md index ffc3d699..b7251b8d 100644 --- a/tests/lib/snapshots/markdown-test.js.md +++ b/tests/lib/snapshots/markdown-test.js.md @@ -31,7 +31,7 @@ Generated by [AVA](https://ava.li). `␊ ` diff --git a/tests/lib/snapshots/markdown-test.js.snap b/tests/lib/snapshots/markdown-test.js.snap index fc310cfd28f22b2e7d20fa14c94bd5f9b806b0fd..9254709e714084b2fe06a32df4d63ca6e3d235bd 100644 GIT binary patch literal 1729 zcmV;y20r;gRzVaDKcVL&25 zB_=Awn2=}$<4KPMjt1k!iw7Txi3ekhob;q8JQ@9~x~F$$nAu$gJvqsA*Zhy~e^veU z&vi}H?$^Fvzxl>D-=4Wtxc-9~|FQ7sfF?gz*J+yf+{fo0zdU&D{OzkRUzmO9#DJo{ zxn9#A`Lz0T^_$CQKY#VL$8KMo*)pK0-#@Hr3%{@bb%VHd^TRKXeYWAjU$ze@>PH(j z?c~K{QRr;E_Qlel-#u{Ql_v)k^{4c^Ws|18e(2hX()kzOea^dna&hwwcTiE!q~EXL zJE*Nw-VNNRZ5q6^uHqI@4GU~|G$2*Ipd(tV>c@%aFrOHL8Iz1fB4C2(R!T*Jp^Unv zV!8!uL&Yt&!xR&kDUUjJ!zElHks1|-k9obu8i6r1lOGzN85%DPl_zyeNg^pJ(WNA6 zNuu9Ac>4{oQUoh>j$?43L2JUT>La6h9dzitE2?^_FruS~J5}8kB8+yJW~0%_HCZfT zo8-W6llV5NnX~!D1BYg#CBL~}M+(knyheCc2Qch;tT7jRUgU5>0;^&s7VijfGzoB; zIZ+l#X=r>}wkAL`BoULCpXMarzTN&zFYD93eJTDJu3J&L*j5fJ?7aug%%0hxb!tv|EUp=cn}I`vdFAOO1R963#=t=-2R(uPa3@N5 z(Op}WsK&Ic&xFGn^j~I}ENVoQ|hTLKiC?tuIRYh27p;SvHCDB4s z>#p6~wInMo&(IVLU@a1>bDa685(&{2+nt&%80c@@18WSPw41DCa z_|pF2fQBLPl5A%VnChsy9GQ0<_rh3m-Y2-A4s4|f?znV`v>*Km-AcJA<;pJb-b!5n z0<@{fc}=tEAKZ5+9@DL7Nlo^?UBSFZBX^bT$Ogxig31tI?jbLd$jrbFhd8(^Feg|B zn{s6nS*>m>fGvc}VEZt*)`_-45sD+}t-^di4rJp?(_fw*nv{1)iw=o@>ytGme&YD? z({*UuyQ$a{+be|CrgOSHd34|Nx^<*Mg&=%wN(+vI%R;wiumd*^tE1hV#bI~k9k=t3 zS|x-F(eZE;RdolB3z~w*e#$xHx>boFdoAu9^r`@ED&1(lE!^~_JWEv;0AmSrdB-deN1!U&n$>>mPKZ(pgC+#l741{& zJ8ajnAg7EPjY1DMyZlnTUNjglgzJ02k_!+ZJ`UheMnK@K0Z(c<6%(EnpIKE{F+~eg z8nygv6=kc%Y{j#cNLLAJ6o-n6Q)(b{jkQGWKB8s@f1V!uas;a9;y9j=FvJVc0`5dJ1RJ>7}G3cvN;%z)f9J&3-LM1di&nkkq|`_~0WALb!cIGlZZWmpFBsok|AG z)-R)|!x)N=28uvx*^t)@5v4+XkyC!Bg_5!`)q#J{gwd^Y z?>}?&;E`Ro&Rv23h+&szaRtpM7GoxMD&=I^Lw~lfzHkYeqdXEwvT_w$jqL5*h&T-X XMSclB0P?_{E4 z@DT(bL4<(f18*Kff+tbDh~PmC2qGdvPI}T)Jc<5Q-P1cWnSE%YCkLjx=6`(utLm?R zE*XY#tMSd|ZI`cn`^@>$r629&kHve24E1?`lVJoWK015fh2e`YTzmidb8~l`9MaU+ zHXFv>pVfb^U%Bwy7cafM{n{I|_YP_54|f{I;_sV(-6F4k`r+5dKHqZtFFS@b^@FX3 zaq5j@BKEdk{A&5n?`}Ky;)BDQ`hE7@ewSgqa_Hj8$_r1ueImGYYH8bLe^^sR_WcsR z!^S4<-Ow$@UBl-$)%+4_V2Q1OMx<_*Ohg-X^Ee4S77|M`Yl<-;BPNOIWK<*>%BlC( zY`^4esrlt@m~sj;9Z;`n`GjjE(xB1`F>f|lE3!sriz5@WBNL^O>XhkdNu(qdx{^d4 zN%Y$XZ@vmv%3y`ga|{l&XhZsSb8KhP1Rc8I%eq-9jhRSruWtHM#$uOkw_2@2nJy%?M?9iN84%-Jzq~Tn~8-&+Q0K-ARTJuQ|2#*sIIW;@AcwK-y(*S3f zC-O)tBNH>Gqe$TLTkmBqCjmSH>eMjuIhkK^&&OVq@QATUd*^mCYUz~)o^#8`aSZl^ zuwu>{i6`kYm2C@04GSE^*-r?)SQ#brgt$c&EVg;uZ%H5)_J z;O}6k7pU|TS39h+_Z_rzdv=R9sXZOAq+uOyM;?t9w5L-LXcDU$0|#XsbUzN`-6-XW z8@8%Ytr=CHNslwg*9;L&$ztum0p`+xSdai=I(u15Z7~4~X<}4W5!PBL)6z*vwUE-f zVfXedt4gagG|eJd%hc*TXCbNy$yv1EbYyCFM(8^4Ck*dzXM}CK| z>>rM390M=ScJ6?!kE+j+ecf>{ji={*l1u8rR+{9VPnSve(VsM(jEgd^>H_bq)de6x zm&!uWw#(tc{fClq(|Lk4RPWmpEd*5f>tshYIIb0RhWJVkd72122Rj_&=!U?&Xa#KA zS50KSx~&1W7_Wfs$KYBo+8RYDk7c(C3&SK*jW0`ob!KEr-60)1B>weJHkkOy01x<=(_yBdji+Gu5f1`=2zOBP}W=;Tuz0^gLXZrZbB@xN%q$J;GTM_eb7yJO5~!01Ht!Jvc6C8XgBI=Zu?9O+fZK+(aUO|=1#}c>-c^86AOT*-LwtsVoAab#(7H2f- zg!xM3tEGI!^Oek2DXK_fO~n~CQaK7$hB6j&9?Fb$MBP4&ZpEgV?TOp}KRlR2-kkKP2334z#89k;|lHVLwc% zP#mj{jjq5{0Y=@N5O`o==;hHat5_&ajE;`3YQV*!-{%m9J*1+ia2CBmN?L*^0m)s0 zNpu0OGoHm)9`E^7S2Z7U0$Z{EFB|C&C|*}0TvuXX*XWj5#p7aSteU#H^iL7Wok01z zUdq)Z2Go14Er18&z)2K|+o!h-SF*I^Kx%(G*3^w24Xv1o!t}YzsuW743h?^#Gu9=a z8HxuU!g67J^;1XBsQH`2y(=yFL3Sebte{XT-GRxrs}N! zs*!beko*4*`Y(3S`mf$`;$x#T`*j?VSTyG#srw`e!ABa!aQn!13_*K7@tQ6>oer3t zUslfKyHJO=-tQ}%Vp Svl~TT8}l!i*8=N#6951$K|E{#