Page MenuHomePhabricator

Enable templatestyles in Modern for "document" rules
Open, Needs TriagePublic

Description

There are a batch of rules in Modern main.css starting with #content a.external { and continuing to

#content a.external[ href$='.pdf' ],
#content a.external[ href$='.PDF' ],
#content a.external[ href*='.pdf#' ],
#content a.external[ href*='.PDF#' ],
#content a.external[ href*='.pdf?' ],
#content a.external[ href*='.PDF?' ],
.link-document {
	/* @embed */
	background: url( images/document.png ) center right no-repeat;
	padding-right: 12px;
}

which should probably reference .mw-parser-output instead of #content, so that TemplateStyles can reasonably override the default styling. All/the majority of the other skins had this change made. It should be backward compatible with wiki-defined values simply to swap #content 1:1 with .mw-parser-output, as the wiki-defined values will still be more specific.

Discussion arose in the context of Help talk:CS1.

Event Timeline

Izno created this task.Jun 10 2019, 2:29 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 10 2019, 2:29 PM
Izno updated the task description. (Show Details)Jun 10 2019, 2:31 PM
Tgr moved this task from Up next to External on the TemplateStyles board.Jun 10 2019, 5:49 PM
MJL added a subscriber: MJL.Jun 20 2019, 6:08 AM
Aklapper updated the task description. (Show Details)May 10 2020, 1:30 PM
Izno updated the task description. (Show Details)Jun 20 2020, 2:56 PM
Izno updated the task description. (Show Details)
Izno updated the task description. (Show Details)Jun 20 2020, 3:01 PM