Page MenuHomePhabricator

IP masking: Use temporary icon in Vector and Minerva skins
Closed, ResolvedPublic1 Estimated Story Points

Description

The Vector and Minerva skins need updating to use the correct IP masked icon (added in T338165). We should update it.

Screenshot 2023-06-29 at 1.30.21 PM.png (145×1 px, 27 KB)

TODO

https://phabricator.wikimedia.org/T340789#9039700

QA Results - Beta

ACStatusDetails
1T340789#9039700
2T340789#9039700

Event Timeline

LGoto set the point value for this task to 1.Jul 6 2023, 5:34 PM

Change 940235 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Vector@master] Add temporary user icon

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

Change 940237 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] Display temporary user icon

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

Change 940236 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/MinervaNeue@master] Add temporary user icon

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

Change 940235 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Add temporary user icon

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

Change 940263 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Vector@master] Remove anonymous user icon

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

Change 940237 merged by jenkins-bot:

[mediawiki/core@master] Display temporary user icon

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

Change 940263 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Remove anonymous user icon

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

Change 940236 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Add temporary user icon

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

Test wiki created on Patch demo by Jdlrobson using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/67c6ac3826/w

Test Result - Beta

Status: ✅ PASS
Environment: beta
OS: macOS Ventura
Browser: Chrome
Device: MBP
Emulated Device:NA

Test Artifact(s):

QA Steps

The new IP masked user icon (below) should be used in Vector and Minerva

image.png (20×20 px, 3 KB)

✅ AC1: Vector

Screenshot 2023-07-24 at 4.58.31 PM.png (331×995 px, 50 KB)

✅ AC2: Minerva
I couldn't get the skin to switch to minerva using the param, but this looks good in mobile.

Screenshot 2023-07-24 at 4.59.17 PM.png (504×993 px, 56 KB)

Edtadros updated the task description. (Show Details)
Edtadros subscribed.

Looks good, resolving.

Test wiki on Patch demo by Jdlrobson using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/67c6ac3826/w/