Ori.livneh has submitted this change and it was merged.
Change subject: Revert "Load all CSS in the top queue"
......................................................................
Revert "Load all CSS in the top queue"
Did not have a positive impact on perf.
This reverts commit 6ee94ca47c7075aae8f72dd2d84729431c306c36.
Change-Id: Ibc8c88286e1695e37f173ee818bf10b2548952ec
---
M includes/OutputPage.php
1 file changed, 5 insertions(+), 1 deletion(-)
Approvals:
Ori.livneh: Verified; Looks good to me, approved
diff --git a/includes/OutputPage.php b/includes/OutputPage.php
index dd05d27..f2c60a9 100644
--- a/includes/OutputPage.php
+++ b/includes/OutputPage.php
@@ -3077,6 +3077,10 @@
ResourceLoaderModule::TYPE_SCRIPTS
);
+ $links[] = $this->makeResourceLoaderLink(
$this->getModuleStyles( true, 'bottom' ),
+ ResourceLoaderModule::TYPE_STYLES
+ );
+
// Modules requests - let the client calculate dependencies and
batch requests as it likes
// Only load modules that have marked themselves for loading at
the bottom
$modules = $this->getModules( true, 'bottom' );
@@ -3668,7 +3672,7 @@
$otherTags = array(); // Tags to append after the normal <link>
tags
$resourceLoader = $this->getResourceLoader();
- $moduleStyles = $this->getModuleStyles( true );
+ $moduleStyles = $this->getModuleStyles( true, 'top' );
// Per-site custom styles
$moduleStyles[] = 'site';
--
To view, visit https://gerrit.wikimedia.org/r/232374
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ibc8c88286e1695e37f173ee818bf10b2548952ec
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: wmf/1.26wmf18
Gerrit-Owner: Ori.livneh <[email protected]>
Gerrit-Reviewer: Ori.livneh <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
