Move mediawiki.special.js to a separate module
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	• Spage
	Apr 11 2013, 7:23 PM

Description

The 'mediawiki.special' module defines

'scripts' => 'resources/mediawiki.special/mediawiki.special.js',
'styles' => 'resources/mediawiki.special/mediawiki.special.css',

but several special pages call only

$out->addModuleStyles( 'mediawiki.special' );

ignoring the JS. mediawiki.special.js currently only defines an mediaWiki.special namespace, so this is not causing any harm. Conversely, Special:RecentChanges does not addModuleStyles( 'mediawiki.special' ) and instead mentions it as a dependency of its script module, so it does not load mediawiki.special.css if JS is disabled; RecentPages doesn't seem to use any of the styles in mediawiki.special.css so again no harm.

I noticed this while researching how code handles addModule{s,Scripts,Styles}. Roan suggested that if code has CSS that should load when JS is disabled then it should be in a separate module to avoid this confusion, I don't see a naming convention for this (a '_nojs' suffix perhaps?).

Version: 1.22.0
Severity: minor

Details

Reference: bz47134

Related Objects

Mentioned Here: rMWc740604ad3e7: mediawiki.special: Remove unused mediawiki.special.js
T92747: Style modules should be considered "ready" when they are applied to a page

Event Timeline

• bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:30 AM

• bzimport added a project: MediaWiki-ResourceLoader.

• bzimport set Reference to bz47134.

• bzimport added a subscriber: Unknown Object (MLST).

• Spage created this task.Apr 11 2013, 7:23 PM

./includes/specials/SpecialStatistics.php:		$this->getOutput()->addModuleStyles( 'mediawiki.special' );
./includes/specials/SpecialPrefixindex.php:		$out->addModuleStyles( 'mediawiki.special' );
./includes/specials/SpecialBlockList.php:		$out->addModuleStyles( 'mediawiki.special' );
./includes/specials/SpecialEmailuser.php:		$out->addModuleStyles( 'mediawiki.special' );
./includes/specials/SpecialAllPages.php:		$out->addModuleStyles( 'mediawiki.special' );
./includes/specials/SpecialListgrouprights.php:		$out->addModuleStyles( 'mediawiki.special' );
./includes/specials/SpecialSpecialpages.php:		$out->addModuleStyles( 'mediawiki.special' );
./includes/specials/SpecialProtectedpages.php:		$this->getOutput()->addModuleStyles( 'mediawiki.special' );
./includes/specials/SpecialUserrights.php:		$out->addModuleStyles( 'mediawiki.special' );
./includes/specials/SpecialAllMessages.php:		$out->addModuleStyles( 'mediawiki.special' );
./includes/specials/SpecialContributions.php:		$out->addModuleStyles( 'mediawiki.special' );

There is no bug that I can see, though there is technical debt.

We never load "mediawiki.special" as a module, only as a stylesheet. So this is consistent.

The mediawiki.special.js file is for dependent modules, such as "mediawiki.special.recentchanges". This would likely cause the stylesheet to load a second time (T92747).

Krinkle moved this task from Inbox to Backlog on the MediaWiki-ResourceLoader board.Apr 28 2015, 6:39 PM

Krinkle moved this task from Backlog to Accepted Enhancement on the MediaWiki-ResourceLoader board.

Danny_B moved this task from Unsorted to Move on the Technical-Debt board.Jan 23 2016, 12:18 AM

Module no longer exists as of rMWc740604ad3e7: mediawiki.special: Remove unused mediawiki.special.js.

Krinkle moved this task from Accepted Enhancement to Assigned on the MediaWiki-ResourceLoader board.Aug 19 2016, 6:35 AM

Krinkle added a project: Performance-Team.

Move mediawiki.special.js to a separate moduleClosed, ResolvedPublicActions

Description

Details

Related Objects

Event Timeline

Move mediawiki.special.js to a separate module
Closed, ResolvedPublic
Actions