jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/367686 )
Change subject: Hide loading spinner also on API fail ...................................................................... Hide loading spinner also on API fail Change-Id: I65c1adf8058f2cd0a18466b0d92e43d5ec67d8f7 --- M modules/jquery.wikiEditor.preview.js 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: jenkins-bot: Verified TheDJ: Looks good to me, approved diff --git a/modules/jquery.wikiEditor.preview.js b/modules/jquery.wikiEditor.preview.js index faac2b8..d25dbd1 100644 --- a/modules/jquery.wikiEditor.preview.js +++ b/modules/jquery.wikiEditor.preview.js @@ -68,6 +68,8 @@ disableeditsection: true, useskin: mw.config.get( 'skin' ), uselang: mw.config.get( 'wgUserLanguage' ) + } ).always( function () { + context.modules.preview.$preview.find( '.wikiEditor-preview-loading' ).hide(); } ).done( function ( data ) { var loadmodules, $content; if ( !data.parse || !data.parse.text ) { @@ -84,7 +86,6 @@ mw.loader.load( loadmodules ); context.modules.preview.previewText = wikitext; - context.modules.preview.$preview.find( '.wikiEditor-preview-loading' ).hide(); $content = context.modules.preview.$preview.find( '.wikiEditor-preview-contents' ) .detach() .html( data.parse.text ); @@ -126,6 +127,9 @@ postPromise = api.post( postdata ); $.when( postPromise, mw.loader.using( 'mediawiki.diff.styles' ) ) + .always( function () { + context.$changesTab.find( '.wikiEditor-preview-loading' ).hide(); + } ) .done( function ( postResult ) { var diff; try { @@ -141,7 +145,6 @@ } catch ( e ) { // "data.blah is undefined" error, ignore } - context.$changesTab.find( '.wikiEditor-preview-loading' ).hide(); } ); } } ); -- To view, visit https://gerrit.wikimedia.org/r/367686 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I65c1adf8058f2cd0a18466b0d92e43d5ec67d8f7 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/WikiEditor Gerrit-Branch: master Gerrit-Owner: Fomafix <foma...@googlemail.com> Gerrit-Reviewer: TheDJ <hartman.w...@gmail.com> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits