Page MenuHomePhabricator
Create Task
Maniphest T28131

Extension style-sheets are loaded after user style-sheets.
Closed, ResolvedPublic
Actions

Description

I have just switched from REL_1_16 to trunk. I have the CodeReview extension installed, and have a bunch of local CSS rules (in [[MediaWiki:Common.css]]) which customise the styling of the CodeReview output.

In 1.16, before the resource loader was added, the load order for CSS stylesheets was extensions, then core, then user styles.

In trunk, this appears to have changed to be core, user styles, extensions.

The correct order shold be core, extensions, user styles.

The current situation means that my local styles are being over-ridden by the default CodeReview stylings, and there is no way to get round this - I can't even use !important, as the CodeReview styles are already using this!

Version: 1.17.x
Severity: major

Details

Reference
bz26131

Event Timeline

• bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:22 PM
• bzimport added a project: MediaWiki-ResourceLoader.
• bzimport set Reference to bz26131.
HappyDog created this task.Nov 26 2010, 7:26 PM
HappyDog added a comment.Nov 26 2010, 7:36 PM
  • Bug 26128 has been marked as a duplicate of this bug. ***
HappyDog added a comment.Nov 26 2010, 7:37 PM

(From bug 26128, comment #1, by Aaron Schulz)

Probably caused by r75331.

Catrope added a comment.Nov 30 2010, 2:41 PM

We should force user scripts and styles to be the absolute last thing on the page; apparently we're not doing a good enough job of that right now.

• bzimport added a comment.Dec 1 2010, 5:55 PM

a.d.bergi wrote:

Its the same with the liquid-thread-extension (tried at http://translatewiki.net). I needed to put a !important behind the "corrections".

• TrevorParscal added a comment.Dec 4 2010, 12:00 AM

r77693 resolves this by adding user and user.options scripts at the end of the body, and pushing any user* modules to the end of the head.

Catrope added a comment.Dec 24 2010, 1:30 AM

(In reply to comment #5)

r77693 resolves this by adding user and user.options scripts at the end of the
body, and pushing any user* modules to the end of the head.

This didn't actually work because dynamically added <style> tags were still being added below the <link> tag for user styles. Really fixed in r78943.

Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits