Visit https://en.m.wikipedia.org/wiki/Leonard_Cohen?oldid=747517267
The box in the top says that the last modifier is Linealchamp.
The last modified bar at the end disagrees.
It shows the current editor too.
We should hide the bar in the footer which gives information that is not relevant (not appropriate) here.
Acceptance criteria
- If a page is an old revision and I view it there is no last modified bar
- If a page is a current revision and I view it there is a last modified bar.
- MinervaNeue should not use MobilePage class - use methods in core.
Developer notes
- MinervaNeue uses https://github.com/wikimedia/mediawiki-extensions-MobileFrontend/blob/master/includes/models/MobilePage.php. It might be good to disentangle the dependency here as part of this change as relying on this means Minerva depends on MobileFrontend.
- Rely on internal Revision methods rather than query strings parameters.
- Take the time to review the codebase and refactor so we consistently check if the page is a revision
Testing notes
- Go to any article on BetaCluster that has more than one edit, example: https://en.m.wikipedia.beta.wmflabs.org/wiki/Planet
- The footer should contain "Last edited {TIME} ago by {AUTHOR}
- click "Last edited" link to see the article history page
- on history page click the previous edit (not the latest one)
- you should land on page saying "This is an old revision of this page ....."
- when you scroll down to the footer "Last edited {TIME} ago by {AUTHOR}" should not be visible because you're viewing the historical page