Alex Monk has uploaded a new change for review.

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

Change subject: Simplify some SET logic
......................................................................

Simplify some SET logic

Change-Id: Ie89260607f0253dd7d749af7426702985a0340f4
---
M modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.init.js
1 file changed, 8 insertions(+), 12 deletions(-)


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

diff --git a/modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.init.js 
b/modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.init.js
index 22e5ea7..2ce0bcb 100644
--- a/modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.init.js
+++ b/modules/ve-mw/init/targets/ve.init.mw.DesktopArticleTarget.init.js
@@ -771,28 +771,24 @@
                        if (
                                conf.singleEditTab &&
                                mw.user.options.get( 'visualeditor-tabs' ) !== 
'multi-tab' &&
-                               userPrefEnabled
-                       ) {
-                               loadVEInPage = pageCanLoadVE && (
+                               userPrefEnabled &&
+                               pageCanLoadVE && (
                                        mw.user.options.get( 
'visualeditor-tabs' ) === 'prefer-ve' ||
                                        (
                                                getLastEditor() !== 'wikitext' 
&&
                                                mw.user.options.get( 
'visualeditor-tabs' ) !== 'prefer-wt'
                                        )
-                               );
+                               )
+                       ) {
                                // Handle section edit link clicks
                                $( '.mw-editsection a' ).on( 'click', function 
( e ) {
-                                       if ( loadVEInPage ) {
-                                               init.onEditSectionLinkClick( e 
);
-                                       }
+                                       init.onEditSectionLinkClick( e );
                                } );
                                // Allow instant switching to edit mode, 
without refresh
                                $( '#ca-edit' ).on( 'click', function ( e ) {
-                                       if ( loadVEInPage ) {
-                                               trackActivateStart( { type: 
'page', mechanism: 'click' } );
-                                               activateTarget();
-                                               e.preventDefault();
-                                       }
+                                       trackActivateStart( { type: 'page', 
mechanism: 'click' } );
+                                       activateTarget();
+                                       e.preventDefault();
                                } );
                        } else if ( userPrefEnabled ) {
                                init.setupSkin();

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie89260607f0253dd7d749af7426702985a0340f4
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