Tpt has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/358158 )
Change subject: Restricts modules loaded by addModuleStyles() to styles only
......................................................................
Restricts modules loaded by addModuleStyles() to styles only
Bug: T167353
Bug: T92459
Change-Id: I3618015426edeb4e1378fa72bee4b4d4da53756a
---
M extension.json
M includes/page/EditPagePage.php
M includes/page/PageViewAction.php
M modules/ve/pageTarget/ve.init.mw.ProofreadPagePageTarget.init.js
4 files changed, 7 insertions(+), 15 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/ProofreadPage
refs/changes/58/358158/1
diff --git a/extension.json b/extension.json
index 47afd82..7404cf2 100644
--- a/extension.json
+++ b/extension.json
@@ -115,11 +115,7 @@
},
"ext.proofreadpage.page": {
"position": "top",
- "styles": "page/ext.proofreadpage.page.css",
- "dependencies": [
- "ext.proofreadpage.base",
- "ext.proofreadpage.ve.pageTarget.init"
- ]
+ "styles": "page/ext.proofreadpage.page.css"
},
"ext.proofreadpage.page.edit": {
"styles": "page/ext.proofreadpage.page.edit.css",
@@ -129,7 +125,8 @@
"jquery.prpZoom",
"mediawiki.user",
"user.options",
- "jquery.textSelection"
+ "jquery.textSelection",
+ "ext.proofreadpage.ve.pageTarget.init"
],
"messages": [
"proofreadpage_quality0_category",
diff --git a/includes/page/EditPagePage.php b/includes/page/EditPagePage.php
index 66e6ec8..1baa64c 100644
--- a/includes/page/EditPagePage.php
+++ b/includes/page/EditPagePage.php
@@ -116,9 +116,7 @@
$out->addHTML( $this->pagePage->getPageContainerEnd() );
$out->addModules( 'ext.proofreadpage.page.edit' );
- $out->addModuleStyles( [
- 'ext.proofreadpage.page.edit',
'ext.proofreadpage.page', 'ext.proofreadpage.base'
- ] );
+ $out->addModuleStyles( [ 'ext.proofreadpage.base',
'ext.proofreadpage.page' ] );
}
/**
diff --git a/includes/page/PageViewAction.php b/includes/page/PageViewAction.php
index 80a0f0c..4e577e2 100644
--- a/includes/page/PageViewAction.php
+++ b/includes/page/PageViewAction.php
@@ -43,10 +43,8 @@
$out->addHTML( $page->getPageContainerEnd() );
// add modules
- $out->addModules( 'ext.proofreadpage.page' );
- $out->addModuleStyles( [
- 'ext.proofreadpage.base', 'ext.proofreadpage.page'
- ] );
+ $out->addModules( 'ext.proofreadpage.ve.pageTarget.init' );
+ $out->addModuleStyles( [ 'ext.proofreadpage.base',
'ext.proofreadpage.page' ] );
$out->addJsConfigVars( [
'prpPageQuality' => $content->getLevel()->getLevel()
] );
diff --git a/modules/ve/pageTarget/ve.init.mw.ProofreadPagePageTarget.init.js
b/modules/ve/pageTarget/ve.init.mw.ProofreadPagePageTarget.init.js
index c28d735..556a56e 100644
--- a/modules/ve/pageTarget/ve.init.mw.ProofreadPagePageTarget.init.js
+++ b/modules/ve/pageTarget/ve.init.mw.ProofreadPagePageTarget.init.js
@@ -8,8 +8,7 @@
( function () {
mw.loader.using( 'ext.visualEditor.desktopArticleTarget.init', function
() {
// Adding this module to VisualEditorPluginModules would cause
it to load on
- // all VE-loading pages (even non PRP namespaces). Instead add
it to the plugin
- // list here, only on pages that have loaded
'ext.proofreadpage.page' first.
+ // all VE-loading pages (even non PRP namespaces).
mw.libs.ve.addPlugin( 'ext.proofreadpage.ve.pageTarget' );
} );
}() );
--
To view, visit https://gerrit.wikimedia.org/r/358158
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3618015426edeb4e1378fa72bee4b4d4da53756a
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ProofreadPage
Gerrit-Branch: master
Gerrit-Owner: Tpt <thoma...@hotmail.fr>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
