The structure data diff that displays @ https://commons.m.wikimedia.org/wiki/Special:MobileDiff/388741403 is confusing.
There might be two bugs here but I want to focus on the lead paragraph issue. I suggest the Structured-Data-Backlog team also look at this diff and create a new task if necessary for anything other than the move paragraph issue I am describing here.
The diff can somewhat be reproduced on desktop using diff-type=inline parameter.
https://commons.wikimedia.org/w/index.php?title=File:%D0%A4%D0%BE%D1%82%D0%BE_%D0%90%D0%BB%D0%B8%D1%88%D0%B5%D1%80_%D0%A2%D0%B0%D1%88%D0%BC%D0%B0%D1%82%D0%BE%D0%B2_%D0%91%D0%BE%D0%B3%D0%B0%D1%80%D0%B4%D0%B8%D0%B5%D0%B2%D0%B8%D1%87.jpg&diff=388741403&diff-type=inline
Related to this, on https://en.m.wikipedia.beta.wmflabs.org/wiki/Special:MobileDiff/411429 the move paragraph doesn't seem to match the mocks in T197491 anymore.
Mock:
https://wikimedia.invisionapp.com/public/share/3HEQCYR25#/screens/266827752
Current behaviour:
Developer notes
I think the issue here is that the mediawiki.diff.styles module is loading and wasn't loaded before, so desktop and mobile styles are conflicting.
These rules (and possibly other) should be disabled on Minerva
Option 1) moved to skinStyles in core so that they are substituted by the Minerva ones; 2) override in Minerva:
.mw-diff-movedpara-right::after, .rtl .mw-diff-movedpara-left::after {
content: '↩';
}
.mw-diff-movedpara-left::after, .rtl .mw-diff-movedpara-right::after {
content: '↪';
}QA steps
https://en.m.wikipedia.beta.wmflabs.org/wiki/Special:MobileDiff/412207 should match the mock.
QA Results
| AC | Status | Details |
|---|---|---|
| 1 | ✅ | T243235#5841699 |
