The Wayback Machine - https://web.archive.org/web/20210906193321/https://phabricator.wikimedia.org/T18049
Page MenuHomePhabricator

Make CodeReview stylesheet URL user editable
Closed, ResolvedPublic

Description

Add editable stylesheet (MediaWiki:CodeReview.css) instead of linking hardcoded one like eg. GeSHi does.


Version: unspecified
Severity: enhancement

Details

Reference
bz16049

Event Timeline

bzimport raised the priority of this task from to Lowest.Nov 21 2014, 10:22 PM2014-11-21 22:22:33 (UTC+0)
bzimport set Reference to bz16049.
bzimport added a subscriber: Unknown Object (MLST).
Comment Actions

MediaWiki:common.css and the skin/user css files are not OK?

Comment Actions

(In reply to comment #1)

MediaWiki:common.css and the skin/user css files are not OK?

No. Why should I send two complete stylesheets to the user when I want completely other design? Worthless and unnecessary mess and load.

Comment Actions

(In reply to comment #1)

MediaWiki:common.css and the skin/user css files are not OK?

No; we separate style sheet files just for this reason: style sheet belonging to an extension should be separate from that for the MediaWiki proper.

(In reply to comment #0)

Add editable stylesheet (MediaWiki:CodeReview.css) instead of linking hardcoded
one like eg. GeSHi does.

I don't think it should be used "instead of" a hardcoded one. The original stylesheet can still be hardcoded, while a custom one in MediaWiki namespace can overwrite its definitions.

Comment Actions

This doesn't seem to do what was requested in the bug -- the OP asked for an on-wiki editable CSS page, like the GeSHI syntax highlighting extension has.

This is providing a LocalSettings.php-level override for the location of the extensions directory where the CSS and JS files are to be loaded... which shouldn't be needed, and if it were it should be a general system for all extensions to be able to use (eg to pull extension styles from a static server, the way we can override the wiki's core style path)

Reverted in r44459.

Comment Actions

Finally fixed in gerrit change 76045.

Morty Proxy This is a proxified and sanitized view of the page, visit original site.