| stjn | |
| Mar 23 2024, 5:03 PM |
| F54255194: 2024-05-23_14-57-33.png | |
| Thu, May 23, 9:59 PM |
| F53412016: screenshot 353.png | |
| Thu, May 16, 1:36 AM |
| F49499190: screenshot 281.png | |
| Apr 30 2024, 7:08 PM |
| F49499189: screenshot 280.png | |
| Apr 30 2024, 7:08 PM |
Steps to replicate the issue (include links if applicable):
Compare:
Italic and smaller font is done by styles in Russian Wikipedia, but tag markers also lack parentheses () that are added via CSS. They should have them.
| AC | Status | Details |
|---|---|---|
| 1 | ✅ | T360843#9802724 |
| AC | Status | Details |
|---|---|---|
| 1 | ✅ | T360843#9827847 |
Change #1013700 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):
[mediawiki/core@master] Special:NewPages should generate markup consistent with other pager pages
Change #1015656 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] Style Special:NewPage consistently with Special:RecentChanges
Right now the NewPages HTML is inconsistent with other pages that look similar. There are two issues here that need to be fixed in core for this to be styled correctly in Minerva and consistent with other pages:
/** @var bool Whether to group items by date */ public $mGroupByDate = true;
Change #1013700 merged by jenkins-bot:
[mediawiki/core@master] Special:NewPages should generate markup consistent with other pager pages
Status: ❓
Environment: beta
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA
Test Artifact(s):
@Jdlrobson, the task description isn't very clear as to what I should be looking for. Please see the two images in beta. If these are a pass feel free to move to ready for signoff and resolve, also please leave a comment/screenshot annotation that clarifies what we should be looking for specifically when we test this in production.
Seems like tag markers are still displayed without parentheses, @Jdlrobson are you sure it is resolved?
Thanks for the ping! I was mistaken. Looks like we fixed the markup but not the styles:
@jsn.sherman will review https://gerrit.wikimedia.org/r/c/mediawiki/skins/MinervaNeue/+/1015656 shortly
ack, removing the deploy tag to make clearer that this wasn't resolved in that release.
Thanks @Jdlrobson, from the comments it looks like this will need to be retested again when the new patch is deployed in beta. I will remove the Verified tag and we will remove it from the deployment ticket table.
Change #1015656 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Style Special:NewPage consistently with Special:RecentChanges
Sneak preview on https://en.m.wikipedia.beta.wmflabs.org/wiki/Special:NewPages
Should be in production on 16th.
Hey @Edtadros when QAing - we're looking for clear markers for each piece of information e.g. a boundary such as "(" or a new line.
Tags can be seen at this URL (https://en.m.wikipedia.beta.wmflabs.org/wiki/Special:NewPages?namespace=all&tagfilter=mobile+edit&username=&size-mode=min&size=&wpFormIdentifier=newpagesform&minervanightmode=0)
Status: ✅ PASS
Environment: beta
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA
Test Artifact(s):
✅ AC1: There should be clear markers for each piece of information e.g. a boundary such as "(" or a new line.
Status: ✅ PASS
Environment: PROD
OS: macOS Sonoma 14.5
Browser: Chrome 125
Device: MBA
Emulated Device:NA
Test Artifact(s):
✅ AC1: There should be clear markers for each piece of information e.g. a boundary such as "(" or a new line.
