On Wed, Feb 6, 2013 at 10:24 AM, Jon Robson <[email protected]> wrote:

> Forwarding to wikitech with RFC:
>
> Currently the mobile site scrubs an elements with the noprint class.
> It also scrubs elements with the nomobile class. Each "scrub" of
> element effects performance [citation needed] so this would be a good
> thing to do.
>
> It has been suggested that we stop scrubbing .noprint elements and
> there is a patchset to do so:
> https://gerrit.wikimedia.org/r/#/c/43852/
>

I agree, we should remove the scrubbing of .noprint on mobile.

IIRC we originally scrubbed .noprint elements simply to keep the document
size down; using .nomobile is better since we can target things more
appropriately.

For instance, .noprint might be used around an animated GIF, with a set of
still images in a div that's shown only under the print stylesheet.
Scrubbing that .noprint section on mobile would be wrong, since the
animation will display fine on mobile.

Pratical consideration: some templates might want to add the 'nomobile'
class where they currently only use 'noprint'. But that's an easy fix and
can be done as they are noticed.

-- brion
_______________________________________________
Wikitech-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Reply via email to