Page MenuHomePhabricator

Extend mw-config's logos management system to also cover wordmarks (wmgSiteLogoWordmark)
Closed, ResolvedPublic

Description

Commits like https://gerrit.wikimedia.org/r/c/operations/mediawiki-config/+/789613 shouldn't be so manual, and should get auto-crushed so that the SVGs aren't bloated. Can we extend logos/manage.py to handle these?

Event Timeline

And we have some bloated SVGs get merged recently (7ce1a48).

Good idea. So ideally wordmarks and taglines should be firstly uploaded to commons, then defined in some yaml file like logos/config.yaml. Patch uploader should run some script - possibly logos/manage.py to download them, adjust to proper size (tagline should have a height smaller than 50px, and the height of wordmark should smaller than 30px).

Change 829298 had a related patch set uploaded (by Stang; author: Stang):

[operations/mediawiki-config@master] logos: Cover wordmark/tagline in manage.py

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

Change 829561 had a related patch set uploaded (by Stang; author: Stang):

[operations/mediawiki-config@master] Replace wordmark/tagline with correct naming style

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

Change 829563 had a related patch set uploaded (by Stang; author: Stang):

[operations/mediawiki-config@master] Move wmgSiteLogoWordmark and wmgSiteLogoTagline to logos.php

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

Change 829760 had a related patch set uploaded (by Stang; author: Stang):

[operations/mediawiki-config@master] Re-download and optimize wordmark/tagline svg file

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

Change 829764 had a related patch set uploaded (by Stang; author: Stang):

[operations/mediawiki-config@master] Drop unused wordmark/tagline

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

Change 829298 merged by jenkins-bot:

[operations/mediawiki-config@master] logos: Cover wordmark/tagline in manage.py

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

Change 829561 merged by jenkins-bot:

[operations/mediawiki-config@master] Replace wordmark/tagline with correct naming style

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

Mentioned in SAL (#wikimedia-operations) [2022-10-12T09:21:21Z] <urbanecm@deploy1002> Started scap: Backport for [[gerrit:829561|Replace wordmark/tagline with correct naming style (T307705)]]

Mentioned in SAL (#wikimedia-operations) [2022-10-12T09:21:44Z] <urbanecm@deploy1002> urbanecm and stang: Backport for [[gerrit:829561|Replace wordmark/tagline with correct naming style (T307705)]] synced to the testservers: mwdebug2002.codfw.wmnet, mwdebug1002.eqiad.wmnet, mwdebug2001.codfw.wmnet, mwdebug1001.eqiad.wmnet

Mentioned in SAL (#wikimedia-operations) [2022-10-12T09:25:41Z] <urbanecm@deploy1002> Finished scap: Backport for [[gerrit:829561|Replace wordmark/tagline with correct naming style (T307705)]] (duration: 04m 20s)

Thanks for your work on this @Stang. I merged the manage.py change (and the add-only patch of the SVG files). Can you please schedule the remaining patches for a B&C window? I can deploy them all by myself, but I'd appreciate help with checking the icons don't break.

Also, I didn't want to block CR on this, but I think the logos/README.md should be updated more to reflect the newly-added feature. It should at least mention the newly-existing fields in config.yaml, and perhaps mention that running tox -e logos -- update xxwiki will take care of both logos. Can you have a look at this please @Stang?

Thanks a lot! I will resolve the merge conflict and polish the document.

Change 829760 abandoned by Stang:

[operations/mediawiki-config@master] Re-download and optimize wordmark/tagline svg file

Reason:

already resolved

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

Change 829563 merged by jenkins-bot:

[operations/mediawiki-config@master] Move wmgSiteLogoWordmark and wmgSiteLogoTagline to logos.php

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

Mentioned in SAL (#wikimedia-operations) [2022-10-12T13:14:00Z] <urbanecm@deploy1002> Started scap: Backport for [[gerrit:829563|Move wmgSiteLogoWordmark and wmgSiteLogoTagline to logos.php (T307705)]]

Mentioned in SAL (#wikimedia-operations) [2022-10-12T13:14:23Z] <urbanecm@deploy1002> urbanecm and stang: Backport for [[gerrit:829563|Move wmgSiteLogoWordmark and wmgSiteLogoTagline to logos.php (T307705)]] synced to the testservers: mwdebug1002.eqiad.wmnet, mwdebug2002.codfw.wmnet, mwdebug2001.codfw.wmnet, mwdebug1001.eqiad.wmnet

Mentioned in SAL (#wikimedia-operations) [2022-10-12T13:21:07Z] <urbanecm@deploy1002> Finished scap: Backport for [[gerrit:829563|Move wmgSiteLogoWordmark and wmgSiteLogoTagline to logos.php (T307705)]] (duration: 07m 06s)

Change 841942 had a related patch set uploaded (by Stang; author: Stang):

[operations/mediawiki-config@master] Re-download and optimize wordmark/tagline svg file

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

Change 829764 merged by jenkins-bot:

[operations/mediawiki-config@master] Drop unused wordmark/tagline

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

Change 841942 merged by jenkins-bot:

[operations/mediawiki-config@master] Re-download and optimize wordmark/tagline svg file

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

Mentioned in SAL (#wikimedia-operations) [2022-10-12T20:16:34Z] <samtar@deploy1002> Started scap: Backport for [[gerrit:829764|Drop unused wordmark/tagline (T307705)]], [[gerrit:841942|Re-download and optimize wordmark/tagline svg file (T307705)]]

Mentioned in SAL (#wikimedia-operations) [2022-10-12T20:16:57Z] <samtar@deploy1002> samtar and stang: Backport for [[gerrit:829764|Drop unused wordmark/tagline (T307705)]], [[gerrit:841942|Re-download and optimize wordmark/tagline svg file (T307705)]] synced to the testservers: mwdebug1001.eqiad.wmnet, mwdebug2001.codfw.wmnet, mwdebug2002.codfw.wmnet, mwdebug1002.eqiad.wmnet

Mentioned in SAL (#wikimedia-operations) [2022-10-12T20:21:27Z] <samtar@deploy1002> Finished scap: Backport for [[gerrit:829764|Drop unused wordmark/tagline (T307705)]], [[gerrit:841942|Re-download and optimize wordmark/tagline svg file (T307705)]] (duration: 04m 53s)

Change 842010 had a related patch set uploaded (by Stang; author: Stang):

[operations/mediawiki-config@master] logos: Document how to update wordmark/tagline via manage.py

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

Change 842010 merged by jenkins-bot:

[operations/mediawiki-config@master] logos: Document how to update wordmark/tagline via manage.py

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

Thanks @Stang for working on this!

Change 842923 had a related patch set uploaded (by Stang; author: Stang):

[operations/mediawiki-config@master] logos: Fix bugs when a variant param is specific

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

Change 842923 merged by jenkins-bot:

[operations/mediawiki-config@master] logos: Fix bug when a variant param is specific

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

Change 843484 had a related patch set uploaded (by Stang; author: Stang):

[operations/mediawiki-config@master] logos: Set a higher precision for scour

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

Change 843484 merged by jenkins-bot:

[operations/mediawiki-config@master] logos: Set a higher precision for scour

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

Change 857030 had a related patch set uploaded (by Stang; author: Stang):

[operations/mediawiki-config@master] logos: Remove duplicated code

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

Change 857030 merged by jenkins-bot:

[operations/mediawiki-config@master] logos: Remove duplicated code

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

Mentioned in SAL (#wikimedia-operations) [2022-11-15T21:14:40Z] <cjming@deploy1002> Started scap: Backport for [[gerrit:857030|logos: Remove duplicated code (T307705)]]

Mentioned in SAL (#wikimedia-operations) [2022-11-15T21:15:04Z] <cjming@deploy1002> cjming and stang: Backport for [[gerrit:857030|logos: Remove duplicated code (T307705)]] synced to the testservers: mwdebug1001.eqiad.wmnet, mwdebug1002.eqiad.wmnet, mwdebug2001.codfw.wmnet, mwdebug2002.codfw.wmnet

Mentioned in SAL (#wikimedia-operations) [2022-11-15T21:19:11Z] <cjming@deploy1002> Finished scap: Backport for [[gerrit:857030|logos: Remove duplicated code (T307705)]] (duration: 04m 31s)