Remove or regenerate PNG files for Translate
Closed, ResolvedPublic
Actions

Description

Per discussion from https://gerrit.wikimedia.org/r/#/c/380062/

The png files should be regenerated from the upgraded SVG files. However, these days basically all browsers that support JavaScript also support SVG. Hence, we can reduce our work by first doing an audit for which images only appear if JavaScript is enabled. This will both reduce further maintenance work as well as slim down the stylesheets.

I am adding the easy tag, even though this is not trivial:

First identify in which CSS/JS/PHP files the image is referenced
Those only mentioned in JS do not need PNG
Those mentioned in PHP likely need PNG
Those mentioned in CSS should be checked with Resources.php to find the resource modules
Those resource module names can be searched again to see on which pages they are used
Check those pages without JavaScript to see if some image appears without, those should keep PNG

I imagine most of these files are used on special pages such as Special:Translate and Special:AggregateGroups. Basic Translate setup is needed as documented in mediawiki.org

Details

	Subject	Repo	Branch	Lines +/-
	Remove search.png for Translate	mediawiki/extensions/Translate	master	+5 -7
	Remove or regenerate PNG files for Translate	mediawiki/extensions/Translate	master	+42 -56

Customize query in gerrit

Related Objects

Mentioned Here: T184070: Add "This page requires JavaScript" notes to special pages in the Translate extension
T179563: Lacking mixing for SVG only background images

Event Timeline

Nikerabbit created this task.Oct 18 2017, 11:26 AM

Restricted Application added subscribers: TerraCodes, Aklapper. · View Herald TranscriptOct 18 2017, 11:26 AM

This should be done in two independent steps:

First remove the PNG files
Then regenerate the remaining ones

There is some guidance how to generate an optimized PNG file from SVG.

@Nikerabbit Suitable GCI task?

Not the easiest task, but should be doable in GCI.

Aklapper added a project: Google-Code-in-2017.Oct 23 2017, 2:29 PM

Imported as https://codein.withgoogle.com/tasks/5663788328026112/

Nikerabbit triaged this task as Low priority.Nov 30 2017, 2:18 PM

eflyjason subscribed.Dec 31 2017, 9:26 AM

Framawiki assigned this task to eflyjason.Dec 31 2017, 2:09 PM

Framawiki subscribed.

Change 401658 had a related patch set uploaded (by Eflyjason; owner: Eflyjason):
[mediawiki/extensions/Translate@master] Remove or regenerate PNG files for Translate

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

gerritbot added a project: Patch-For-Review.Jan 3 2018, 3:01 AM

We have to remove the .png reference from the .less/.css files right?

Yes those must be removed. I filed T179563: Lacking mixing for SVG only background images earlier to clarify how to do this. It is not yet resolved, but I suggest we just standardize on this code when PNG is not used:

LESS files: .background-image( '....svg' );
CSS files: background-image: /* @embed */ url( ....svg );

In some cases this means taking the url out of background: rule.

Change 401658 merged by jenkins-bot:
[mediawiki/extensions/Translate@master] Remove or regenerate PNG files for Translate

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

Nikerabbit closed this task as Resolved.Jan 4 2018, 10:38 AM

Nikerabbit removed a project: Patch-For-Review.

ReleaseTaggerBot added a project: MW-1.31-release-notes (WMF-deploy-2018-01-09 (1.31.0-wmf.16)).Jan 4 2018, 11:01 AM

We can remove search.png too after T184070: Add "This page requires JavaScript" notes to special pages in the Translate extension :)