Alex Monk has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/294190

Change subject: On serialization failure, reset document opacity.
......................................................................

On serialization failure, reset document opacity.

In case we were switching to source.
Normally MWWikitextSwitchConfirmDialog#getActionProcess would do this for us,
but this closeWindow call doesn't trigger that, we can't get it by overriding
MWWikitextSwitchConfirmDialog#close either.

Bug: T134333
Change-Id: I66a12ff6d13601250b9d470e1be54fe38a1ef06c
---
M modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.js
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor 
refs/changes/90/294190/1

diff --git a/modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.js 
b/modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.js
index 7bae6fe..1fc36d9 100644
--- a/modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.js
+++ b/modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.js
@@ -945,6 +945,7 @@
        alert( ve.msg( 'visualeditor-serializeerror', status ) );
 
        this.getSurface().getDialogs().closeWindow( 'wikitextswitchconfirm' );
+       this.resetDocumentOpacity();
 
        // It's possible to get here while the save dialog has never been 
opened (if the user uses
        // the switch to source mode option)

-- 
To view, visit https://gerrit.wikimedia.org/r/294190
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I66a12ff6d13601250b9d470e1be54fe38a1ef06c
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Alex Monk <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to