Page MenuHomePhabricator

Timeless: Use classes for content styles instead of ids (issues with specificity and consistency with other skins)
Closed, ResolvedPublic

Description

See here, please:
https://ru.wikipedia.org/wiki/Участник:Saint_Johann/Мастерская?useskin=timeless

No other skin does style elements like #mw-content h1 (one ID style equals 255 classes that need to override it). Please move those styles to a class, like .mw-body-content, because supporting Timeless would be extremely costly to others otherwise.

(Also, it is not the best idea to add your own styles to the MediaWiki classes, but that is not the topic of this task.)

Event Timeline

Isarra renamed this task from Timeless skin has major problems with CSS specificity to Timeless: Use classes for content styles instead of ids (issues with specificity and consistency with other skins).Apr 19 2018, 4:23 PM

Ideally should all be consistent with mediawiki.skinning stuff in core.

Change 430281 had a related patch set uploaded (by Isarra; owner: Isarra):
[mediawiki/skins/Timeless@master] Make misc content styles selector less specific

https://gerrit.wikimedia.org/r/430281

Change 430281 merged by jenkins-bot:
[mediawiki/skins/Timeless@master] Make misc content styles selector less specific

https://gerrit.wikimedia.org/r/430281

Isarra claimed this task.

If this isn't actually fully resolved please yell at me.