HomePhabricator

Hide marked empty elements by default (stage 1)

Authored by tstarling on Jul 13 2016, 2:01 AM.

Description

Hide marked empty elements by default (stage 1)

We originally imagined rolling out the display of empty elements
simultaneously with the Html5Depurate, but now we have added support for
marking empty elements to Html5Depurate and plan on having some sort of
longer migration period. So, move the relevant CSS to content.css, and
remove the concept of CSS dependant on tidy driver.

Add a body class which will allow the effect to be toggled in a gadget or
extension. Actual toggling in the CSS will be in the stage 2 patch, to be
deployed after the varnish cache and parser cache have expired.

I originally imagined that there would be a gadget that overrides the
rule with an !important selector, but that method does not allow you to
recover the original display property, which is often overridden by the
style attribute or site CSS to be "inline".

Also, in RaggettWrapper, switch to the new class mw-empty-elt, following
Html5Depurate, instead of mw-empty-li. The old class will be removed in
the stage 2 patch.

Change-Id: Ic0f432c43a006629ca5a1a7c2dda3552ceb4dc4f

Details

Committed
LegoktmJul 14 2016, 9:24 PM
Parents
rMW7dd1914e734c: Merge "Don't use $wgContentNamespaces directly"
Branches
Unknown
Tags
Unknown
References
refs/changes/13/295613/6
ChangeId
Ic0f432c43a006629ca5a1a7c2dda3552ceb4dc4f