[MediaWiki-commits] [Gerrit] mediawiki...ProofreadPage[master]: Makes sure that "Proofread tools" toolbar is always loaded a...

2016-12-14 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/327027 )

Change subject: Makes sure that "Proofread tools" toolbar is always loaded 
after Page: pages edit interface setup
..


Makes sure that "Proofread tools" toolbar is always loaded after Page: pages 
edit interface setup

Bug: T152945
Change-Id: Ib3acca7a464dc086f0f4deb05e4c2c9ddffe6513
---
M modules/page/ext.proofreadpage.page.edit.js
1 file changed, 14 insertions(+), 25 deletions(-)

Approvals:
  jenkins-bot: Verified
  Tpt: Looks good to me, approved
  Phe: Looks good to me, but someone else must approve



diff --git a/modules/page/ext.proofreadpage.page.edit.js 
b/modules/page/ext.proofreadpage.page.edit.js
index 720440b..575f9cd 100644
--- a/modules/page/ext.proofreadpage.page.edit.js
+++ b/modules/page/ext.proofreadpage.page.edit.js
@@ -138,9 +138,9 @@
}
 
/**
-* Add some buttons to the toolbar
+* Setup the editing interface
 */
-   function addButtons() {
+   function setupWikitextEditor() {
var iconPath = mw.config.get( 'wgExtensionAssetsPath' ) + 
'/ProofreadPage/modules/page/images/',
tools = {
zoom: {
@@ -222,6 +222,10 @@
 
if ( getBooleanUserOption( 'usebetatoolbar' ) ) {
mw.loader.using( 'ext.wikiEditor.toolbar', function () {
+   $editForm.find( '.prp-page-edit-body' ).append( 
$( '#wpTextbox1' ) );
+   $editForm.find( '.editOptions' ).before( 
$editForm.find( '.wikiEditor-ui' ) );
+   $editForm.find( '.wikiEditor-ui-text' ).append( 
$editForm.find( '.prp-page-container' ) );
+
$edit.wikiEditor( 'addToToolbar', {
sections: {
'proofreadpage-tools': {
@@ -232,6 +236,13 @@
}
} );
} );
+
+   // load the "dialogs" module of WikiEditor if enabled , 
bug: 72960
+   if ( getBooleanUserOption( 'usebetatoolbar-cgd' ) ) {
+   mw.loader.load( 'ext.wikiEditor.dialogs' );
+   }
+
+   // TODO: other modules of WikiEditor may miss, see bug 
72960.
} else if ( getBooleanUserOption( 'showtoolbar' ) ) {
mw.loader.using( 'mediawiki.toolbar', function () {
$.each( tools, function ( group, list ) {
@@ -248,27 +259,6 @@
}
 
/**
-* Improve the WikiEditor interface
-*/
-   function setupWikiEditor() {
-   // Ignore "showtoolbar", for consistency with the default 
behavior (bug 30795)
-   if ( !getBooleanUserOption( 'usebetatoolbar' ) ) {
-   return;
-   }
-   mw.loader.using( 'ext.wikiEditor', function () {
-   $editForm.find( '.prp-page-edit-body' ).append( $( 
'#wpTextbox1' ) );
-   $editForm.find( '.editOptions' ).before( 
$editForm.find( '.wikiEditor-ui' ) );
-   $editForm.find( '.wikiEditor-ui-text' ).append( 
$editForm.find( '.prp-page-container' ) );
-   } );
-
-   // load the "dialogs" module of WikiEditor if enabled , bug: 
72960
-   if ( getBooleanUserOption( 'usebetatoolbar-cgd' ) ) {
-   mw.loader.load( 'ext.wikiEditor.dialogs' );
-   }
-   // TODO: other modules of WikiEditor may miss, see bug 72960.
-   }
-
-   /**
 * Init global variables of the script
 */
function initEnvironment() {
@@ -282,10 +272,9 @@
 
$( function () {
initEnvironment();
-   setupWikiEditor();
+   setupWikitextEditor();
setupPreferences();
setupPageQuality();
-   addButtons();
} );
 
// zoom should be initialized after the page is rendered

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib3acca7a464dc086f0f4deb05e4c2c9ddffe6513
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ProofreadPage
Gerrit-Branch: master
Gerrit-Owner: Tpt 
Gerrit-Reviewer: GOIII 
Gerrit-Reviewer: Phe 
Gerrit-Reviewer: Tpt 
Gerrit-Reviewer: jenkins-bot <>

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


[MediaWiki-commits] [Gerrit] mediawiki...ProofreadPage[master]: Makes sure that "Proofread tools" toolbar is always loaded a...

2016-12-13 Thread Tpt (Code Review)
Tpt has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/327027 )

Change subject: Makes sure that "Proofread tools" toolbar is always loaded 
after Page: pages edit interface setup
..

Makes sure that "Proofread tools" toolbar is always loaded after Page: pages 
edit interface setup

Bug: T152945
Change-Id: Ib3acca7a464dc086f0f4deb05e4c2c9ddffe6513
---
M modules/page/ext.proofreadpage.page.edit.js
1 file changed, 14 insertions(+), 25 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/ProofreadPage 
refs/changes/27/327027/1

diff --git a/modules/page/ext.proofreadpage.page.edit.js 
b/modules/page/ext.proofreadpage.page.edit.js
index 720440b..575f9cd 100644
--- a/modules/page/ext.proofreadpage.page.edit.js
+++ b/modules/page/ext.proofreadpage.page.edit.js
@@ -138,9 +138,9 @@
}
 
/**
-* Add some buttons to the toolbar
+* Setup the editing interface
 */
-   function addButtons() {
+   function setupWikitextEditor() {
var iconPath = mw.config.get( 'wgExtensionAssetsPath' ) + 
'/ProofreadPage/modules/page/images/',
tools = {
zoom: {
@@ -222,6 +222,10 @@
 
if ( getBooleanUserOption( 'usebetatoolbar' ) ) {
mw.loader.using( 'ext.wikiEditor.toolbar', function () {
+   $editForm.find( '.prp-page-edit-body' ).append( 
$( '#wpTextbox1' ) );
+   $editForm.find( '.editOptions' ).before( 
$editForm.find( '.wikiEditor-ui' ) );
+   $editForm.find( '.wikiEditor-ui-text' ).append( 
$editForm.find( '.prp-page-container' ) );
+
$edit.wikiEditor( 'addToToolbar', {
sections: {
'proofreadpage-tools': {
@@ -232,6 +236,13 @@
}
} );
} );
+
+   // load the "dialogs" module of WikiEditor if enabled , 
bug: 72960
+   if ( getBooleanUserOption( 'usebetatoolbar-cgd' ) ) {
+   mw.loader.load( 'ext.wikiEditor.dialogs' );
+   }
+
+   // TODO: other modules of WikiEditor may miss, see bug 
72960.
} else if ( getBooleanUserOption( 'showtoolbar' ) ) {
mw.loader.using( 'mediawiki.toolbar', function () {
$.each( tools, function ( group, list ) {
@@ -248,27 +259,6 @@
}
 
/**
-* Improve the WikiEditor interface
-*/
-   function setupWikiEditor() {
-   // Ignore "showtoolbar", for consistency with the default 
behavior (bug 30795)
-   if ( !getBooleanUserOption( 'usebetatoolbar' ) ) {
-   return;
-   }
-   mw.loader.using( 'ext.wikiEditor', function () {
-   $editForm.find( '.prp-page-edit-body' ).append( $( 
'#wpTextbox1' ) );
-   $editForm.find( '.editOptions' ).before( 
$editForm.find( '.wikiEditor-ui' ) );
-   $editForm.find( '.wikiEditor-ui-text' ).append( 
$editForm.find( '.prp-page-container' ) );
-   } );
-
-   // load the "dialogs" module of WikiEditor if enabled , bug: 
72960
-   if ( getBooleanUserOption( 'usebetatoolbar-cgd' ) ) {
-   mw.loader.load( 'ext.wikiEditor.dialogs' );
-   }
-   // TODO: other modules of WikiEditor may miss, see bug 72960.
-   }
-
-   /**
 * Init global variables of the script
 */
function initEnvironment() {
@@ -282,10 +272,9 @@
 
$( function () {
initEnvironment();
-   setupWikiEditor();
+   setupWikitextEditor();
setupPreferences();
setupPageQuality();
-   addButtons();
} );
 
// zoom should be initialized after the page is rendered

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib3acca7a464dc086f0f4deb05e4c2c9ddffe6513
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ProofreadPage
Gerrit-Branch: master
Gerrit-Owner: Tpt 

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