Page MenuHomePhabricator

Improve list styles on mobile Special:RecentChanges
Closed, ResolvedPublic5 Estimated Story Points


As assessed in T224432, the list styles in Minerva for the Special:RecentChanges page need to be modified to ensure the list of results is mobile friendly.

Current mockups for list items

image.png (1×395 px, 134 KB)

Technical considerations

Summarizing T224432#5336883 and T224432#5343460, The Special:RecentChanges list shares similarities with the action=history page but there are also some differences. The structure of the <ul><li> elements differ between the two pages, but the inline elements like .mw-changeslist-links .mw-diff-bytes are the same.

Common styles should be factored out of the history page and placed into a new SkinStyles module called mediawiki.special.changeslist (this will override this module in core).

For styling the distinct <ul><li> elements, we have existing mixins in minerva.mixins.less that can be used and expanded upon for this purpose (e.g. .changeslist-row(), .changeslist-row-item() ). These RecentChanges specific styles should be placed in a new RL module.

Event Timeline

ovasileva moved this task from Incoming to Upcoming on the Readers-Web-Backlog board.

Change 526130 had a related patch set uploaded (by Jdrewniak; owner: Jdrewniak):
[mediawiki/skins/MinervaNeue@master] Adding skin styles for the changeslist module.

Change 526130 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Adding skin styles for the changeslist module.

Edtadros added a subscriber: Edtadros.

@Jdrewniak let me know where to test this. I assume I just need to make sure the Recent Changes page looks like the screenshot in the description.

@Edtadros I think @alexhollender pretty much has the QA for this task covered in T229360, so I think I'll reassign this task to him and put it in sign-off.
There has been one bug identified with the list styles in that task, so I think once we take care of that, this task can be signed off.