jenkins-bot has submitted this change and it was merged.

Change subject: mw.ViewPageTarget: Check elementsThatHadOurAccessKey before 
accessing
......................................................................


mw.ViewPageTarget: Check elementsThatHadOurAccessKey before accessing

Follows-up I31d6b9ea065b3be85ed0.

Bug: 62329
Change-Id: I83a6f20cafd6193d64142914f72c1496d03294a5
---
M modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Alex Monk: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js 
b/modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js
index 80e0489..6185a3c 100644
--- a/modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js
+++ b/modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js
@@ -226,7 +226,7 @@
                ) {
                        this.deactivating = true;
                        // User interface changes
-                       if ( ve.msg( 'accesskey-save' ) !== '-' && ve.msg( 
'accesskey-save' ) !== '' ) {
+                       if ( this.elementsThatHadOurAccessKey ) {
                                this.elementsThatHadOurAccessKey.attr( 
'accesskey', ve.msg( 'accesskey-save' ) );
                        }
                        this.restorePage();

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I83a6f20cafd6193d64142914f72c1496d03294a5
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Krinkle <krinklem...@gmail.com>
Gerrit-Reviewer: Alex Monk <kren...@gmail.com>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to